Fix potential corruption using SAS controller on HiSilicon arm64 boards
Bug #1794156 reported by
dann frazier
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux (Ubuntu) |
Fix Released
|
Undecided
|
dann frazier | ||
Bionic |
Fix Released
|
Undecided
|
dann frazier | ||
Cosmic |
Fix Released
|
Undecided
|
dann frazier |
Bug Description
[Impact]
On HiSilicon ARM server systems, there is a potential for corruption when multiple CPUs are accessing the same controller.
[Test Case]
I don't have a reproducer for the failure, so will be regression tested only on D05 and D06 platforms.
[Fix]
commit 1c09b663168bb5f
Author: Xiaofei Tan <email address hidden>
Date: Wed Jul 18 22:14:32 2018 +0800
scsi: hisi_sas: add memory barrier in task delivery function
[Regression Risk]
The biggest regression risk I see would be a performance impact. But correct data is better than fast data :)
CVE References
Changed in linux (Ubuntu): | |
status: | New → In Progress |
Changed in linux (Ubuntu Bionic): | |
status: | New → In Progress |
Changed in linux (Ubuntu): | |
assignee: | nobody → dann frazier (dannf) |
Changed in linux (Ubuntu Bionic): | |
assignee: | nobody → dann frazier (dannf) |
Changed in linux (Ubuntu Cosmic): | |
status: | In Progress → Fix Committed |
Changed in linux (Ubuntu Bionic): | |
status: | In Progress → Fix Committed |
tags: | added: cscc |
To post a comment you must log in.
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- bionic' to 'verification- done-bionic' . If the problem still exists, change the tag 'verification- needed- bionic' to 'verification- failed- bionic' .
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!