scanning for LUNs

2013-04-04 Thread K. Y. Srinivasan
Here is the code snippet for scanning LUNS (drivers/scsi/scsi_scan.c in function __scsi_scan_target()): /* * Scan LUN 0, if there is some response, scan further. Ideally, we * would not configure LUN 0 until all LUNs are scanned. */ res =

[PATCH 4/6] Drivers: scsi: storvsc: Update the storage protocol to win8 level

2013-05-15 Thread K. Y. Srinivasan
Update the storage protocol to the win8 level. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 203 1 files changed, 168 insertions(+), 35 deletions(-) diff --git

[PATCH 6/6] Drivers: scsi: storvsc: Support FC devices

2013-05-15 Thread K. Y. Srinivasan
Add support for synthetic Fiber Channel devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers

[PATCH 0/6] Drivers: hv/scsi: Implement multi-channel support

2013-05-15 Thread K. Y. Srinivasan
patches. K. Y. Srinivasan (6): Drivers: hv: vmbus: Implement multi-channel support Drivers: hv: Add the GUID fot synthetic fiber channel device Drivers: scsi: storvsc: Make the scsi timeout a module parameter Drivers: scsi: storvsc: Update the storage protocol to win8 level Drivers: scsi

[PATCH 1/6] Drivers: hv: vmbus: Implement multi-channel support

2013-05-15 Thread K. Y. Srinivasan
of a given device across all available CPUs. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/hv/channel.c | 41 ++-- drivers/hv/channel_mgmt.c | 117 +++-- include/linux

[PATCH 5/6] Drivers: scsi: storvsc: Implement multi-channel support

2013-05-15 Thread K. Y. Srinivasan
Implement multi-channel support for the storage devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 124 ++-- 1 files changed, 120 insertions(+), 4 deletions

[PATCH 2/6] Drivers: hv: Add the GUID fot synthetic fiber channel device

2013-05-15 Thread K. Y. Srinivasan
In preparation for supporting synthetic Fiber Channel device, add the GUID for this service. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- include/linux/hyperv.h | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff

[PATCH 3/6] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-05-15 Thread K. Y. Srinivasan
The standard scsi timeout is not appropriate in some of the environments where Hyper-V is deployed. Set this timeout appropriately for all devices managed by this driver. Further make this a module parameter. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya

[PATCH V1 0/7] Drivers: hv/scsi: Implement multi-channel support

2013-05-16 Thread K. Y. Srinivasan
the scsi patches. In this version, based on Greg's comments, I have fixed the names of the exported symbols from the vmbus driver. K. Y. Srinivasan (7): Drivers: hv: vmbus: Implement multi-channel support Drivers: hv: Add the GUID fot synthetic fiber channel device Drivers: scsi: storvsc: Make

[PATCH V1 2/7] Drivers: hv: Add the GUID fot synthetic fiber channel device

2013-05-16 Thread K. Y. Srinivasan
In preparation for supporting synthetic Fiber Channel device, add the GUID for this service. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- include/linux/hyperv.h | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff

[PATCH V1 1/7] Drivers: hv: vmbus: Implement multi-channel support

2013-05-16 Thread K. Y. Srinivasan
of a given device across all available CPUs. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/hv/channel.c | 41 ++-- drivers/hv/channel_mgmt.c | 116 +++-- include/linux

[PATCH V1 3/7] Drivers: scsi: storvsc: Make the scsi timeout a module parameter

2013-05-16 Thread K. Y. Srinivasan
The standard scsi timeout is not appropriate in some of the environments where Hyper-V is deployed. Set this timeout appropriately for all devices managed by this driver. Further make this a module parameter. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya

[PATCH V1 5/7] Drivers: scsi: storvsc: Implement multi-channel support

2013-05-16 Thread K. Y. Srinivasan
Implement multi-channel support for the storage devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 124 ++-- 1 files changed, 120 insertions(+), 4 deletions

[PATCH V1 6/7] Drivers: scsi: storvsc: Support FC devices

2013-05-16 Thread K. Y. Srinivasan
Add support for synthetic Fiber Channel devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers

[PATCH V1 7/7] Drivers: scsi: storvsc: Increase the value of STORVSC_MAX_IO_REQUESTS

2013-05-16 Thread K. Y. Srinivasan
Increase the value of STORVSC_MAX_IO_REQUESTS to 200 requests. The current ringbuffer size can support this higher value. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions

[PATCH V2 0/5] Drivers: scsi: storvsc

2013-06-03 Thread K. Y. Srinivasan
related changes. I have since separated the VMBUS chages and these have already been checked in. K. Y. Srinivasan (5): Drivers: scsi: storvsc: Make the scsi timeout a module parameter Drivers: scsi: storvsc: Update the storage protocol to win8 level Drivers: scsi: storvsc: Implement multi

[PATCH 4/5] Drivers: scsi: storvsc: Support FC devices

2013-06-03 Thread K. Y. Srinivasan
Add support for synthetic Fiber Channel devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers

[PATCH 5/5] Drivers: scsi: storvsc: Increase the value of STORVSC_MAX_IO_REQUESTS

2013-06-03 Thread K. Y. Srinivasan
Increase the value of STORVSC_MAX_IO_REQUESTS to 200 requests. The current ringbuffer size can support this higher value. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions

[PATCH 3/5] Drivers: scsi: storvsc: Implement multi-channel support

2013-06-03 Thread K. Y. Srinivasan
Implement multi-channel support for the storage devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 131 ++- 1 files changed, 127 insertions(+), 4 deletions

[PATCH 2/5] Drivers: scsi: storvsc: Update the storage protocol to win8 level

2013-06-03 Thread K. Y. Srinivasan
Update the storage protocol to the win8 level. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 203 1 files changed, 168 insertions(+), 35 deletions(-) diff --git

[PATCH 5/5] Drivers: scsi: storvsc: Increase the value of STORVSC_MAX_IO_REQUESTS

2013-06-04 Thread K. Y. Srinivasan
Increase the value of STORVSC_MAX_IO_REQUESTS to 200 requests. The current ringbuffer size can support this higher value. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions

[PATCH 4/5] Drivers: scsi: storvsc: Support FC devices

2013-06-04 Thread K. Y. Srinivasan
Add support for synthetic Fibre Channel devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers

[PATCH 3/5] Drivers: scsi: storvsc: Implement multi-channel support

2013-06-04 Thread K. Y. Srinivasan
Implement multi-channel support for the storage devices. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 131 ++- 1 files changed, 127 insertions(+), 4 deletions

[PATCH 2/5] Drivers: scsi: storvsc: Update the storage protocol to win8 level

2013-06-04 Thread K. Y. Srinivasan
Update the storage protocol to the win8 level. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 203 1 files changed, 168 insertions(+), 35 deletions(-) diff --git

[PATCH 1/5] Drivers: scsi: storvsc: Increase the value of scsi timeout for storvsc devices

2013-06-04 Thread K. Y. Srinivasan
since (a) the host is already implementing a variety of recovery strategies and (b) implementing a recovery strategy at the VM level may be more appropriate in cases where storage latencies exceed a certain threshold. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang

[PATCH 1/1] [SCSI] Fix a bug in deriving the FLUSH_TIMEOUT from the basic I/O timeout

2014-06-04 Thread K. Y. Srinivasan
Commit ID: 7e660100d85af860e7ad763202fff717adcdaacd added code to derive the FLUSH_TIMEOUT from the basic I/O timeout. However, this patch did not use the basic I/O timeout of the device. Fix this bug. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/sd.c |4 +++- 1 files

[PATCH 0/8] Drivers: scsi: storvsc: Bug fixes and improvements

2014-07-08 Thread K. Y. Srinivasan
In this patch set I have fixed a few bugs and implemented some enhancements. K. Y. Srinivasan (8): Drivers: scsi: storvsc: Change the limits to reflect the values on the host Drivers: scsi: storvsc: Filter commands based on the storage protocol version Drivers: scsi: storvsc: Fix

[PATCH 2/8] Drivers: scsi: storvsc: Filter commands based on the storage protocol version

2014-07-08 Thread K. Y. Srinivasan
Going forward it is possible that some of the commands that are not currently implemented will be implemented on future Windows hosts. Make command filtering depend on the host version. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c

[PATCH 8/8] drivers: scsi: storvsc: Correctly handle TEST_UNIT_READY failure

2014-07-08 Thread K. Y. Srinivasan
On some Windows hosts on FC SANs, TEST_UNIT_READY can return SRB_STATUS_ERROR. Correctly handle this. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git

