Re: [Patch V2 3/9] [SCSI] aacraid: Enable MSI interrupt for series-6 controller
On 06/11/2015 03:42 AM, rajinikanth.panduran...@pmcs.com wrote: From: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com Description: Enable MSI interrupt mode for series-6 controller. What seem problematic to me is that you still keep the module option 'aac_msi' and a poor user might try IRQ handling. 0=PIC(default), 1=MSI, 2=MSI-X(unsupported, uses MSI)); to manipulate the driver naturally withou any success. Please either change the above description or add a new option so it is not confusing. Btw msi-x is now supported? Cheers, Tomas Signed-off-by: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com --- drivers/scsi/aacraid/src.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/aacraid/src.c b/drivers/scsi/aacraid/src.c index b147341..eb07b3d 100644 --- a/drivers/scsi/aacraid/src.c +++ b/drivers/scsi/aacraid/src.c @@ -742,7 +742,7 @@ int aac_src_init(struct aac_dev *dev) if (dev-comm_interface != AAC_COMM_MESSAGE_TYPE1) goto error_iounmap; - dev-msi = aac_msi !pci_enable_msi(dev-pdev); + dev-msi = !pci_enable_msi(dev-pdev); dev-aac_msix[0].vector_no = 0; dev-aac_msix[0].dev = dev; -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [Patch V2 3/9] [SCSI] aacraid: Enable MSI interrupt for series-6 controller
Reviewed-by: Mahesh Rajashekhara mahesh.rajashekh...@pmcs.com -Original Message- From: Rajinikanth Pandurangan Sent: Thursday, June 11, 2015 7:12 AM To: jbottom...@parallels.com; linux-scsi@vger.kernel.org Cc: aacr...@pmc-sierra.com; Harry Yang; Mahesh Rajashekhara; Rich Bono; Achim Leubner; Murthy Bhat; Rajinikanth Pandurangan Subject: [Patch V2 3/9] [SCSI] aacraid: Enable MSI interrupt for series-6 controller From: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com Description: Enable MSI interrupt mode for series-6 controller. Signed-off-by: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com --- drivers/scsi/aacraid/src.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/aacraid/src.c b/drivers/scsi/aacraid/src.c index b147341..eb07b3d 100644 --- a/drivers/scsi/aacraid/src.c +++ b/drivers/scsi/aacraid/src.c @@ -742,7 +742,7 @@ int aac_src_init(struct aac_dev *dev) if (dev-comm_interface != AAC_COMM_MESSAGE_TYPE1) goto error_iounmap; - dev-msi = aac_msi !pci_enable_msi(dev-pdev); + dev-msi = !pci_enable_msi(dev-pdev); dev-aac_msix[0].vector_no = 0; dev-aac_msix[0].dev = dev; -- 1.9.3 -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [Patch V2 3/9] [SCSI] aacraid: Enable MSI interrupt for series-6 controller
On Wed, Jun 10, 2015 at 06:42:25PM -0700, rajinikanth.panduran...@pmcs.com wrote: From: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com Description: Enable MSI interrupt mode for series-6 controller. Signed-off-by: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com --- drivers/scsi/aacraid/src.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/aacraid/src.c b/drivers/scsi/aacraid/src.c index b147341..eb07b3d 100644 --- a/drivers/scsi/aacraid/src.c +++ b/drivers/scsi/aacraid/src.c @@ -742,7 +742,7 @@ int aac_src_init(struct aac_dev *dev) if (dev-comm_interface != AAC_COMM_MESSAGE_TYPE1) goto error_iounmap; - dev-msi = aac_msi !pci_enable_msi(dev-pdev); + dev-msi = !pci_enable_msi(dev-pdev); dev-aac_msix[0].vector_no = 0; dev-aac_msix[0].dev = dev; -- 1.9.3 -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Reviewed-by: Johannes Thumshirn jthumsh...@suse.de -- Johannes Thumshirn Storage jthumsh...@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[Patch V2 3/9] [SCSI] aacraid: Enable MSI interrupt for series-6 controller
From: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com Description: Enable MSI interrupt mode for series-6 controller. Signed-off-by: Rajinikanth Pandurangan rajinikanth.panduran...@pmcs.com --- drivers/scsi/aacraid/src.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/aacraid/src.c b/drivers/scsi/aacraid/src.c index b147341..eb07b3d 100644 --- a/drivers/scsi/aacraid/src.c +++ b/drivers/scsi/aacraid/src.c @@ -742,7 +742,7 @@ int aac_src_init(struct aac_dev *dev) if (dev-comm_interface != AAC_COMM_MESSAGE_TYPE1) goto error_iounmap; - dev-msi = aac_msi !pci_enable_msi(dev-pdev); + dev-msi = !pci_enable_msi(dev-pdev); dev-aac_msix[0].vector_no = 0; dev-aac_msix[0].dev = dev; -- 1.9.3 -- To unsubscribe from this list: send the line unsubscribe linux-scsi in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html