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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.