Various storvsc bug fixes.
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| linux (Ubuntu) |
Fix Released
|
High
|
Andy Whitcroft | ||
| Utopic |
Fix Released
|
High
|
Andy Whitcroft | ||
Bug Description
Over last week we have made changes and fix bugs in storvsc , please include these patch set .
[PATCH V3 1/7] Drivers: scsi: storvsc: Change the limits to reflect the values
on the host In Hyper-V, each port/path/target maps to 1 scsi host adapter.
In reality, the path/target is not used (ie always set to 0) so our scsi host
adapter essentially has 1 bus with 1 target that contains up to 256 luns.
[PATCH V3 2/7] 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.
[PATCH V3 3/7] 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.
[PATCH V3 4/7] 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.
[PATCH 5/7] Drivers: scsi: storvsc: Implement a timedout 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.
[PATCH 6/7] 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.
[PATCH 7/7] 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.
Thanks,
Abhishek
| Brad Figg (brad-figg) wrote Missing required logs. | #1 |
| Changed in linux (Ubuntu): | |
| status: | New → Incomplete |
| Joseph Salisbury (jsalisbury) wrote | #2 |
| Changed in linux (Ubuntu): | |
| importance: | Undecided → Medium |
| tags: | added: kernel-da-key kernel-hyper-v |
| Changed in linux (Ubuntu): | |
| status: | Incomplete → Triaged |
| Changed in linux (Ubuntu): | |
| importance: | Medium → High |
| vivek yadav (vyadav) wrote | #3 |
| Joseph Salisbury (jsalisbury) wrote | #4 |
| Andy Whitcroft (apw) wrote | #5 |
| Andy Whitcroft (apw) wrote | #6 |
| Changed in linux (Ubuntu Utopic): | |
| assignee: | nobody → Andy Whitcroft (apw) |
| status: | Triaged → Fix Committed |
| Launchpad Janitor (janitor) wrote | #7 |
| Changed in linux (Ubuntu Utopic): | |
| status: | Fix Committed → Fix Released |
This bug is missing log files that will aid in diagnosing the problem. From a terminal window please run:
apport-collect 1347169
and then change the status of the bug to 'Confirmed'.
If, due to the nature of the issue you have encountered, you are unable to run this command, please add a comment stating that fact and change the bug status to 'Confirmed'.
This change has been made by an automated script, maintained by the Ubuntu Kernel Team.