mbox series

[SRU,0/2,N] ACPI: PM: s2idle: Linux ACPI s2idle patches to fix suspend/resume issue

Message ID 20240619120651.1381251-1-thibault.ferrante@canonical.com
Headers show
Series ACPI: PM: s2idle: Linux ACPI s2idle patches to fix suspend/resume issue | expand

Message

Thibault Ferrante June 19, 2024, 12:06 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2069231

[ Impact ]

Framework (https://frame.work/) has root cause an artifact with their MTL
laptops being unable to suspend and the reason is similar to many other MTL systems,
their ACPI tables use the modern Microsoft UUIDs for Modern Standby.

[ Fix ]

Clean cherry-pick of two upstream commits merged in 6.9.

[ Test Plan ]

Go to suspend with a framework laptop.

[ Where problems could occur ]

Regression can happen when going into suspend.

Rafael J. Wysocki (2):
  ACPI: PM: s2idle: Enable Low-Power S0 Idle MSFT UUID for non-AMD
    systems
  ACPI: PM: s2idle: Evaluate all Low-Power S0 Idle _DSM functions

 drivers/acpi/x86/s2idle.c | 35 +++++++++++++++++++++++++----------
 1 file changed, 25 insertions(+), 10 deletions(-)

Comments

Manuel Diewald June 19, 2024, 2:13 p.m. UTC | #1
On Wed, Jun 19, 2024 at 02:06:49PM +0200, Thibault Ferrante wrote:
> BugLink: https://bugs.launchpad.net/bugs/2069231
> 
> [ Impact ]
> 
> Framework (https://frame.work/) has root cause an artifact with their MTL
> laptops being unable to suspend and the reason is similar to many other MTL systems,
> their ACPI tables use the modern Microsoft UUIDs for Modern Standby.
> 
> [ Fix ]
> 
> Clean cherry-pick of two upstream commits merged in 6.9.
> 
> [ Test Plan ]
> 
> Go to suspend with a framework laptop.
> 
> [ Where problems could occur ]
> 
> Regression can happen when going into suspend.
> 
> Rafael J. Wysocki (2):
>   ACPI: PM: s2idle: Enable Low-Power S0 Idle MSFT UUID for non-AMD
>     systems
>   ACPI: PM: s2idle: Evaluate all Low-Power S0 Idle _DSM functions
> 
>  drivers/acpi/x86/s2idle.c | 35 +++++++++++++++++++++++++----------
>  1 file changed, 25 insertions(+), 10 deletions(-)
> 
> -- 
> 2.43.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Acked-by: Manuel Diewald <manuel.diewald@canonical.com>
Chris Chiu June 20, 2024, 2:56 a.m. UTC | #2
On Wed, Jun 19, 2024 at 8:09 PM Thibault Ferrante
<thibault.ferrante@canonical.com> wrote:
>
> BugLink: https://bugs.launchpad.net/bugs/2069231
>
> [ Impact ]
>
> Framework (https://frame.work/) has root cause an artifact with their MTL
> laptops being unable to suspend and the reason is similar to many other MTL systems,
> their ACPI tables use the modern Microsoft UUIDs for Modern Standby.
>
> [ Fix ]
>
> Clean cherry-pick of two upstream commits merged in 6.9.
>
> [ Test Plan ]
>
> Go to suspend with a framework laptop.
>
> [ Where problems could occur ]
>
> Regression can happen when going into suspend.
>
> Rafael J. Wysocki (2):
>   ACPI: PM: s2idle: Enable Low-Power S0 Idle MSFT UUID for non-AMD
>     systems
>   ACPI: PM: s2idle: Evaluate all Low-Power S0 Idle _DSM functions
>
>  drivers/acpi/x86/s2idle.c | 35 +++++++++++++++++++++++++----------
>  1 file changed, 25 insertions(+), 10 deletions(-)
>
> --
> 2.43.0
>
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Acked-by: Chris Chiu <chris.chiu@canonical.com>
Paolo Pisati June 20, 2024, 1:56 p.m. UTC | #3
On Wed, Jun 19, 2024 at 02:06:49PM +0200, Thibault Ferrante wrote:
> BugLink: https://bugs.launchpad.net/bugs/2069231

Clean cherry-picks.

Acked-by: Paolo Pisati <paolo.pisati@canonical.com>
Stefan Bader June 21, 2024, 2:06 p.m. UTC | #4
On 19.06.24 14:06, Thibault Ferrante wrote:
> BugLink: https://bugs.launchpad.net/bugs/2069231
> 
> [ Impact ]
> 
> Framework (https://frame.work/) has root cause an artifact with their MTL
> laptops being unable to suspend and the reason is similar to many other MTL systems,
> their ACPI tables use the modern Microsoft UUIDs for Modern Standby.
> 
> [ Fix ]
> 
> Clean cherry-pick of two upstream commits merged in 6.9.
> 
> [ Test Plan ]
> 
> Go to suspend with a framework laptop.
> 
> [ Where problems could occur ]
> 
> Regression can happen when going into suspend.
> 
> Rafael J. Wysocki (2):
>    ACPI: PM: s2idle: Enable Low-Power S0 Idle MSFT UUID for non-AMD
>      systems
>    ACPI: PM: s2idle: Evaluate all Low-Power S0 Idle _DSM functions
> 
>   drivers/acpi/x86/s2idle.c | 35 +++++++++++++++++++++++++----------
>   1 file changed, 25 insertions(+), 10 deletions(-)
> 

Applied to noble:linux/master-next. Thanks.

-Stefan