linux-kernel - Re: Regression from "ACPI: OSI: Remove Linux-Dell-Video _OSI string"? (was: Re: Bug#1036530: linux-signed-amd64: Hard lock up of system)

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]

Date:   Sun, 28 May 2023 07:44:05 -0500
From:   Mario Limonciello <mario.limonciello@....com>
To:     Salvatore Bonaccorso <carnil@...ian.org>,
        Nick Hastings <nicholaschastings@...il.com>,
        1036530@...s.debian.org
Cc:     "Rafael J. Wysocki" <rafael@...nel.org>,
        Len Brown <lenb@...nel.org>, linux-acpi@...r.kernel.org,
        linux-kernel@...r.kernel.org, regressions@...ts.linux.dev
Subject: Re: Regression from "ACPI: OSI: Remove Linux-Dell-Video _OSI string"?
 (was: Re: Bug#1036530: linux-signed-amd64: Hard lock up of system)

On 5/28/23 01:49, Salvatore Bonaccorso wrote:
> Hi Mario
> 
> Nick Hastings reported in Debian in https://bugs.debian.org/1036530
> lockups from his system after updating from a 6.0 based version to
> 6.1.y. >
> #regzbot ^introduced 24867516f06d
> 
> he bisected the issue and tracked it down to:
> 
> On Sun, May 28, 2023 at 10:14:51AM +0900, Nick Hastings wrote:
>> Control: tags -1 - moreinfo
>>
>> Hi,
>>
>> I repeated the git bisect, and the bad commit seems to be:
>>
>> (git)-[v6.1-rc1~206^2~4^5~3|bisect] % git bisect bad
>> 24867516f06dabedef3be7eea0ef0846b91538bc is the first bad commit
>> commit 24867516f06dabedef3be7eea0ef0846b91538bc
>> Author: Mario Limonciello <mario.limonciello@....com>
>> Date:   Tue Aug 23 13:51:31 2022 -0500
>>
>>      ACPI: OSI: Remove Linux-Dell-Video _OSI string
>>      
>>      This string was introduced because drivers for NVIDIA hardware
>>      had bugs supporting RTD3 in the past.
>>      
>>      Before proprietary NVIDIA driver started to support RTD3, Ubuntu had
>>      had a mechanism for switching PRIME on and off, though it had required
>>      to logout/login to make the library switch happen.
>>      
>>      When the PRIME had been off, the mechanism had unloaded the NVIDIA
>>      driver and put the device into D3cold, but the GPU had never come back
>>      to D0 again which is why ODMs used the _OSI to expose an old _DSM
>>      method to switch the power on/off.
>>      
>>      That has been fixed by commit 5775b843a619 ("PCI: Restore config space
>>      on runtime resume despite being unbound"). so vendors shouldn't be
>>      using this string to modify ASL any more.
>>      
>>      Reviewed-by: Lyude Paul <lyude@...hat.com>
>>      Signed-off-by: Mario Limonciello <mario.limonciello@....com>
>>      Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
>>
>>   drivers/acpi/osi.c | 9 ---------
>>   1 file changed, 9 deletions(-)
>>
>> This machine is a Dell with an nvidia chip so it looks like this really
>> could be the commit that that is causing the problems. The description
>> of the commit also seems (to my untrained eye) to be consistent with the
>> error reported on the console when the lockup occurs:
>>
>> [   58.729863] ACPI Error: Aborting method \_SB.PCI0.PGON due to previous error (AE_AML_LOOP_TIMEOUT) (20220331/psparse-529)
>> [   58.729904] ACPI Error: Aborting method \_SB.PCI0.PEG0.PG00._ON due to previous error (AE_AML_LOOP_TIMEOUT) (20220331/psparse-529)
>> [   60.083261] vfio-pci 0000:01:00.0 Unable to change power state from D3cold to D0, device inaccessible
>>
>> Hopefully this is enough information for experts to resolve this.
> 
> Does this ring some bell for you? Do you need any further information
> from Nick?
> 
> Regards,
> Salvatore

Hi Salvatore,

Have Nick try using "pcie_port_pm=off" and see if it helps the issue.

Does this happen in the latest 6.4 RC as well?

I think we need to see a full dmesg and acpidump to better characterize it.