Re: [Patch V2 3/9] [SCSI] aacraid: Enable MSI interrupt for series-6 controller

2015-06-25 Thread Tomas Henzl
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

2015-06-23 Thread Mahesh Rajashekhara
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

2015-06-16 Thread Johannes Thumshirn
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

2015-06-10 Thread rajinikanth.pandurangan
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