Can't disable USB port from BIOS
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
HWE Next |
Fix Released
|
High
|
AceLan Kao | ||
linux (Ubuntu) |
Fix Released
|
High
|
AceLan Kao | ||
Xenial |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
From BIOS, there is front USB port disable function. It is supposed the option in BIOS can
disable USB port and due to AMD chip limitation, it need OS’s cowork.
Otherwise the USB disable function will be invalid when resuming from S3 or reinsert USB device.
=== SRU Justification ===
[Impact]
In BIOS, there is front USB port disable function.
It is supposed the option in BIOS can disable USB port, but due to AMD chip's
limitation, it needs OS’s cowork. Otherwise the USB disable function will
be invalid when resuming from S3 or reinsert USB device.
[Fix]
After checking with Windows, we found there are 2 bits that are default on
in Ubuntu is disabled in Windows. They are
#define PORT_WKOC_E (1<<22) /* wake on overcurrent (enable) */
#define PORT_WKCONN_E (1<<20) /* wake on connect (enable) */
So, this patch introduce a quirk to not setup the wakeup bits only for
AMD USB chips.
[Test Case]
Verified on the machine has this issue, and confirm this patch works.
[Regression Potential]
This patch only affects the listed USB ID, and we don't cert this kind of
wakeup method, so should have no regression concerns.
Wake up from USB device is still working, and the patch behavior is confirmed by
the project lead, Tsai, Nicholas, from AMD in the discussion thread[1].
[Misc]
This patch is accepted by the maintainer but still not shown up in any git tree
yet. The discussion thread can be found here[1]. To fit the project schedule,
we need this commit in our kernel as soon as possbile, and can't wait to
next SRU release cycle.
CVE References
tags: | added: originate-from-1682706 somerville |
Changed in hwe-next: | |
assignee: | nobody → AceLan Kao (acelankao) |
status: | New → In Progress |
tags: | added: patch |
description: | updated |
Changed in hwe-next: | |
importance: | Undecided → High |
Changed in linux (Ubuntu): | |
importance: | Undecided → High |
Changed in linux (Ubuntu Xenial): | |
status: | New → Fix Committed |
Changed in hwe-next: | |
status: | In Progress → Fix Committed |
Changed in hwe-next: | |
status: | Fix Committed → Fix Released |
Changed in linux (Ubuntu): | |
status: | In Progress → Fix Released |
ASMedia provide a patch for this and has submitted to upstream.
Currently, no reply from the maintainer, but we need this fix as soon as possible, so we submit the SRU patch before this patch is accepted by upstream.