[PATCH 1/1] scsi: storvsc: Support manual scan of FC hosts on Hyper-V

2016-03-12 Thread K. Y. Srinivasan
The default user scan function associated with FC (fc_user_scan) is not suitable for FC hosts on Hyper-V since we don't have an rport associated with FC host on Hyper-V . Set it to NULL so we can support manual scan of FC targets on Hyper-V. Signed-off-by: K. Y. Srinivasan <k...@microsoft.

[PATCH v4 1/1] scsi: storvsc: Fix a build issue reported by kbuild test robot

2016-02-26 Thread K. Y. Srinivasan
orvsc_drv.c:(.exit.text+0x123c): undefined reference to >> `fc_release_transport' With this commit, the storvsc driver depends on FC atttributes. Make this dependency explicit. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reported-by: Fengguang Wu <fengguang...@intel.com&

[PATCH 1/1] scsi: storvsc: Fix a build issue reported by kbuild test robot

2016-02-26 Thread K. Y. Srinivasan
orvsc_drv.c:(.exit.text+0x123c): undefined reference to >> `fc_release_transport' With this commit, the storvsc driver depends on FC atttributes. Make this dependency explicit. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reported-by: Fengguang Wu <fengguang...@intel.com> ---

[PATCH V2 1/1] scsi: storvsc: Fix a build issue reported by kbuild test robot

