← Back to team overview

kernel-packages team mailing list archive

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

 

List of just the sha1s:

f3cfabce7a2e92564d380de3aad4b43901fb7ae6
3533f8603d28b77c62d75ec899449a99bc6b77a1
f885fb73f64154690c2158e813de56363389ffec
56b26e69c8283121febedd12b3cc193384af46b9
adb6f9e1a8c6af1037232b59edb11277471537ea
8caf92d80526f3d7cc96831ec18b384ebcaccdf0
52f9614dd8294e95d2c0929c2d4f64b077ae486f
4cd83ecdac20d30725b4f96e5d7814a1e290bc7e
26b9fd8b3452dcf0a8862e307ee23f442f63fb51

These are all merged for v3.17-rc1.

** 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:
  
- <TBI>
+ 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>
+ 
+ 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>
+ 
+ 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>
+ 
+ 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>
+ 
+ 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>
+ 
+ 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>
+ 
+ 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>
+ 
+ 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>
+ 
+ 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>

** Changed in: linux (Ubuntu Utopic)
       Status: Fix Released => In Progress

** Changed in: linux (Ubuntu Utopic)
     Assignee: (unassigned) => Andy Whitcroft (apw)

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

  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>

  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>

  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>

  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>

  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>

  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>

  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>

  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>

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


References