Fix after-suspend-mediacard/sdhc-insert test failed
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
HWE Next |
New
|
Undecided
|
Unassigned | ||
linux (Ubuntu) |
Invalid
|
Undecided
|
AceLan Kao | ||
Jammy |
Invalid
|
Undecided
|
AceLan Kao | ||
Mantic |
Won't Fix
|
Undecided
|
AceLan Kao | ||
Noble |
Fix Released
|
Undecided
|
AceLan Kao | ||
linux-oem-6.1 (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Jammy |
Fix Released
|
Undecided
|
koba | ||
Mantic |
Invalid
|
Undecided
|
Unassigned | ||
Noble |
Invalid
|
Undecided
|
Unassigned | ||
linux-oem-6.5 (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Jammy |
Fix Released
|
Undecided
|
koba | ||
Mantic |
Invalid
|
Undecided
|
Unassigned | ||
Noble |
Invalid
|
Undecided
|
Unassigned | ||
linux-oem-6.8 (Ubuntu) |
Invalid
|
Undecided
|
AceLan Kao | ||
Jammy |
Invalid
|
Undecided
|
Unassigned | ||
Mantic |
Invalid
|
Undecided
|
Unassigned | ||
Noble |
Fix Released
|
Undecided
|
AceLan Kao |
Bug Description
SRU Justification for Noble
[Impact]
checkbox test case, after-suspend-
[Fix]
The new series of the fix has been merged into v6.9
6d4266675279 PCI/ASPM: Update save_state when configuration changes
64dbb2d70744 PCI/ASPM: Disable L1 before configuring L1 Substates
c198fafa0125 PCI/ASPM: Call pci_save_
17423360a27a PCI/ASPM: Save L1 PM Substates Capability for suspend/resume
1e11b5494c3d PCI/ASPM: Move pci_save_
f3994bba8200 PCI/ASPM: Always build aspm.c
fa84f4435a62 PCI/ASPM: Move pci_configure_ltr() to aspm.c
[Test Case]
1. suspend and resume.
2. check if the error appears in dmesg
~~~
pcieport 0000:00:1c.0: pciehp: Slot(5): Card not present
rtsx_pci 0000:05:00.0: Unable to change power state from D0 to D3hot, device inaccessible
rtsx_pci 0000:05:00.0: Unable to change power state from D3cold to D0, device inaccessible
~~~
[Where the problems could occur]
Hard to evaluate, these patches change the PCI/ASPM behavior and may lead to PCI devices work abnormal.
=======
[Impact]
checkbox test case, after-suspend-
[Fix]
Commit a7152be79b62 ("Revert "PCI/ASPM: Save L1 PM Substates Capability
for suspend/resume"") reverted saving and restoring of ASPM L1 Substates
due to a regression that caused resume from suspend to fail on certain
systems. However, we never added this capability back and this is now
causing systems fail to enter low power CPU states, drawing more power
from the battery.
The original revert mentioned that we restore L1 PM substate configuration
even though ASPM L1 may already be enabled. This is due the fact that
the pci_restore_
Try to enable this functionality again following PCIe r6.0.1, sec 5.5.4
more closely by:
1) Do not restore ASPM configuration in pci_restore_
do that after PCIe capability is restored in pci_restore_
following PCIe r6.0, sec 5.5.4.
2) ASPM is first enabled on the upstream component and then downstream
(this is already forced by the parent-child ordering of Linux
Device Power Management framework).
3) Program ASPM L1 PM substate configuration before L1 enables.
4) Program ASPM L1 PM substate enables last after rest of the fields
in the capability are programmed.
5) Add denylist that skips restoring on the ASUS and TUXEDO systems
where these regressions happened, just in case. For the TUXEDO case
we only skip restore if the BIOS is involved in system suspend
(that's forcing "mem_sleep=deep" in the command line). This is to
avoid possible power regression when the default suspend to idle is
used, and at the same time make sure the devices continue working
after resume when the BIOS is involved.
[Test Case]
1. suspend and resume.
2. check if the error appears in dmesg
~~~
pcieport 0000:00:1c.0: pciehp: Slot(5): Card not present
rtsx_pci 0000:05:00.0: Unable to change power state from D0 to D3hot, device inaccessible
rtsx_pci 0000:05:00.0: Unable to change power state from D3cold to D0, device inaccessible
~~~
[where the issue could happen]
low, the patch works well on the reported malfunctioned ASUS platform too.
CVE References
Changed in linux (Ubuntu Noble): | |
status: | New → In Progress |
assignee: | nobody → koba (kobako) |
Changed in linux-oem-6.1 (Ubuntu Jammy): | |
status: | New → In Progress |
assignee: | nobody → koba (kobako) |
Changed in linux-oem-6.5 (Ubuntu Jammy): | |
status: | New → In Progress |
assignee: | nobody → koba (kobako) |
tags: | added: oem-priority originate-from-2035307 stella |
description: | updated |
Changed in linux-oem-6.5 (Ubuntu Jammy): | |
status: | In Progress → Fix Committed |
Changed in linux (Ubuntu Mantic): | |
assignee: | nobody → koba (kobako) |
status: | New → In Progress |
Changed in linux (Ubuntu Jammy): | |
assignee: | nobody → koba (kobako) |
status: | New → In Progress |
Changed in linux-oem-6.1 (Ubuntu Mantic): | |
status: | New → Invalid |
Changed in linux-oem-6.1 (Ubuntu Noble): | |
status: | New → Invalid |
Changed in linux-oem-6.5 (Ubuntu Mantic): | |
status: | New → Invalid |
Changed in linux-oem-6.5 (Ubuntu Noble): | |
status: | New → Invalid |
Changed in linux-oem-6.1 (Ubuntu Jammy): | |
status: | In Progress → Fix Committed |
tags: |
added: verification-done-jammy-linux-oem-6.1 removed: verification-needed-jammy-linux-oem-6.1 |
Changed in linux (Ubuntu Jammy): | |
assignee: | koba (kobako) → AceLan Kao (acelankao) |
Changed in linux (Ubuntu Mantic): | |
assignee: | koba (kobako) → AceLan Kao (acelankao) |
Changed in linux (Ubuntu Noble): | |
assignee: | koba (kobako) → AceLan Kao (acelankao) |
Changed in linux (Ubuntu Jammy): | |
status: | In Progress → Invalid |
Changed in linux (Ubuntu Mantic): | |
status: | In Progress → Won't Fix |
Changed in linux-oem-6.8 (Ubuntu Jammy): | |
status: | New → Invalid |
Changed in linux-oem-6.8 (Ubuntu Mantic): | |
status: | New → Invalid |
Changed in linux-oem-6.8 (Ubuntu Noble): | |
assignee: | nobody → AceLan Kao (acelankao) |
status: | New → In Progress |
description: | updated |
description: | updated |
Changed in linux (Ubuntu Noble): | |
status: | In Progress → Fix Committed |
tags: |
added: verification-done-noble-linux removed: verification-needed-noble-linux |
Changed in linux (Ubuntu): | |
status: | Fix Committed → Invalid |
Changed in linux-oem-6.8 (Ubuntu): | |
status: | In Progress → Invalid |
This bug is awaiting verification that the linux-oem- 6.5/6.5. 0-1008. 8 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- jammy-linux- oem-6.5' to 'verification- done-jammy- linux-oem- 6.5'. If the problem still exists, change the tag 'verification- needed- jammy-linux- oem-6.5' to 'verification- failed- jammy-linux- oem-6.5' .
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!