hisi_sas: driver robustness fixes

Bug #1739807 reported by dann frazier on 2017-12-22
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Status tracked in Bionic
Artful
High
dann frazier
Bionic
High
dann frazier

Bug Description

[Impact]
There are several bugs in the driver that can cause it to fail during teardown/error recovery.

[Test Case]
Regression tested by issuing sg_reset commands in a loop during an iozone3 run on a disk attached to a hisi_sas controller.

$ iozone -a &
$ while :; do sudo sg_reset --device /dev/sda; sleep 5; done

[Regression Risk]
All fixes are encapsulated within a single driver which was explicitly tested for regressions.

dann frazier (dannf) on 2017-12-22
Changed in linux (Ubuntu Artful):
status: New → In Progress
assignee: nobody → dann frazier (dannf)
importance: Undecided → High
Changed in linux (Ubuntu Bionic):
importance: Undecided → High
dann frazier (dannf) on 2017-12-22
description: updated
dann frazier (dannf) on 2017-12-22
description: updated
Seth Forshee (sforshee) on 2018-01-03
Changed in linux (Ubuntu Bionic):
status: In Progress → Fix Committed
Changed in linux (Ubuntu Artful):
status: In Progress → Fix Committed

This bug is awaiting verification that the kernel in -proposed solves the problem. Please test the kernel and update this bug with the results. If the problem is solved, change the tag 'verification-needed-artful' to 'verification-done-artful'. If the problem still exists, change the tag 'verification-needed-artful' to 'verification-failed-artful'.

If verification is not done by 5 working days from today, this fix will be dropped from the source code, and this bug will be closed.

See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you!

tags: added: verification-needed-artful
dann frazier (dannf) wrote :

artful verification - I've completed the iozone test on a filesystem atop a disk connected to a hisi sas controller w/o error.

description: updated
tags: added: verification-done-artful
removed: verification-needed-artful
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers