linux-kernel - Re: [PATCH 2/2] Revert "PCI/ASPM: Refactor L1 PM Substates Control Register programming"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20230204174525.GA917@wunner.de>
Date:   Sat, 4 Feb 2023 18:45:25 +0100
From:   Lukas Wunner <lukas@...ner.de>
To:     Bjorn Helgaas <helgaas@...nel.org>
Cc:     linux-pci@...r.kernel.org, Thomas Witt <kernel@...t.link>,
        Vidya Sagar <vidyas@...dia.com>, linux-kernel@...r.kernel.org,
        Bjorn Helgaas <bhelgaas@...gle.com>
Subject: Re: [PATCH 2/2] Revert "PCI/ASPM: Refactor L1 PM Substates Control
 Register programming"

On Fri, Feb 03, 2023 at 04:48:20PM -0600, Bjorn Helgaas wrote:
> Thomas Witt reported that 5e85eba6f50d ("PCI/ASPM: Refactor L1 PM Substates
> Control Register programming") broke suspend/resume on a Tuxedo
> Infinitybook S 14 v5, which seems to use a Clevo L140CU Mainboard.
> 
> The main symptom is:
> 
>   iwlwifi 0000:02:00.0: Unable to change power state from D3hot to D0, device inaccessible
>   nvme 0000:03:00.0: Unable to change power state from D3hot to D0, device inaccessible
> 
> and the machine is only partially usable after resume.  It can't run dmesg
> and can't do a clean reboot.  This happens on every suspend/resume cycle.
> 
> Revert 5e85eba6f50d until we can figure out the root cause.
> 
> Fixes: https://bugzilla.kernel.org/show_bug.cgi?id=216877

Link: https://bugzilla.kernel.org/show_bug.cgi?id=216877
Fixes: 5e85eba6f50d ("PCI/ASPM: Refactor L1 PM Substates Control Register programming")
Cc: stable@...r.kernel.org # v6.1+

> Reported-by: Thomas Witt <kernel@...t.link>
> Tested-by: Thomas Witt <kernel@...t.link>
> Signed-off-by: Bjorn Helgaas <bhelgaas@...gle.com>
> Cc: Vidya Sagar <vidyas@...dia.com>