Linux Power Management development
 help / color / mirror / Atom feed
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



  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