kernel-packages team mailing list archive

Thread
Date
[Bug 1354397] Re: Latest fixes from storvsc & scsi drivers for Hyper-V/Azure

To: kernel-packages@xxxxxxxxxxxxxxxxxxx
From: Andy Whitcroft <apw@xxxxxxxxxxxxx>
Date: Tue, 12 Aug 2014 14:12:40 -0000
Reply-to: Bug 1354397 <1354397@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx
There are two additional foundational patches needed, one for each of
these sets (the last patch is logically separate above):

  commit c1d40a527e885a40bb9ea6c46a1b1145d42b66a0
  Author: Martin K. Petersen <martin.petersen@xxxxxxxxxx>
  Date:   Tue Jul 15 12:49:17 2014 -0400

    scsi: add a blacklist flag which enables VPD page inquiries

and

  commit a118c6c1d907e52286df25ee1e8b217f25d6f73d
  Author: Christoph Hellwig <hch@xxxxxx>
  Date:   Sat Jun 28 12:08:05 2014 +0200

    sd: don't use scsi_setup_blk_pc_cmnd for flush requests


** Description changed:

  A number of significant fixes have landed upstream for Hyper-V/Azure
  clouds.  The upstream devs for this recommend we apply the below list of
  patches:
  
  commit 26b9fd8b3452dcf0a8862e307ee23f442f63fb51
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Fri Jul 18 17:11:27 2014 +0200
  
-     sd: fix a bug in deriving the FLUSH_TIMEOUT from the basic I/O timeout
-     
-     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 <kys@xxxxxxxxxxxxx>
-     Reviewed-by: James Bottomley <JBottomley@xxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     sd: fix a bug in deriving the FLUSH_TIMEOUT from the basic I/O
+ timeout
  
  commit 4cd83ecdac20d30725b4f96e5d7814a1e290bc7e
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:26 2014 -0700
  
-     Drivers: scsi: storvsc: Change the limits to reflect the values on the host
-     
-     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 <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     Drivers: scsi: storvsc: Change the limits to reflect the values on
+ the host
  
  commit 52f9614dd8294e95d2c0929c2d4f64b077ae486f
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:27 2014 -0700
  
-     Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported by the Host
-     
-     Set cmd_per_lun to reflect value supported by the Host.
-     
-     Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported
+ by the Host
  
  commit 8caf92d80526f3d7cc96831ec18b384ebcaccdf0
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:28 2014 -0700
  
-     Drivers: scsi: storvsc: Filter commands based on the storage protocol version
-     
-     Going forward it is possible that some of the commands that are not currently
-     implemented will be implemented on future Windows hosts. Even if they are not
-     implemented, we are told the host will corrrectly handle unsupported
-     commands (by returning appropriate return code and sense information).
-     Make command filtering depend on the host version.
-     
-     Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     Drivers: scsi: storvsc: Filter commands based on the storage
+ protocol version
  
  commit adb6f9e1a8c6af1037232b59edb11277471537ea
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:29 2014 -0700
  
-     Drivers: scsi: storvsc: Fix a bug in handling VMBUS protocol version
-     
-     Based on the negotiated VMBUS protocol version, we adjust the size of the storage
-     protocol messages. The two sizes we currently handle are pre-win8 and post-win8.
-     In WS2012 R2, we are negotiating higher VMBUS protocol version than the win8
-     version. Make adjustments to correctly handle this.
-     
-     Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     Drivers: scsi: storvsc: Fix a bug in handling VMBUS protocol version
  
  commit 56b26e69c8283121febedd12b3cc193384af46b9
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:30 2014 -0700
  
-     Drivers: scsi: storvsc: Implement a eh_timed_out handler
-     
-     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 <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     [hch: added a better comment explaining the issue]
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     Drivers: scsi: storvsc: Implement a eh_timed_out handler
  
  commit f885fb73f64154690c2158e813de56363389ffec
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:31 2014 -0700
  
-     drivers: scsi: storvsc: Set srb_flags in all cases
-     
-     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 <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     drivers: scsi: storvsc: Set srb_flags in all cases
  
  commit 3533f8603d28b77c62d75ec899449a99bc6b77a1
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:32 2014 -0700
  
-     drivers: scsi: storvsc: Correctly handle TEST_UNIT_READY failure
-     
-     On some Windows hosts on FC SANs, TEST_UNIT_READY can return SRB_STATUS_ERROR.
-     Correctly handle this. Note that there is sufficient sense information to
-     support scsi error handling even in this case.
-     
-     Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Cc: <stable@xxxxxxxxxxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     drivers: scsi: storvsc: Correctly handle TEST_UNIT_READY failure
  
  commit f3cfabce7a2e92564d380de3aad4b43901fb7ae6
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Mon Jul 21 16:06:01 2014 -0700
  
-     Drivers: add blist flags
-     
-     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 <kys@xxxxxxxxxxxxx>
-     Reviewed-by: Hannes Reinecke <hare@xxxxxxx>
-     Signed-off-by: Christoph Hellwig <hch@xxxxxx>
+     Drivers: add blist flags

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1354397

Title:
  Latest fixes from storvsc & scsi drivers for Hyper-V/Azure

Status in “linux” package in Ubuntu:
  In Progress
Status in “linux” source package in Trusty:
  In Progress
Status in “linux” source package in Utopic:
  In Progress

Bug description:
  A number of significant fixes have landed upstream for Hyper-V/Azure
  clouds.  The upstream devs for this recommend we apply the below list
  of patches:

  commit 26b9fd8b3452dcf0a8862e307ee23f442f63fb51
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Fri Jul 18 17:11:27 2014 +0200

      sd: fix a bug in deriving the FLUSH_TIMEOUT from the basic I/O
  timeout

  commit 4cd83ecdac20d30725b4f96e5d7814a1e290bc7e
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:26 2014 -0700

      Drivers: scsi: storvsc: Change the limits to reflect the values on
  the host

  commit 52f9614dd8294e95d2c0929c2d4f64b077ae486f
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:27 2014 -0700

      Drivers: scsi: storvsc: Set cmd_per_lun to reflect value supported
  by the Host

  commit 8caf92d80526f3d7cc96831ec18b384ebcaccdf0
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:28 2014 -0700

      Drivers: scsi: storvsc: Filter commands based on the storage
  protocol version

  commit adb6f9e1a8c6af1037232b59edb11277471537ea
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:29 2014 -0700

      Drivers: scsi: storvsc: Fix a bug in handling VMBUS protocol
  version

  commit 56b26e69c8283121febedd12b3cc193384af46b9
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:30 2014 -0700

      Drivers: scsi: storvsc: Implement a eh_timed_out handler

  commit f885fb73f64154690c2158e813de56363389ffec
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:31 2014 -0700

      drivers: scsi: storvsc: Set srb_flags in all cases

  commit 3533f8603d28b77c62d75ec899449a99bc6b77a1
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Sat Jul 12 09:48:32 2014 -0700

      drivers: scsi: storvsc: Correctly handle TEST_UNIT_READY failure

  commit f3cfabce7a2e92564d380de3aad4b43901fb7ae6
  Author: K. Y. Srinivasan <kys@xxxxxxxxxxxxx>
  Date:   Mon Jul 21 16:06:01 2014 -0700

      Drivers: add blist flags

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1354397/+subscriptions
References

[Bug 1354397] [NEW] Latest fixes from storvsc & scsi drivers for Hyper-V/Azure
From: Andy Whitcroft, 2014-08-08