[PATCH 1/8] Drivers: scsi: storvsc: Change the limits to reflect the values on the host

2014-07-08 Thread K. Y. Srinivasan
Hyper-V hosts can support multiple targets and multiple channels and larger number of LUNs per target. Update the code to reflect this. With this patch we can correctly enumerate all the paths in a multi-path storage environment. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta

[PATCH 5/8] Drivers: scsi: storvsc: Fix a bug in the handling of SRB status flags

2014-07-08 Thread K. Y. Srinivasan
SRB status can have additional information. Mask these out before processing SRB status. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/drivers/scsi

[PATCH 3/8] Drivers: scsi: storvsc: Fix a bug in handling VMBUS protocol version

2014-07-08 Thread K. Y. Srinivasan
this. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 759853c..d9d8051 100644 --- a/drivers/scsi

[PATCH 6/8] Drivers: scsi: storvsc: Implement an abort handler

2014-07-08 Thread K. Y. Srinivasan
Implement a simple abort handler. The host does not support Abort; just ensure that all inflight I/Os have been accounted for. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 22 ++ 1 files changed, 22 insertions(+), 0 deletions(-) diff

[PATCH 7/8] drivers: scsi: storvsc: Set srb_flags in all cases

2014-07-08 Thread K. Y. Srinivasan
Correctly set SRB flags for all valid I/O directions. Some IHV drivers on the Windows host require this. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git

[PATCH 4/8] Drivers: scsi: storvsc: Filter WRITE_SAME_16

2014-07-08 Thread K. Y. Srinivasan
Host does not handle WRITE_SAME_16; filter this command out. This patch is required to handle large devices (greater than 2 TB disks). Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0

[PATCH V2 0/8] Drivers: scsi: storvsc: Bug fixes and improvements

2014-07-09 Thread K. Y. Srinivasan
In this patch set I have fixed a few bugs and implemented some enhancements. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org K. Y. Srinivasan (8): Drivers: scsi: storvsc: Change the limits to reflect the values on the host Drivers: scsi

[PATCH V2 7/8] drivers: scsi: storvsc: Set srb_flags in all cases

2014-07-09 Thread K. Y. Srinivasan
Correctly set SRB flags for all valid I/O directions. Some IHV drivers on the Windows host require this. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 12 +--- 1 files changed, 5 insertions(+), 7 deletions(-) diff --git

[PATCH V2 1/8] Drivers: scsi: storvsc: Change the limits to reflect the values on the host

2014-07-09 Thread K. Y. Srinivasan
Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 47 +--- 1 files changed, 31 insertions(+), 16 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers

[PATCH V2 4/8] Drivers: scsi: storvsc: Fix a bug in handling VMBUS protocol version

2014-07-09 Thread K. Y. Srinivasan
this. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 17 ++--- 1 files changed, 10 insertions(+), 7 deletions(-) diff --git

[PATCH V2 8/8] drivers: scsi: storvsc: Correctly handle TEST_UNIT_READY failure

2014-07-09 Thread K. Y. Srinivasan
On some Windows hosts on FC SANs, TEST_UNIT_READY can return SRB_STATUS_ERROR. Correctly handle this. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi

[PATCH V2 6/8] Drivers: scsi: storvsc: Implement an abort handler

2014-07-09 Thread K. Y. Srinivasan
Implement a simple abort handler. The host does not support Abort; just ensure that all inflight I/Os have been accounted for. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi

[PATCH V2 3/8] Drivers: scsi: storvsc: Filter commands based on the storage protocol version

2014-07-09 Thread K. Y. Srinivasan
). Make command filtering depend on the host version. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 16 +--- 1 files changed

[PATCH V2 5/8] Drivers: scsi: storvsc: Fix a bug in the handling of SRB status flags

2014-07-09 Thread K. Y. Srinivasan
SRB status can have additional information. Mask these out before processing SRB status. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c

[PATCH V2 2/8] Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host

2014-07-09 Thread K. Y. Srinivasan
Set cmd_per_lun to reflect value supported by the Host. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1

[PATCH V2 2/8] Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host

2014-07-10 Thread K. Y. Srinivasan
Set cmd_per_lun to reflect value supported by the Host. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1

