From: "Linux regression tracking (Thorsten Leemhuis)" <regressions@leemhuis.info>
To: David McFarland <corngood@gmail.com>
Cc: "Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
"Linux kernel regressions list" <regressions@lists.linux.dev>,
"Chris Feng" <chris.feng@mediatek.com>,
linux-pm@vger.kernel.org, "Alex Hung" <alexhung@gmail.com>,
"Hans de Goede" <hdegoede@redhat.com>,
"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
"platform-driver-x86@vger.kernel.org"
<platform-driver-x86@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/1] Failure to hibernate on Dell Latitude 7430
Date: Thu, 21 Mar 2024 10:31:18 +0100 [thread overview]
Message-ID: <7ed15eea-24f9-4c75-abf0-f120c7f5b09a@leemhuis.info> (raw)
In-Reply-To: <20240318191153.6978-1-corngood@gmail.com>
On 18.03.24 20:11, David McFarland wrote:
> I have a Dell Latitude 7430, and recently whenever I hibernate with
> `systemctl hibernate`, the machine would immediately wake.
Thanks for the report and the proposed patch.
> I bisected it to:
>
> 0c4cae1bc00d PM: hibernate: Avoid missing wakeup events during hibernation
I CCed the author of that change, which should at least be aware of this
any maybe help out.
Also CCed the people that take care of the code your patch modifies,
they should be the best to judge what should be done here.
FWIW, the start of the thread and the proposed patch can be found here:
https://lore.kernel.org/all/20240318191153.6978-1-corngood@gmail.com/
Ciao, Thorsten
> However, the underlying problem seems to be that during hibernation, my
> system gets a 0xcf (power button release) event, and the above change
> causes it to abort hibernation correctly.
>
> I also noticed that holding the power button down (when it's configured
> to suspend) causes the system to suspend and then wake upon release, if
> it's held long enough.
>
> I'm attaching a patch which fixes the problem for me, by skipping the
> wake on any of the release events. These events are all marked
> KEY_IGNORE, so think this is a reasonable thing to do.
>
> I'm a little worried about the consequences of doing this
> unconditionally in intel-hid. Perhaps it should be a quirk?
>
> David McFarland (1):
> platform/x86/intel/hid: Don't wake on 5-button releases
>
> drivers/platform/x86/intel/hid.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
P.S.: To be sure the issue doesn't fall through the cracks unnoticed,
I'm adding it to regzbot, the Linux kernel regression tracking bot:
#regzbot ^introduced 0c4cae1bc00d
#regzbot title PM: hibernate: & platform/x86/intel/hid: hibernate on
Dell Latitude 7430 fails
#regzbot ignore-activity
--
Everything you wanna know about Linux kernel regression tracking:
https://linux-regtracking.leemhuis.info/about/#tldr
That page also explains what to do if mails like this annoy you.
next parent reply other threads:[~2024-03-21 9:31 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20240318191153.6978-1-corngood@gmail.com>
2024-03-21 9:31 ` Linux regression tracking (Thorsten Leemhuis) [this message]
[not found] ` <20240318191153.6978-2-corngood@gmail.com>
2024-03-29 13:51 ` [PATCH 1/1] platform/x86/intel/hid: Don't wake on 5-button releases Linux regression tracking (Thorsten Leemhuis)
2024-03-29 18:06 ` David McFarland
2024-03-30 6:54 ` Linux regression tracking (Thorsten Leemhuis)
2024-04-01 15:36 ` Enrik Berkhan
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=7ed15eea-24f9-4c75-abf0-f120c7f5b09a@leemhuis.info \
--to=regressions@leemhuis.info \
--cc=alexhung@gmail.com \
--cc=chris.feng@mediatek.com \
--cc=corngood@gmail.com \
--cc=hdegoede@redhat.com \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rafael.j.wysocki@intel.com \
--cc=regressions@lists.linux.dev \
/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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).