libsas: old linkrate advertised after phy disabled
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux (Ubuntu) |
In Progress
|
Undecided
|
dann frazier | ||
Disco |
Fix Released
|
Medium
|
dann frazier | ||
Eoan |
Won't Fix
|
Undecided
|
dann frazier |
Bug Description
[Impact]
sysfs continues to expose a valid phy linkrate even after the phy has been disabled.
[Test Case]
Using hisi_sas, a libsas-based driver:
(initramfs) cd /sys/class/
(initramfs) cat negotiated_linkrate
12.0 Gbit
(initramfs) echo 0 > enable
[ 59.172411] hisi_sas_v3_hw 0000:74:02.0: phydown: phy0 phy_state=0xfe
[ 59.178851] hisi_sas_v3_hw 0000:74:02.0: erroneous completion iptt=4028 task=00000000bb
[ 59.194139] sas: smp_execute_
(initramfs) cat negotiated_linkrate
12.0 Gbit
(Note to self: d06-1 has the setup to reproduce this)
[Fix]
d8649fc1c5e40 scsi: libsas: Do discovery on empty PHY to update PHY info
085f104a83d56 scsi: libsas: Inject revalidate event for root port event
[Regression Risk]
Impact is limited to drivers built on top of libsas.
CVE References
Changed in linux (Ubuntu Disco): | |
status: | New → In Progress |
assignee: | nobody → dann frazier (dannf) |
summary: |
- phy linkrate persists after deactivation + libsas: old linkrate advertised after phy disabled |
description: | updated |
description: | updated |
Changed in linux (Ubuntu Disco): | |
importance: | Undecided → Medium |
Changed in linux (Ubuntu Disco): | |
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- disco' to 'verification- done-disco' . If the problem still exists, change the tag 'verification- needed- disco' to 'verification- failed- disco'.
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!