>-----Original Message----- >From: Jiang Liu [mailto:jiang.liu@xxxxxxxxxxxxxxx] >Sent: Monday, November 03, 2014 6:14 PM >To: Neela Syam Kolli; James E.J. Bottomley >Cc: Jiang Liu; Alexander Gordeev; linux-pci@xxxxxxxxxxxxxxx; linux- >scsi@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx >Subject: [PATCH] megaraid_sas : Fix bug in handling return value of >pci_enable_msix_range() > >Function pci_enable_msix_range() may return negative values for error >conditions. So it's a bug by checking (pci_enable_msix_range() != 0) for >success and causes failure to megaraid driver when MSI is disabled. >[ 16.487267] megaraid_sas 0000:02:00.0: Controller type: iMR >[ 16.487275] genirq: Flags mismatch irq 0. 00000000 (megasas) vs. 00015a00 (tii >mer) >[ 16.487347] megasas: Failed to register IRQ for vector 0. > >Fixes: 8ae80ed1734b "megaraid: Use pci_enable_msix_range() instead of >pci_enable_msix()" > >Signed-off-by: Jiang Liu <jiang.liu@xxxxxxxxxxxxxxx> >Cc: Alexander Gordeev <agordeev@xxxxxxxxxx> >Cc: "Bjorn Helgaas <bhelgaas@xxxxxxxxxx>" >Cc: linux-pci@xxxxxxxxxxxxxxx >Cc: <stable@xxxxxxxxxxxxxxx> # 3.17 >--- > drivers/scsi/megaraid/megaraid_sas_base.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c >b/drivers/scsi/megaraid/megaraid_sas_base.c >index f6a69a3b1b3f..5640ad1c8214 100644 >--- a/drivers/scsi/megaraid/megaraid_sas_base.c >+++ b/drivers/scsi/megaraid/megaraid_sas_base.c >@@ -4453,7 +4453,7 @@ static int megasas_init_fw(struct megasas_instance >*instance) > instance->msixentry[i].entry = i; > i = pci_enable_msix_range(instance->pdev, instance- >>msixentry, > 1, instance->msix_vectors); >- if (i) >+ if (i > 0) > instance->msix_vectors = i; > else > instance->msix_vectors = 0; Acked-by: Sumit Saxena <sumit.saxena@xxxxxxxxxxxxx> >-- >1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