[PATCH V3 0/7] Drivers: scsi: storvsc: Bug fixes and improvements

2014-07-12 Thread K. Y. Srinivasan
In this patch set I have fixed a few bugs and implemented some enhancements. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org K. Y. Srinivasan (7): Drivers: scsi: storvsc: Change the limits to reflect the values on the host Drivers: scsi

[PATCH V3 1/7] Drivers: scsi: storvsc: Change the limits to reflect the values on the host

2014-07-12 Thread K. Y. Srinivasan
Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 47 +--- 1 files changed, 31 insertions(+), 16 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers

[PATCH 6/7] drivers: scsi: storvsc: Set srb_flags in all cases

2014-07-12 Thread K. Y. Srinivasan
Correctly set SRB flags for all valid I/O directions. Some IHV drivers on the Windows host require this. The host validates the command and SRB flags prior to passing the command down to native driver stack. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org

[PATCH 7/7] drivers: scsi: storvsc: Correctly handle TEST_UNIT_READY failure

2014-07-12 Thread K. Y. Srinivasan
Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 9342ba4..29d0329 100644 --- a/drivers/scsi

[PATCH V3 2/7] Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host

2014-07-12 Thread K. Y. Srinivasan
Set cmd_per_lun to reflect value supported by the Host. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1

[PATCH 5/7] Drivers: scsi: storvsc: Implement a timedout handler

2014-07-12 Thread K. Y. Srinivasan
On Azure, we have seen instances of unbounded I/O latencies. To deal with this issue, implement handler that can reset the timeout. Note that the host gaurantees that it will respond to each command that has been issued. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta

[PATCH V3 4/7] Drivers: scsi: storvsc: Fix a bug in handling VMBUS protocol version

2014-07-12 Thread K. Y. Srinivasan
this. In this version of the patch I have addressed comments from Christoph Hellwig h...@infradead.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 17 ++--- 1 files changed, 10 insertions(+), 7 deletions(-) diff --git

[PATCH 1/1] Drivers: scsi: storvsc: Add blist flags

2014-07-20 Thread K. Y. Srinivasan
Add blist flags to permit the reading of the VPD pages even when the target may claim SPC-2 compliance. MSFT targets currently claim SPC-2 compliance while they implement post SPC-2 features. With this patch we can correctly handle WRITE_SAME_16 issues. Signed-off-by: K. Y. Srinivasan k

[PATCH 1/1] Drivers: scsi: storvsc: Add blist flags

2014-07-21 Thread K. Y. Srinivasan
Add blist flags to permit the reading of the VPD pages even when the target may claim SPC-2 compliance. MSFT targets currently claim SPC-2 compliance while they implement post SPC-2 features. With this patch we can correctly handle WRITE_SAME_16 issues. Signed-off-by: K. Y. Srinivasan k

[PATCH 2/2] Drivers: scsi: storvsc: Force discovery of LUNs that may have been removed.

2014-08-16 Thread K. Y. Srinivasan
for an I/O to remove the device, force the discovery of LUNs that may have been removed prior to discovering LUNs that may have been added. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 26 ++ 1 files changed, 26 insertions(+), 0

[PATCH 0/2] Drivers: scsi: storvsc: Fix issues with hot-add/remove of LUNs

2014-08-16 Thread K. Y. Srinivasan
This patch-set addresses issues with LUN hot-add and remove. When the host notifies the guest that a scan is needed, scan the host. Also, prior to discovering new devices that may have been added, ensure we handle the LUN remove case first. K. Y. Srinivasan (2): Drivers: scsi: storvsc

[PATCH 1/2] Drivers: scsi: storvsc: In responce to a scan event, scan the host

2014-08-16 Thread K. Y. Srinivasan
The virtual HBA that storvsc implements can support multiple channels and targets. So, scan the host when the host notifies that a scan is needed. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 19 +++ 1 files changed, 7 insertions(+), 12

[PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-02 Thread K. Y. Srinivasan
Get rid of the warning messages since they will clutter up various system logs and are of questionable value to the end user. For debugging purposes, this information can be gotten by setting the scsi log level appropriately. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi

[PATCH 1/1] Drivers: scsi: storvsc: Account for in-transit packets in the RESET path

2012-10-02 Thread K. Y. Srinivasan
. Currently, the host does not do anything with the RESET command. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi

[PATCH 1/1] Drivers: scsi: storvsc: Initialize the sglist

2013-01-24 Thread K. Y. Srinivasan
Properly initialize scatterlist before using it. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: Stable sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c

[PATCH 1/1] Drivers: hv: balloon: Make adjustments to the pressure report

2013-01-25 Thread K. Y. Srinivasan
The host expects that the pressure report includes the pressure due to the pages that have been ballooned. Make necessary adjustments to reflect that. Also, include the free memory information in the pressure report. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/hv/hv_balloon.c

[PATCH 1/1] Drivers: hv: balloon: Make adjustments to the pressure report

2013-01-25 Thread K. Y. Srinivasan
The host expects that the pressure report includes the pressure due to the pages that have been ballooned. Make necessary adjustments to reflect that. Also, include the free memory information in the pressure report. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/hv/hv_balloon.c

[PATCH 1/1] Drivers: scsi: storvsc: Initialize the sglist

2013-01-25 Thread K. Y. Srinivasan
Properly initialize scatterlist before using it. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index

[PATCH RESEND 1/1] Drivers: scsi: storvsc: Initialize the sglist

2013-02-06 Thread K. Y. Srinivasan
Properly initialize scatterlist before using it. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index

[PATCH 0/4] Drivers: scsi: storvsc

2013-02-21 Thread K. Y. Srinivasan
This patch set (two of the patches are being resent) fixes and enhances the functionality of the Hyper-V storage driver K. Y. Srinivasan (3): Drivers: scsi: storvsc: Initialize the sglist Drivers: scsi: storvsc: Restructure error handling code on command completion Drivers: scsi

[PATCH 3/4] Drivers: scsi: storvsc: Restructure error handling code on command completion

2013-02-21 Thread K. Y. Srinivasan
In preparation for handling additional sense codes, restructure and cleanup the error handling code in the command completion code path. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 101

[PATCH RESEND 1/4] Drivers: scsi: storvsc: Initialize the sglist

2013-02-21 Thread K. Y. Srinivasan
Initialize sglist before using it. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 270b3cf..5ada1d0

[PATCH 4/4] Drivers: scsi: storvsc: Handle dynamic resizing of the device

2013-02-21 Thread K. Y. Srinivasan
Handle LUN size changes by re-scanning the device. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/scsi/storvsc_drv.c | 31 +++ 1 files changed, 31 insertions(+), 0 deletions(-) diff --git a/drivers

[PATCH RESEND 2/4] scsi: storvsc: avoid usage of WRITE_SAME

2013-02-21 Thread K. Y. Srinivasan
. WRITE_SAME isnt really supported by both versions, so disable the command usage globally. Signed-off-by: Olaf Hering o...@aepfle.de Cc: KY Srinivasan k...@microsoft.com Cc: linux-scsi@vger.kernel.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |4 1 files

Drivers: scsi: FLUSH timeout

2013-09-20 Thread K. Y. Srinivasan
The SD_FLUSH_TIMEOUT value is currently hardcoded. On our cloud, we sometimes hit this timeout. I was wondering if we could make this a module parameter. If this is acceptable, I can send you a patch for this. Regards, K. Y -- To unsubscribe from this list: send the line unsubscribe linux-scsi

[PATCH 1/1] Drivers: scsi: Derive the FLUSH_TIMEOUT from the basic I/O timeout

2013-10-04 Thread K. Y. Srinivasan
. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/sd.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index e62d17d..8aff306 100644 --- a/drivers/scsi/sd.c +++ b/drivers/scsi/sd.c @@ -829,7 +829,7 @@ static int

[PATCH 1/1] Drivers: scsi: storvsc: Change the limits to reflect the values on the host

2014-05-22 Thread K. Y. Srinivasan
Hyper-V hosts can support multiple targets and multiple channels and larger number of LUNs per target. Update the code to reflect this. With this patch we can correctly enumerate all the paths in a multi-path storage environment. MS-TFS: 173725 Signed-off-by: K. Y. Srinivasan k

[PATCH 4/4] Drivers: scsi: storvsc: Force SPC-3 compliance on win8 and win8 r2 hosts

2014-12-16 Thread K. Y. Srinivasan
claiming SPC-3 compliance on these hosts as well. This will enable TRIM support on these hosts. Suggested by: James Bottomley james.bottom...@hansenpartnership.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 13 + 1 files changed, 13

[PATCH 0/4] Drivers: scsi: storvsc: Fix miscellaneous issues

2014-12-16 Thread K. Y. Srinivasan
The first two patches in this series are a resend; these were submitted some months ago and as far as I know, there were no outstanding issues. Win8 and win8 r2 hosts do support SPC-3 features but claim SPC-2 compliance. This issue is fixed here as well. K. Y. Srinivasan (4): Drivers: scsi

[PATCH 2/4] Drivers: scsi: storvsc: Force discovery of LUNs that may have been removed.

2014-12-16 Thread K. Y. Srinivasan
for an I/O to remove the device, force the discovery of LUNs that may have been removed prior to discovering LUNs that may have been added. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 26 ++ 1 files changed, 26 insertions(+), 0

[PATCH 3/4] Drivers: scsi: storvsc: Fix a bug in storvsc limits

2014-12-16 Thread K. Y. Srinivasan
Commit 4cd83ecdac20d30725b4f96e5d7814a1e290bc7e changed the limits to reflect the values on the host. It turns out that WS2008R2 cannot correctly handle these new limits. Fix this bug by setting the limits based on the host. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi

[PATCH 1/4] Drivers: scsi: storvsc: In responce to a scan event, scan the host

2014-12-16 Thread K. Y. Srinivasan
The virtual HBA that storvsc implements can support multiple channels and targets. So, scan the host when the host notifies that a scan is needed. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 19 +++ 1 files changed, 7 insertions(+), 12

[PATCH 1/3] scsi: storvsc: Retrieve information about the capability of the target

2015-03-09 Thread K. Y. Srinivasan
The storage protocol informs the guest of the I/O capabilities of the storage stack. Retrieve this information and use it in the guest. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6

[PATCH 3/3] scsi: storvsc: Enable clustering

2015-03-09 Thread K. Y. Srinivasan
Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index a36c232..d7ebf00 100644 --- a/drivers/scsi/storvsc_drv.c +++ b/drivers/scsi

[PATCH 0/3] scsi: storvsc: Increase the tablesize based on host's capabilities

2015-03-09 Thread K. Y. Srinivasan
Presently, storvsc limits the I/O size arbitrarily. Make this configurable based on what the host advertises. K. Y. Srinivasan (3): scsi: storvsc: Retrieve information about the capability of the target scsi: storvsc: Set the tablesize based on the information given by the host scsi

[PATCH 2/3] scsi: storvsc: Set the tablesize based on the information given by the host

2015-03-09 Thread K. Y. Srinivasan
Set the tablesize based on what the host has informed us. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c | 70 ++- 1 files changed, 55 insertions(+), 15 deletions(-) diff --git

[PATCH V2 1/7] scsi: storvsc: Increase the ring buffer size

2015-03-27 Thread K. Y. Srinivasan
Increase the default ring buffer size as this can significantly improve performance especially on high latency storage back-ends. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions

[PATCH V2 6/7] scsi: storvsc: Don't assume that the scatterlist is not chained

2015-03-27 Thread K. Y. Srinivasan
The current code assumes that the scatterlists presented are not chained. Fix the code to not make this assumption. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- V2: Got rid of double assignment. Olaf Hering o...@aepfle.de drivers/scsi

[PATCH V2 3/7] scsi: storvsc: Always send on the selected outgoing channel

2015-03-27 Thread K. Y. Srinivasan
The current code always sent packets without data on the primary channel. Properly distribute sending of packets with no data amongst all available channels. I would like to thank Long Li for noticing this problem. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon

[PATCH V2 5/7] scsi: storvsc: Fix a bug in copy_from_bounce_buffer()

2015-03-27 Thread K. Y. Srinivasan
We may exit this function without properly freeing up the maapings we may have acquired. Fix the bug. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 15 --- 1 files changed, 8

[PATCH 7/7] scsi: storvsc: Set the tablesize based on the information given by the host

2015-03-27 Thread K. Y. Srinivasan
Set the tablesize based on the information given by the host. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com Tested-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c | 89 +++ 1 files changed

[PATCH V2 4/7] scsi: storvsc: Retrieve information about the capability of the target

2015-03-27 Thread K. Y. Srinivasan
The storage protocol informs the guest of the I/O capabilities of the storage stack. Retrieve this information and use it in the guest. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6

[PATCH V2 2/7] scsi: storvsc: Size the queue depth based on the ringbuffer size

2015-03-27 Thread K. Y. Srinivasan
Size the queue depth based on the ringbuffer size. Also accomodate for the fact that we could have multiple channels (ringbuffers) per adaptor. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- v2: Fixed a bug in computing queue depth

[PATCH V2 0/7] scsi: storvsc: Miscellaneous enhancements and fixes

2015-03-27 Thread K. Y. Srinivasan
issues raised by Olaf Hering o...@aepfle.de, Long Li lon...@microsoft.com and Venkatesh Srinivas venkate...@google.com K. Y. Srinivasan (7): scsi: storvsc: Increase the ring buffer size scsi: storvsc: Size the queue depth based on the ringbuffer size scsi: storvsc: Always send on the selected

[PATCH 1/3] scsi: storvsc: Increase the ring buffer size

2015-02-27 Thread K. Y. Srinivasan
Increase the default ring buffer size as this can significantly improve performance especially on high latency storage back-ends. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions

[PATCH 3/3] scsi: storvsc: Always send on the selected outgoing channel

2015-02-27 Thread K. Y. Srinivasan
The current code always sent packets without data on the primary channel. Properly distribute sending of packets with no data amongst all available channels. I would like to thank Long Li for noticing this problem. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon

[PATCH 2/3] scsi: storvsc: Size the queue depth based on the ringbuffer size

2015-02-27 Thread K. Y. Srinivasan
Size the queue depth based on the ringbuffer size. Also accomodate for the fact that we could have multiple channels (ringbuffers) per adaptor. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c | 27

[PATCH 0/3] scsi: storvsc: Some enhancements

2015-02-27 Thread K. Y. Srinivasan
While testing on some high latency storage backends, to get the advertised IOPS, we have to increase the size of the ringbuffer. Address this issue. K. Y. Srinivasan (3): scsi: storvsc: Increase the ring buffer size Scsi: storvsc: Size the queue depth based on the ringbuffer size scsi

[PATCH 5/7] scsi: storvsc: Fix a bug in copy_from_bounce_buffer()

2015-03-23 Thread K. Y. Srinivasan
We may exit this function without properly freeing up the maapings we may have acquired. Fix the bug. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c | 15 --- 1 files changed, 8 insertions(+), 7 deletions(-) diff --git

[PATCH RESEND 1/7] scsi: storvsc: Increase the ring buffer size

2015-03-23 Thread K. Y. Srinivasan
Increase the default ring buffer size as this can significantly improve performance especially on high latency storage back-ends. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |2 +- 1 files changed, 1 insertions

[PATCH RESEND 4/7] scsi: storvsc: Retrieve information about the capability of the target

2015-03-23 Thread K. Y. Srinivasan
The storage protocol informs the guest of the I/O capabilities of the storage stack. Retrieve this information and use it in the guest. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 ++ 1 files changed, 6

[PATCH 0/7] scsi: storvsc: Miscellaneous enhancements and fixes

2015-03-23 Thread K. Y. Srinivasan
This patch-set addresses perf issues discovered on the Azure storage stack. These patches also fix a couple of bugs. K. Y. Srinivasan (7): scsi: storvsc: Increase the ring buffer size scsi: storvsc: Size the queue depth based on the ringbuffer size scsi: storvsc: Always send on the selected

[PATCH 6/7] scsi: storvsc: Don't assume that the scatterlist is not chained

2015-03-23 Thread K. Y. Srinivasan
The current code assumes that the scatterlists presented are not chained. Fix the code to not make this assumption. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 98 +-- 1 files changed, 57 insertions(+), 41

[PATCH RESEND 2/7] scsi: storvsc: Size the queue depth based on the ringbuffer size

2015-03-23 Thread K. Y. Srinivasan
Size the queue depth based on the ringbuffer size. Also accomodate for the fact that we could have multiple channels (ringbuffers) per adaptor. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c | 27

[PATCH V2 7/7] scsi: storvsc: Set the tablesize based on the information given by the host

2015-03-23 Thread K. Y. Srinivasan
Set the tablesize based on the information given by the host. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- v2: Rebase the code and enable clustering drivers/scsi/storvsc_drv.c | 89 +++ 1 files

  1   2   >