All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
@ 2019-03-08  7:26 ` sagar.biradar
  0 siblings, 0 replies; 6+ messages in thread
From: sagar.biradar @ 2019-03-08  7:26 UTC (permalink / raw)
  To: martin.petersen, jejb
  Cc: linux-scsi, aacraid, dave.carroll, scott.benesh, stable, sagar.biradar

From: Sagar Biradar <sagar.biradar@microchip.com>

To: Martin K. Petersen <martin.petersen@oracle.com>
To: James Bottomley <jejb@linux.vnet.ibm.com>
cc: linux-scsi <linux-scsi@vger.kernel.org>
cc: aacraid@microsemi.com
cc: Dave Carroll <dave.carroll@microchip.com>
cc: Scott Benesh <scott.benesh@microchip.com>
cc: stable@vger.kernel.org

Fix performance issue where the queue depth for SmartIOC logical
volumes is set to 1, and allow the usual logical volume code
to be executed

Fixes: a052865fe2871a3888 (aacraid: Set correct Queue Depth for HBA1000
RAW disks)

Signed-off-by: Sagar Biradar <Sagar.Biradar@microchip.com>
---
 drivers/scsi/aacraid/linit.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/drivers/scsi/aacraid/linit.c b/drivers/scsi/aacraid/linit.c
index 7e56a11..cceffac 100644
--- a/drivers/scsi/aacraid/linit.c
+++ b/drivers/scsi/aacraid/linit.c
@@ -413,13 +413,16 @@ static int aac_slave_configure(struct scsi_device *sdev)
 	if (chn < AAC_MAX_BUSES && tid < AAC_MAX_TARGETS && aac->sa_firmware) {
 		devtype = aac->hba_map[chn][tid].devtype;
 
-		if (devtype == AAC_DEVTYPE_NATIVE_RAW)
+		if (devtype == AAC_DEVTYPE_NATIVE_RAW) {
 			depth = aac->hba_map[chn][tid].qd_limit;
-		else if (devtype == AAC_DEVTYPE_ARC_RAW)
+			set_timeout = 1;
+			goto common_config;
+		}
+		if (devtype == AAC_DEVTYPE_ARC_RAW) {
 			set_qd_dev_type = true;
-
-		set_timeout = 1;
-		goto common_config;
+			set_timeout = 1;
+			goto common_config;
+		}
 	}
 
 	if (aac->jbod && (sdev->type == TYPE_DISK))
-- 
1.8.3.1


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
@ 2019-03-08  7:26 ` sagar.biradar
  0 siblings, 0 replies; 6+ messages in thread
From: sagar.biradar @ 2019-03-08  7:26 UTC (permalink / raw)
  To: martin.petersen, jejb
  Cc: linux-scsi, aacraid, dave.carroll, scott.benesh, stable, sagar.biradar

From: Sagar Biradar <sagar.biradar@microchip.com>

To: Martin K. Petersen <martin.petersen@oracle.com>
To: James Bottomley <jejb@linux.vnet.ibm.com>
cc: linux-scsi <linux-scsi@vger.kernel.org>
cc: aacraid@microsemi.com
cc: Dave Carroll <dave.carroll@microchip.com>
cc: Scott Benesh <scott.benesh@microchip.com>
cc: stable@vger.kernel.org

Fix performance issue where the queue depth for SmartIOC logical
volumes is set to 1, and allow the usual logical volume code
to be executed

Fixes: a052865fe2871a3888 (aacraid: Set correct Queue Depth for HBA1000
RAW disks)

Signed-off-by: Sagar Biradar <Sagar.Biradar@microchip.com>
---
 drivers/scsi/aacraid/linit.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/drivers/scsi/aacraid/linit.c b/drivers/scsi/aacraid/linit.c
index 7e56a11..cceffac 100644
--- a/drivers/scsi/aacraid/linit.c
+++ b/drivers/scsi/aacraid/linit.c
@@ -413,13 +413,16 @@ static int aac_slave_configure(struct scsi_device *sdev)
 	if (chn < AAC_MAX_BUSES && tid < AAC_MAX_TARGETS && aac->sa_firmware) {
 		devtype = aac->hba_map[chn][tid].devtype;
 
-		if (devtype == AAC_DEVTYPE_NATIVE_RAW)
+		if (devtype == AAC_DEVTYPE_NATIVE_RAW) {
 			depth = aac->hba_map[chn][tid].qd_limit;
-		else if (devtype == AAC_DEVTYPE_ARC_RAW)
+			set_timeout = 1;
+			goto common_config;
+		}
+		if (devtype == AAC_DEVTYPE_ARC_RAW) {
 			set_qd_dev_type = true;
-
-		set_timeout = 1;
-		goto common_config;
+			set_timeout = 1;
+			goto common_config;
+		}
 	}
 
 	if (aac->jbod && (sdev->type == TYPE_DISK))
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
  2019-03-08  7:26 ` sagar.biradar
@ 2019-03-08 16:09   ` Dave.Carroll
  -1 siblings, 0 replies; 6+ messages in thread
From: Dave.Carroll @ 2019-03-08 16:09 UTC (permalink / raw)
  To: Sagar.Biradar, martin.petersen, jejb
  Cc: linux-scsi, aacraid, Scott.Benesh, stable, Sagar.Biradar

