diff options
author | Alexander Gordeev <agordeev@redhat.com> | 2014-02-14 14:27:19 -0700 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2014-02-14 14:27:19 -0700 |
commit | fc40363b2140f5777c88e67814fac9327bf1ee68 (patch) | |
tree | 748b4fad10579c2e35c05cd550a71a5bd6f3b6ea /drivers/ata | |
parent | 1f42db786b14a31bf807fc41ee5583a00c08fcb1 (diff) | |
download | op-kernel-dev-fc40363b2140f5777c88e67814fac9327bf1ee68.zip op-kernel-dev-fc40363b2140f5777c88e67814fac9327bf1ee68.tar.gz |
ahci: Fix broken fallback to single MSI mode
Commit 7b92b4f61ec4 ("PCI/MSI: Remove pci_enable_msi_block_auto()")
introduced a regression: if multiple MSI initialization fails, the code
falls back to INTx rather than to single MSI.
Fixes: 7b92b4f61ec4 ("PCI/MSI: Remove pci_enable_msi_block_auto()")
Signed-off-by: Alexander Gordeev <agordeev@redhat.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers/ata')
-rw-r--r-- | drivers/ata/ahci.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c index dc2756f..3c5f35e 100644 --- a/drivers/ata/ahci.c +++ b/drivers/ata/ahci.c @@ -1170,8 +1170,10 @@ static int ahci_init_interrupts(struct pci_dev *pdev, unsigned int n_ports, nvec = rc; rc = pci_enable_msi_block(pdev, nvec); - if (rc) + if (rc < 0) goto intx; + else if (rc > 0) + goto single_msi; return nvec; |