[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <09e24386-de63-e9e9-9e7f-5d04bad62d83@amd.com>
Date: Sun, 28 May 2023 20:14:28 -0500
From: Mario Limonciello <mario.limonciello@....com>
To: Nick Hastings <nicholaschastings@...il.com>
Cc: Salvatore Bonaccorso <carnil@...ian.org>, 1036530@...s.debian.org,
"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 19:56, Nick Hastings wrote:
> Hi,
>
> * Mario Limonciello <mario.limonciello@....com> [230528 21:44]:
>> 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
>>
>
>> Have Nick try using "pcie_port_pm=off" and see if it helps the issue.
>
> I booted into a 6.1 kernel with this option. It has been running without
> problems for 1.5 hours. Usually I would expect the lockup to have
> occurred by now.
>
>> Does this happen in the latest 6.4 RC as well?
>
> I have compiled that kernel and will boot into it after running this one
> with the pcie_port_pm=off for another hour or so.
>
>> I think we need to see a full dmesg and acpidump to better
>> characterize it.
>
> Please find attached. Let me know if there is anything else I can provide.
>
> Regards,
>
> Nick.
I don't see nouveau loading, are you explicitly preventing it from
loading? Can I see the journal from a boot when it reproduced?
Powered by blists - more mailing lists