216768 – Asus Tuf A15 Ryzen 6800H - Lost ACPI events after resume from suspend

Bug 216768 - Asus Tuf A15 Ryzen 6800H - Lost ACPI events after resume from suspend

Summary: Asus Tuf A15 Ryzen 6800H - Lost ACPI events after resume from suspend

Status:	RESOLVED CODE_FIX

Alias:	None

Product:	Power Management
Classification:	Unclassified
Component:	Hibernation/Suspend (show other bugs)
Hardware:	AMD Linux

Importance:	P1 normal
Assignee:	Mario Limonciello (AMD)

URL:
Keywords:

Depends on:
Blocks:

Reported:	2022-12-03 16:32 UTC by Paul
Modified:	2023-01-04 18:51 UTC (History)
CC List:	3 users (show)

See Also:
Kernel Version:	6.1.0-rc7
Subsystem:
Regression:	No
Bisected commit-id:

Attachments
ACPI dump (1.27 MB, text/plain) 2022-12-03 16:35 UTC, Paul	Details
dmidecode dump (13.45 KB, text/plain) 2022-12-03 16:36 UTC, Paul	Details
dmesg dump (129.17 KB, text/plain) 2022-12-03 16:37 UTC, Paul	Details
PATCH V1 (1.29 KB, patch) 2022-12-05 21:27 UTC, Mario Limonciello (AMD)	Details \| Diff
dmidecode for FA507RE (12.48 KB, text/plain) 2022-12-09 16:36 UTC, bilkow	Details
Add an attachment (proposed patch, testcase, etc.)

Description Paul 2022-12-03 16:32:31 UTC

Comment 1 Paul 2022-12-03 16:35:55 UTC

Created attachment 303347 [details]
ACPI dump

Comment 2 Paul 2022-12-03 16:36:35 UTC

Created attachment 303348 [details]
dmidecode dump

Comment 3 Paul 2022-12-03 16:37:46 UTC

Created attachment 303349 [details]
dmesg dump

Comment 4 Paul 2022-12-03 16:46:19 UTC

This is a follow up from:

https://bugzilla.kernel.org/show_bug.cgi?id=216101

Tested on latest BIOS revision available and on 6.1.0-rc7.

The issue persists with the addition that the display brightness fn keys do not work, even before the laptop has been in suspend mode. I hope the logs can help.

(apologies for the edits, a field was highlighted in the form and I pressed enter, which submitted the report)

Comment 5 Mario Limonciello (AMD) 2022-12-05 21:19:02 UTC

Can you please have a try with the kernel command line option 'acpi.prefer_microsoft_dsm_guid=1'?

If that helps, then please try the attached patch (with no parameter).

Comment 6 Mario Limonciello (AMD) 2022-12-05 21:27:34 UTC

Created attachment 303364 [details]
PATCH V1

Comment 7 bilkow 2022-12-09 16:34:12 UTC

Hi, I also have an Asus Tuf A15 (2022) Ryzen 6800H, but my version is FA507RE.

Trying 'acpi.prefer_microsoft_dsm_guid=1' on Linux 6.1.0-RC8 works for me. I haven't tried the patch as it's only for FA507RM.

One thing to note is that if you match as you did for the A17 https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git/commit/?h=bleeding-edge&id=d0f61e89f08dd46a090da50f5d747204673f70ea, it may also patch the 2021 version, which has either a Ryzen 9 5900HX or Ryzen 5 4600H and may not have that same problem.

Comment 8 bilkow 2022-12-09 16:35:27 UTC

Also, I forgot to mention that without the parameter I only had problems after suspend. Before suspend my brightness keys always worked.

Comment 9 bilkow 2022-12-09 16:36:58 UTC

Created attachment 303389 [details]
dmidecode for FA507RE

Comment 10 Mario Limonciello (AMD) 2022-12-15 19:46:06 UTC

I just sent out this series https://lore.kernel.org/linux-acpi/20221215191617.1438-1-mario.limonciello@amd.com/T/#t for review that drops the Rembrandt behavior and parameter.  Here's a new branch https://gitlab.freedesktop.org/superm1/linux/-/commits/mlimonci/gitlab-2292-v3 with the same patches.

I expect it should help both of you guys.

Comment 11 bilkow 2022-12-17 02:42:20 UTC

Thanks a lot!

Tried your branch and resuming works as expected.

Comment 12 Mario Limonciello (AMD) 2022-12-31 05:57:42 UTC

Patches are merged into Linus' master tree.  6.2-rc2 or later have them merged, and they are CC to 6.1 stable so should eventually be there too.

Comment 13 Mario Limonciello (AMD) 2023-01-04 18:51:18 UTC

It's merged for 6.1.3.

Note You need to log in before you can comment on or make changes to this bug.