All of lore.kernel.org
 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 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.