> -----Original Message-----
> From: sagar.biradar@microchip.com [mailto:sagar.biradar@microchip.com]
> Sent: Friday, March 08, 2019 12:27 AM
> To: martin.petersen@oracle.com; jejb@linux.vnet.ibm.com
> Cc: linux-scsi@vger.kernel.org; aacraid@microsemi.com; Dave Carroll -
> C33631 <Dave.Carroll@microchip.com>; Scott Benesh - C33703
> <Scott.Benesh@microchip.com>; stable@vger.kernel.org; Sagar Biradar -
> C34249 <Sagar.Biradar@microchip.com>
> Subject: [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
> 
> Fix performance issue where the queue depth for SmartIOC logical volumes is
> set to 1, and allow the usual logical volume code to be executed
> 
> Fixes: a052865fe2871a3888 (aacraid: Set correct Queue Depth for HBA1000
> RAW disks)
> 
> Signed-off-by: Sagar Biradar <Sagar.Biradar@microchip.com>
> ---
Reviewed-by: Dave Carroll <david.carroll@microsemi.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* RE: [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
@ 2019-03-08 16:09   ` Dave.Carroll
  0 siblings, 0 replies; 6+ messages in thread
From: Dave.Carroll @ 2019-03-08 16:09 UTC (permalink / raw)
  To: martin.petersen, jejb
  Cc: linux-scsi, aacraid, Scott.Benesh, stable, Sagar.Biradar

> -----Original Message-----
> From: sagar.biradar@microchip.com [mailto:sagar.biradar@microchip.com]
> Sent: Friday, March 08, 2019 12:27 AM
> To: martin.petersen@oracle.com; jejb@linux.vnet.ibm.com
> Cc: linux-scsi@vger.kernel.org; aacraid@microsemi.com; Dave Carroll -
> C33631 <Dave.Carroll@microchip.com>; Scott Benesh - C33703
> <Scott.Benesh@microchip.com>; stable@vger.kernel.org; Sagar Biradar -
> C34249 <Sagar.Biradar@microchip.com>
> Subject: [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
> 
> Fix performance issue where the queue depth for SmartIOC logical volumes is
> set to 1, and allow the usual logical volume code to be executed
> 
> Fixes: a052865fe2871a3888 (aacraid: Set correct Queue Depth for HBA1000
> RAW disks)
> 
> Signed-off-by: Sagar Biradar <Sagar.Biradar@microchip.com>
> ---
Reviewed-by: Dave Carroll <david.carroll@microsemi.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
  2019-03-08  7:26 ` sagar.biradar
@ 2019-03-14 11:12   ` Martin K. Petersen
  -1 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2019-03-14 11:12 UTC (permalink / raw)
  To: sagar.biradar
  Cc: martin.petersen, jejb, linux-scsi, aacraid, dave.carroll,
	scott.benesh, stable


Hi Sagar,

> From: Sagar Biradar <sagar.biradar@microchip.com>
>
> To: Martin K. Petersen <martin.petersen@oracle.com>
> To: James Bottomley <jejb@linux.vnet.ibm.com>
> cc: linux-scsi <linux-scsi@vger.kernel.org>
> cc: aacraid@microsemi.com
> cc: Dave Carroll <dave.carroll@microchip.com>
> cc: Scott Benesh <scott.benesh@microchip.com>
> cc: stable@vger.kernel.org
>
> Fix performance issue where the queue depth for SmartIOC logical
> volumes is set to 1, and allow the usual logical volume code
> to be executed

This wasn't a correctly formatted patch. Please see:

	Documentation/process/submitting-patches.rst

and remember to run checkpatch prior to submission.

I fixed it up. Applied to 5.1/scsi-queue. Thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives
@ 2019-03-14 11:12   ` Martin K. Petersen
  0 siblings, 0 replies; 6+ messages in thread
From: Martin K. Petersen @ 2019-03-14 11:12 UTC (permalink / raw)
  To: sagar.biradar
  Cc: martin.petersen, jejb, linux-scsi, aacraid, dave.carroll,
	scott.benesh, stable


Hi Sagar,

> From: Sagar Biradar <sagar.biradar@microchip.com>
>
> To: Martin K. Petersen <martin.petersen@oracle.com>
> To: James Bottomley <jejb@linux.vnet.ibm.com>
> cc: linux-scsi <linux-scsi@vger.kernel.org>
> cc: aacraid@microsemi.com
> cc: Dave Carroll <dave.carroll@microchip.com>
> cc: Scott Benesh <scott.benesh@microchip.com>
> cc: stable@vger.kernel.org
>
> Fix performance issue where the queue depth for SmartIOC logical
> volumes is set to 1, and allow the usual logical volume code
> to be executed

This wasn't a correctly formatted patch. Please see:

	Documentation/process/submitting-patches.rst

and remember to run checkpatch prior to submission.

I fixed it up. Applied to 5.1/scsi-queue. Thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2019-03-14 11:13 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-08  7:26 [PATCH] scsi: aacraid: Fix performance issue(QD) on logical drives sagar.biradar
2019-03-08  7:26 ` sagar.biradar
2019-03-08 16:09 ` Dave.Carroll
2019-03-08 16:09   ` Dave.Carroll
2019-03-14 11:12 ` Martin K. Petersen
2019-03-14 11:12   ` Martin K. Petersen

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.