From: Martin Steigerwald <martin@lichtvoll.de>
To: Denis Benato <benato.denis96@gmail.com>,
mario.limonciello@amd.com, rafael@kernel.org,
len.brown@intel.com, pavel@kernel.org,
gregkh@linuxfoundation.org, dakr@kernel.org,
Mario Limonciello <superm1@kernel.org>
Cc: "AceLan Kao" <acelan.kao@canonical.com>,
"Kai-Heng Feng" <kaihengf@nvidia.com>,
"Mark Pearson" <mpearson-lenovo@squebb.ca>,
"Merthan Karakaş" <m3rthn.k@gmail.com>,
linux-pm@vger.kernel.org
Subject: Re: [PATCH] PM: Use hibernate flows for system power off
Date: Wed, 14 May 2025 18:30:04 +0200 [thread overview]
Message-ID: <2993537.e9J7NaK4W3@lichtvoll.de> (raw)
In-Reply-To: <e3aa4d7d-e854-4ee2-aa5d-6371e8aa4ed8@kernel.org>
Hi.
Mario Limonciello - 14.05.25, 18:15:00 CEST:
> > Hello,
> >
> > As for the previous version this patch makes my laptop shutdown
> > cleanly very quickly and I could not link to any regressions or
> > change in behavior while laptop is on.
> >
> > Tested-by: Denis Benato <benato.denis96@gmail.com>
>
> Thanks Denis, that's great news to hear.
Thanks!
> FTR I noticed that this needs some error handling in case the hibernate
> flows fail, so assuming Rafael is generally amenable to this direction
> the next spin I'll add something like this instead to fall back to the
> previous shutdown flow if that happens.
I do not know the context of this patch except this mailing list thread.
However… on various ThinkPads I had the issue of the device not powering
down with certain kernel versions. Not even during a normal shutdown
attempt. Also not with hibernation to disk. Still holding on to an older
kernel version for a X260 for example. But also a T14 AMD Gen 1 is
affected no longer in frequent use currently while a T14 AMD Gen 2 is
absolutely fine. Could this patch help with that?
I will eventually see I bet.
Currently I only compile my own kernel for my current ThinkPad which is
not affected by this issue.
Best,
--
Martin
next prev parent reply other threads:[~2025-05-14 16:36 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-12 21:26 [PATCH] PM: Use hibernate flows for system power off Mario Limonciello
2025-05-14 15:45 ` Denis Benato
2025-05-14 16:15 ` Mario Limonciello
2025-05-14 16:30 ` Martin Steigerwald [this message]
2025-05-14 17:06 ` Mario Limonciello
2025-05-18 8:24 ` Martin Steigerwald
2025-05-19 21:16 ` Mario Limonciello
2025-05-14 17:18 ` Rafael J. Wysocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2993537.e9J7NaK4W3@lichtvoll.de \
--to=martin@lichtvoll.de \
--cc=acelan.kao@canonical.com \
--cc=benato.denis96@gmail.com \
--cc=dakr@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=kaihengf@nvidia.com \
--cc=len.brown@intel.com \
--cc=linux-pm@vger.kernel.org \
--cc=m3rthn.k@gmail.com \
--cc=mario.limonciello@amd.com \
--cc=mpearson-lenovo@squebb.ca \
--cc=pavel@kernel.org \
--cc=rafael@kernel.org \
--cc=superm1@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox