2018-01-24 22:50:44

by Michael Kelley (EOSG)

Subject: [PATCH 1/1] scsi: storvsc: Increase cmd_per_lun for higher speed devices

Increase cmd_per_lun to allow more I/Os in progress per device,
particularly for NVMe's. The Hyper-V host side can handle the
higher count with no issues.

Signed-off-by: Michael Kelley <[email protected]>
---
drivers/scsi/storvsc_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
index f3264c4..6205107 100644
--- a/drivers/scsi/storvsc_drv.c
+++ b/drivers/scsi/storvsc_drv.c
@@ -1657,7 +1657,7 @@ static int storvsc_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *scmnd)
.eh_timed_out = storvsc_eh_timed_out,
.slave_alloc = storvsc_device_alloc,
.slave_configure = storvsc_device_configure,
- .cmd_per_lun = 255,
+ .cmd_per_lun = 2048,
.this_id = -1,
.use_clustering = ENABLE_CLUSTERING,
/* Make sure we dont get a sg segment crosses a page boundary */
--
1.8.3.1

2018-01-31 04:13:54

by Martin K. Petersen

Subject: Re: [PATCH 1/1] scsi: storvsc: Increase cmd_per_lun for higher speed devices

> Increase cmd_per_lun to allow more I/Os in progress per device,
> particularly for NVMe's. The Hyper-V host side can handle the
> higher count with no issues.

Long/KY/Cathy/Stephen: Please ack/review Michael's patches.

--
Martin K. Petersen Oracle Linux Engineering

2018-01-31 18:35:30

by KY Srinivasan

Subject: RE: [PATCH 1/1] scsi: storvsc: Increase cmd_per_lun for higher speed devices

> -----Original Message-----
> From: Michael Kelley (EOSG)
> Sent: Wednesday, January 24, 2018 2:50 PM
> To: KY Srinivasan <[email protected]>; Stephen Hemminger
> <[email protected]>; [email protected]; Long Li
> <[email protected]>; [email protected];
> [email protected]; [email protected]; linux-
> [email protected]
> Cc: Michael Kelley (EOSG) <[email protected]>
> Subject: [PATCH 1/1] scsi: storvsc: Increase cmd_per_lun for higher speed
> devices
>
> Increase cmd_per_lun to allow more I/Os in progress per device,
> particularly for NVMe's. The Hyper-V host side can handle the
> higher count with no issues.
>
> Signed-off-by: Michael Kelley <[email protected]>

Reviewed-by: K. Y. Srinivasan <[email protected]>
Acked-by: K. Y. Srinivasan <[email protected]>

Thanks,

K. Y
> ---
> drivers/scsi/storvsc_drv.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
> index f3264c4..6205107 100644
> --- a/drivers/scsi/storvsc_drv.c
> +++ b/drivers/scsi/storvsc_drv.c
> @@ -1657,7 +1657,7 @@ static int storvsc_queuecommand(struct Scsi_Host
> *host, struct scsi_cmnd *scmnd)
> .eh_timed_out = storvsc_eh_timed_out,
> .slave_alloc = storvsc_device_alloc,
> .slave_configure = storvsc_device_configure,
> - .cmd_per_lun = 255,
> + .cmd_per_lun = 2048,
> .this_id = -1,
> .use_clustering = ENABLE_CLUSTERING,
> /* Make sure we dont get a sg segment crosses a page boundary */
> --
> 1.8.3.1

2018-02-07 00:41:04

by Martin K. Petersen

Subject: Re: [PATCH 1/1] scsi: storvsc: Increase cmd_per_lun for higher speed devices

Michael,

> Increase cmd_per_lun to allow more I/Os in progress per device,
> particularly for NVMe's. The Hyper-V host side can handle the higher
> count with no issues.

Applied to 4.16/scsi-fixes. Thank you!

--
Martin K. Petersen Oracle Linux Engineering