[SRU][N][PATCH v2 0/3] UBUNTU: SAUCE: System hang when running memory/memory_stress_ng test

[Stefan Bader]

On 12.11.24 05:09, En-Wei Wu wrote:
> BugLink: https://bugs.launchpad.net/bugs/2085092
> 
> [Impact]
> memory/memory_stress_ng test failed when swapping due to the ASPM issue with VMD device.
> 
> [Fix]
> Backport the Endlessos's patchset from https://lore.kernel.org (has been reviewed):
> 
> Link: https://lore.kernel.org/linux-pci/20241001083438.10070-2-jhp@endlessos.org/

The discussion there seems to have started up again towards the end. 
Hard for me to tell how fundamental the raised questions are. From my 
gut feeling the patches might still change between there and the final 
revision.
> 
> [PATCH v12 0/3] PCI: vmd: Enable PCI PM's L1 substates of remapped PCIe Root Port and NVMe
> [PATCH v12 1/3] PCI: vmd: Set PCI devices to D0 before enable PCI PM's L1 substates
> [PATCH v12 2/3] PCI/ASPM: Add notes about enabling PCI-PM L1SS to pci_enable_link_state(_locked)
> [PATCH v12 3/3] PCI/ASPM: Make pci_save_aspm_l1ss_state save both child and parent's L1SS configuration
> 
> [Test Plan]
> 1. Boot up Ubuntu OS
> 2. $ checkbox-cli run com.canonical.certification::stress/store_and_change_oomd_config
> 3. $ checkbox-cli run com.canonical.certification::memory/memory_stress_ng
> 
> [Where problems could occur]
> May cause ASPM regression.

That is the worry here. This was something that felt always like a 
feature where fixing one end would cause issues on the other end. Which 
would steer myself to at least wait for the set hitting linux-next.

> 
> 
> Jian-Hong Pan (3):
>    UBUNTU: SAUCE: PCI: vmd: Set PCI devices to D0 before enable PCI PM's
>      L1 substates
>    UBUNTU: SAUCE: PCI/ASPM: Add notes about enabling PCI-PM L1SS to
>      pci_enable_link_state(_locked)
>    UBUNTU: SAUCE: PCI/ASPM: Make pci_save_aspm_l1ss_state save both child
>      and parent's L1SS configuration
> 
>   drivers/pci/controller/vmd.c | 11 +++++++++--
>   drivers/pci/pcie/aspm.c      | 26 +++++++++++++++++++++++++-
>   2 files changed, 34 insertions(+), 3 deletions(-)
> 

-- 
- Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE8675DEECBEECEA3.asc
Type: application/pgp-keys
Size: 48643 bytes
Desc: OpenPGP public key
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20241112/4f21a325/attachment-0001.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20241112/4f21a325/attachment-0001.sig>