2016-01-29 Thread K. Y. Srinivasan
orvsc_drv.c:(.exit.text+0x123c): undefined reference to >> `fc_release_transport' With this commit, the storvsc driver depends on FC atttributes. Make this dependency explicit. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reported-by: Fengguang Wu <fengguang...@intel.com> ---

[PATCH V2 2/2] scsi: storvsc: Use the specified target ID in device lookup

2016-01-27 Thread K. Y. Srinivasan
The current code assumes that there is only one target in device lookup. Fix this bug. This will alow us to correctly handle hot reomoval of LUNs. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Alex Ng <ale...@microsoft.com> Tested-by: Vivek Yadav <vya..

[PATCH V2 1/2] scsi: storvsc: Install the storvsc specific timeout handler for FC devices

2016-01-27 Thread K. Y. Srinivasan
The default timeout routine used for FC transport is not suitable for FC devices managed by storvsc since FC devices managed by storvsc driver do not have an rport associated with them. Use the time out handler used for SCSI devices for FC devices as well. Signed-off-by: K. Y. Srinivasan &l

[PATCH V2 0/2] scsi: storvsc: Miscellaneous fixes

2016-01-27 Thread K. Y. Srinivasan
Some miscellaneous fixes. V2: addressed comments from Hannes Reinecke <h...@suse.de> K. Y. Srinivasan (2): scsi: storvsc: Install the storvsc specific timeout handler for FC devices scsi: storvsc: Use the specified target ID in device lookup drivers/scsi/storvsc_drv.c

[PATCH V4 4/4] scsi: storvsc: Tighten up the interrupt path

2015-12-23 Thread K. Y. Srinivasan
to storvsc_device and passing it as an argument. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> Reviewed-by: Hannes Reinecke <h...@suse.com> Tested-by: Alex Ng <ale...@micro

[PATCH V4 2/4] scsi: storvsc: Properly support Fibre Channel devices

2015-12-23 Thread K. Y. Srinivasan
For FC devices managed by this driver, atttach the appropriate transport template. This will allow us to create the appropriate sysfs files for these devices. With this we can publish the wwn for both the port and the node. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: L

[PATCH V4 3/4] scsi: storvsc: Refactor the code in storvsc_channel_init()

2015-12-23 Thread K. Y. Srinivasan
The function storvsc_channel_init() repeatedly interacts with the host to extract various channel properties. Refactor this code to eliminate code repetition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by: Johannes Thumsh

[PATCH V4 1/4] scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet

2015-12-23 Thread K. Y. Srinivasan
The hv_fc_wwn_packet is exchanged over vmbus. Make the definition in Linux match the Window's definition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by:

[PATCH V4 0/4] scsi: storvsc: Properly support FC hosts

2015-12-23 Thread K. Y. Srinivasan
;l...@intel.com> V4: Addressed comments from Hannes and James. K. Y. Srinivasan (4): scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet scsi: storvsc: Properly support Fibre Channel devices scsi: storvsc: Refactor the code in storvsc_channel_init() scsi: storvsc: Tighte

[PATCH V3 2/4] scsi: storvsc: Properly support Fibre Channel devices

2015-12-13 Thread K. Y. Srinivasan
For FC devices managed by this driver, atttach the appropriate transport template. This will allow us to create the appropriate sysfs files for these devices. With this we can publish the wwn for both the port and the node. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: L

[PATCH V3 0/4] scsi: storvsc: Properly support FC hosts

2015-12-13 Thread K. Y. Srinivasan
.@intel.com> K. Y. Srinivasan (4): scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet scsi: storvsc: Properly support Fibre Channel devices scsi: storvsc: Refactor the code in storvsc_channel_init() scsi: storvsc: Tighten up the interrupt path drivers/scsi/st

[PATCH V3 4/4] scsi: storvsc: Tighten up the interrupt path

2015-12-13 Thread K. Y. Srinivasan
On the interrupt path, we repeatedly establish the pointer to the storvsc_device. Fix this. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> Tested-by: Alex Ng <ale...@micros

[PATCH V3 3/4] scsi: storvsc: Refactor the code in storvsc_channel_init()

2015-12-13 Thread K. Y. Srinivasan
The function storvsc_channel_init() repeatedly interacts with the host to extract various channel properties. Refactor this code to eliminate code repetition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by: Johannes Thumsh

[PATCH V3 1/4] scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet

2015-12-13 Thread K. Y. Srinivasan
The hv_fc_wwn_packet is exchanged over vmbus. Make the definition in Linux match the Window's definition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> Reviewed-by: Long Li <lon...@microsoft.com> Tested-by: Alex Ng <

[PATCH V2 2/4] scsi: storvsc: Properly support Fibre Channel devices

2015-12-12 Thread K. Y. Srinivasan
For FC devices managed by this driver, atttach the appropriate transport template. This will allow us to create the appropriate sysfs files for these devices. With this we can publish the wwn for both the port and the node. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: L

[PATCH V2 3/4] scsi: storvsc: Refactor the code in storvsc_channel_init()

2015-12-12 Thread K. Y. Srinivasan
The function storvsc_channel_init() repeatedly interacts with the host to extract various channel properties. Refactor this code to eliminate code repetition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by: Johannes Thumsh

[PATCH V2 1/4] scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet

2015-12-12 Thread K. Y. Srinivasan
The hv_fc_wwn_packet is exchanged over vmbus. Make the definition in Linux match the Window's definition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> Reviewed-by: Long Li <lon...@microsoft.com> Tested-by: Alex Ng <

[PATCH V2 4/4] scsi: storvsc: Tighten up the interrupt path

2015-12-12 Thread K. Y. Srinivasan
On the interrupt path, we repeatedly establish the pointer to the storvsc_device. Fix this. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Reviewed-by: Johannes Thumshirn <jthumsh...@suse.de> Tested-by: Alex Ng <ale...@micros

[PATCH V2 0/4] scsi: storvsc: Properly support FC hosts

2015-12-12 Thread K. Y. Srinivasan
Properly support FC hosts. Additional cleanup patches are also included. In this version I have adddressed comments from Dan Carpenter <dan.carpen...@oracle.com> and from Johannes Thumshirn <jthumsh...@suse.de>. K. Y. Srinivasan (4): scsi: storvsc: Fix a bug i

[PATCH 0/4] scsi: storvsc: Properly support FC hosts

2015-12-10 Thread K. Y. Srinivasan
Properly support FC hosts. Additional cleanup patches are also included. K. Y. Srinivasan (4): scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet scsi: storvsc: Properly support Fibre Channel devices scsi: storvsc: Refactor the code in storvsc_channel_init() scsi: storvsc

[PATCH 1/4] scsi: storvsc: Fix a bug in the layout of the hv_fc_wwn_packet

2015-12-10 Thread K. Y. Srinivasan
The hv_fc_wwn_packet is exchanged over vmbus. Make the definition in Linux match the Window's definition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Tested-by: Alex Ng <ale...@microsoft.com> --- drivers/scsi/storvsc_drv

[PATCH 2/4] scsi: storvsc: Properly support Fibre Channel devices

2015-12-10 Thread K. Y. Srinivasan
For FC devices managed by this driver, atttach the appropriate transport template. This will allow us to create the appropriate sysfs files for these devices. With this we can publish the wwn for both the port and the node. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: L

[PATCH 4/4] scsi: storvsc: Tighten up the interrupt path

2015-12-10 Thread K. Y. Srinivasan
On the interrupt path, we repeatedly establish the pointer to the storvsc_device. Fix this. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Tested-by: Alex Ng <ale...@microsoft.com> --- drivers/scsi/storvsc_drv.c | 23

[PATCH 3/4] scsi: storvsc: Refactor the code in storvsc_channel_init()

2015-12-10 Thread K. Y. Srinivasan
The function storvsc_channel_init() repeatedly interacts with the host to extract various channel properties. Refactor this code to eliminate code repetition. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> Tested-by: Alex Ng <ale..

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

2015-10-06 Thread K. Y. Srinivasan
that this patch was not committed upstream. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Reviewed-by: Long Li <lon...@microsoft.com> --- drivers/scsi/storvsc_drv.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc

[PATCH 1/1] storvsc: get rid of bounce buffer

2015-10-01 Thread K. Y. Srinivasan
Kr.má<rkrc...@redhat.com> Signed-off-by: Vitaly Kuznetsov <vkuzn...@redhat.com> Reviewed-by: K. Y. Srinivasan <k...@microsoft.com> Tested-by: Alex Ng <ale...@microsoft.com> Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> --

[PATCH 2/3] scsi: introduce short_inquiry flag for broken host adapters

2015-09-17 Thread K. Y. Srinivasan
ort (5), using 36' messages on console. This can be problematic for slow consoles. Introduce short_inquiry host template flag to avoid printing error messages for such adapters. Signed-off-by: Vitaly Kuznetsov <vkuzn...@redhat.com> Reviewed-by: K. Y. Srinivasan <k...@microsoft.com> Teste

[PATCH 1/3] scsi_scan: don't dump trace when scsi_prep_async_scan() is called twice

2015-09-17 Thread K. Y. Srinivasan
calls scsi_scan_host() and in case previous scan is still running we get the message and stack dump on console. Signed-off-by: Vitaly Kuznetsov <vkuzn...@redhat.com> Reviewed-by: K. Y. Srinivasan <k...@microsoft.com> Tested-by: Alex Ng <ale...@microsoft.com> Signed-off-by: K. Y. Sriniv

[PATCH 0/3] scsi: Some miscellaneous fixes

2015-09-17 Thread K. Y. Srinivasan
Some miscellaneous fixes. K. Y. Srinivasan (1): storvsc: Don't set the SRB_FLAGS_QUEUE_ACTION_ENABLE flag Vitaly Kuznetsov (2): scsi_scan: don't dump trace when scsi_prep_async_scan() is called twice scsi: introduce short_inquiry flag for broken host adapters drivers/scsi/scsi_scan.c

[PATCH RESEND 3/3] storvsc: Don't set the SRB_FLAGS_QUEUE_ACTION_ENABLE flag

2015-09-17 Thread K. Y. Srinivasan
Don't set the SRB_FLAGS_QUEUE_ACTION_ENABLE flag since we are not specifying tags. Qlogic driver does not work correctly if the SRB_FLAGS_QUEUE_ACTION_ENABLE is set and no action tag is specified. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Tested-by: Vivek Yadav <vya...@micr

[PATCH 1/1] storvsc: Don't set the SRB_FLAGS_QUEUE_ACTION_ENABLE flag

2015-08-31 Thread K. Y. Srinivasan
Don't set the SRB_FLAGS_QUEUE_ACTION_ENABLE flag since we are not specifying tags. Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> Cc: sta...@vger.kernel.org --- drivers/scsi/storvsc_drv.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/storvsc

[PATCH 1/1] scsi: storvsc: Use ARRAY_SIZE() to compute the elements in the protocol array

2015-08-14 Thread K. Y. Srinivasan
-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 64fa305..4dffc3b 100644 --- a/drivers/scsi/storvsc_drv.c +++ b/drivers/scsi/storvsc_drv.c

[PATCH RESEND V2 7/7] scsi: storvsc: Set the error code correctly in failure conditions

2015-08-13 Thread K. Y. Srinivasan
In the function storvsc_channel_init(), error code was not getting set correctly in some of the failure cases. Fix this issue. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/scsi/storvsc_drv.c | 12 +--- 1 files

[PATCH RESEND V2 1/7] scsi: storvsc: Rather than look for sets of specific protocol versions, make decisions based on ranges.

2015-08-13 Thread K. Y. Srinivasan
Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 11 +++ 1 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 3c6584f..582f3b5 100644 --- a/drivers

[PATCH RESEND V2 6/7] scsi: storvsc: Allow write_same when host is windows 10

2015-08-13 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Allow WRITE_SAME for Windows10 and above hosts. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 +- 1 files

[PATCH RESEND V2 4/7] scsi: storvsc: use correct defaults for values determined by protocol negotiation

2015-08-13 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Use correct defaults for values determined by protocol negotiation, instead of resetting them with every scsi controller. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k

[PATCH RESEND V2 2/7] scsi: storvsc: Use a single value to track protocol versions

2015-08-13 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Use a single value to track protocol versions to simplify comparisons and to be consistent with vmbus version tracking. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k

[PATCH RESEND V2 0/7] scsi: storvsc: Some miscellaneous cleanup

2015-08-13 Thread K. Y. Srinivasan
Cleanup version handling as well as base feature detection on storage version as opposed to host version. In this version, I have addressed comments from Dan Carpenter. K. Y. Srinivasan (1): scsi: storvsc: Set the error code correctly in failure conditions Keith Mange (6): scsi: storvsc

[PATCH RESEND V2 3/7] scsi: storvsc: Untangle the storage protocol negotiation from the vmbus protocol negotiation.

2015-08-13 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Currently we are making decisions based on vmbus protocol versions that have been negotiated; use storage potocol versions instead. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y

[PATCH RESEND V2 5/7] scsi: storvsc: use storage protocol version to determine storage capabilities

2015-08-13 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Use storage protocol version instead of vmbus protocol version when determining storage capabilities. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com

[PATCH 1/1] Drivers: hv: vmbus: fix init_vp_index() for reloading hv_netvsc

2015-08-13 Thread K. Y. Srinivasan
From: Dexuan Cui de...@microsoft.com This fixes the recent commit 3b71107d73b16074afa7658f3f0fcf837aabfe24: Drivers: hv: vmbus: Further improve CPU affiliation logic Without the fix, reloading hv_netvsc hangs the guest. Signed-off-by: Dexuan Cui de...@microsoft.com Signed-off-by: K. Y

[PATCH V2 0/7] scsi: storvsc: Some miscellaneous cleanup

2015-06-03 Thread K. Y. Srinivasan
Cleanup version handling as well as base feature detection on storage version as opposed to host version. In this version, I have addressed comments from Dan Carpenter. K. Y. Srinivasan (1): scsi: storvsc: Set the error code correctly in failure conditions Keith Mange (6): scsi: storvsc

[PATCH V2 1/7] scsi: storvsc: Rather than look for sets of specific protocol versions, make decisions based on ranges.

2015-06-03 Thread K. Y. Srinivasan
Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 11 +++ 1 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 3c6584f..582f3b5 100644 --- a/drivers

[PATCH V2 5/7] scsi: storvsc: use storage protocol version to determine storage capabilities

2015-06-03 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Use storage protocol version instead of vmbus protocol version when determining storage capabilities. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com

[PATCH V2 4/7] scsi: storvsc: use correct defaults for values determined by protocol negotiation

2015-06-03 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Use correct defaults for values determined by protocol negotiation, instead of resetting them with every scsi controller. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k

[PATCH V2 2/7] scsi: storvsc: Use a single value to track protocol versions

2015-06-03 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Use a single value to track protocol versions to simplify comparisons and to be consistent with vmbus version tracking. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k

[PATCH V2 6/7] scsi: storvsc: Allow write_same when host is windows 10

2015-06-03 Thread K. Y. Srinivasan
From: Keith Mange keith.ma...@microsoft.com Allow WRITE_SAME for Windows10 and above hosts. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |6 +- 1 files

[PATCH V2 7/7] scsi: storvsc: Set the error code correctly in failure conditions

2015-06-03 Thread K. Y. Srinivasan
In the function storvsc_channel_init(), error code was not getting set correctly in some of the failure cases. Fix this issue. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reported-by: Dan Carpenter dan.carpen...@oracle.com --- drivers/scsi/storvsc_drv.c | 12 +--- 1 files

[PATCH 2/6] scsi: storvsc: Use a single value to track protocol versions

2015-05-29 Thread K. Y. Srinivasan
From: keith.ma...@microsoft.com keith.ma...@microsoft.com Use a single value to track protocol versions to simplify comparisons and to be consistent with vmbus version tracking. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y

[PATCH 4/6] scsi: storvsc: use correct defaults for values determined by protocol negotiation

2015-05-29 Thread K. Y. Srinivasan
From: keith.ma...@microsoft.com keith.ma...@microsoft.com Use correct defaults for values determined by protocol negotiation, instead of resetting them with every scsi controller. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y

[PATCH 5/6] scsi: storvsc: use storage protocol version to determine storage capabilities

2015-05-29 Thread K. Y. Srinivasan
From: keith.ma...@microsoft.com keith.ma...@microsoft.com Use storage protocol version instead of vmbus protocol version when determining storage capabilities. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k

[PATCH 3/6] hv:scsi:Untangle the storage protocol negotiation from the vmbus protocol negotiation.

2015-05-29 Thread K. Y. Srinivasan
-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 109 +++- 1 files changed, 87 insertions(+), 22 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 5f9d133..f29871e 100644 --- a/drivers/scsi

[PATCH 1/6] scsi: storvsc: Rather than look for sets of specific protocol versions, make decisions based on ranges.

2015-05-29 Thread K. Y. Srinivasan
-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c | 11 +++ 1 files changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c index 3c6584f..582f3b5 100644

[PATCH 0/6] scsi: storvsc: Some miscellaneous cleanup

2015-05-29 Thread K. Y. Srinivasan
Cleanup version handling as well as base feature detection on storage version as opposed to host version. keith.ma...@microsoft.com (6): scsi: storvsc: Rather than look for sets of specific protocol versions, make decisions based on ranges. scsi: storvsc: Use a single value to track

[PATCH 6/6] scsi: storvsc: Allow write_same when host is windows 10

2015-05-29 Thread K. Y. Srinivasan
From: keith.ma...@microsoft.com keith.ma...@microsoft.com Allow WRITE_SAME for Windows10 and above hosts. Tested-by: Alex Ng ale...@microsoft.com Signed-off-by: Keith Mange keith.ma...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/scsi/storvsc_drv.c |6

[PATCH 1/1] scsi: storvsc: Set the SRB flags correctly when no data transfer is needed

2015-04-24 Thread K. Y. Srinivasan
Set the SRB flags correctly when there is no data transfer. Cc: sta...@vger.kernel.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers

[PATCH 1/1] scsi: storvsc: Set the SRB flags correctly when no data transfer is needed

2015-04-24 Thread K. Y. Srinivasan
Set the SRB flags correctly when there is no data transfer. Cc: sta...@vger.kernel.org Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reviewed-by: Long Li lon...@microsoft.com --- drivers/scsi/storvsc_drv.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers

[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 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

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

2015-03-23 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 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 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 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/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 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: 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 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 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

  1   2   >