From: Muhammad Usama Anjum <usama.anjum@collabora.com>
To: Mario Limonciello <superm1@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: usama.anjum@collabora.com,
"Rafael J. Wysocki" <rafael@kernel.org>,
Len Brown <lenb@kernel.org>, Pavel Machek <pavel@kernel.org>,
Danilo Krummrich <dakr@kernel.org>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Thomas Gleixner <tglx@linutronix.de>,
Peter Zijlstra <peterz@infradead.org>,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pm@vger.kernel.org, linux-input@vger.kernel.org,
kernel@collabora.com, Askar Safin <safinaskar@gmail.com>
Subject: Re: [PATCH 2/4] ACPI: button: Cancel hibernation if button is pressed during hibernation
Date: Wed, 26 Nov 2025 18:26:36 +0500 [thread overview]
Message-ID: <d4ae5b48-d437-494d-8253-0bb66fedd78e@collabora.com> (raw)
In-Reply-To: <92de74fb-fc75-4260-bb1d-2d6dd522183f@kernel.org>
On 11/26/25 5:55 PM, Mario Limonciello wrote:
>>> First I need a board/machine with serial console access to view all logs in real
>>> time. :)
>>
>> usb debug cables might be your solution.
>>
> Just a word of warning before you go too far down this path to get a console working from XHCI debug.
>
> This is probably a Hal changing a light bulb problem [1]. Last time I tried XHCI debug cables on some modern AMD systems I ran into a problem that the BAR is too big for early_ioremap(). Looking through LKML it's come up a few times in the past too [2] [3].
>
> Link: https://youtu.be/5W4NFcamRhM?si=qOFrCTzvK6-H-4AX [1]
> Link: https://lore.kernel.org/linux-usb/ZCOq3PUBCtHkwdnw@mail-itl/ [2]
> Link: https://lore.kernel.org/linux-usb/CAAcb1K_bezseTM8DrOrzVUi_W+nZoE2N0CO4k3AQWPw7=7pyjw@mail.gmail.com/ [3]
>
> The other obvious idea is to use netconsole, but you need a PCIe Ethernet controller, but I think you'll have more success in development using KVM as these are generic architectural problems.
>
> To help you get started with this I may point out something that was shared to me for another hibernate bug [4].
>
> Link: https://lore.kernel.org/linux-pm/20251105180506.137448-1-safinaskar@gmail.com/ [4]
>
> Askar Safin (CC'ed) produced a script that does a very minimal kernel build, sets up a VM with the right sizes of disks/swap/etc. It's trivial to make kernel changes and re-run the script, and you can also attach a debugger to the KVM instance. Maybe you can adapt something like that. You can wrap it with 'time' calls to actually measure performance for any ideas and prove them out too.
I was just going to try it. Thank you so much for double confirming. I'll
test and see.
--
---
Thanks,
Usama
next prev parent reply other threads:[~2025-11-26 13:27 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-07 18:44 [PATCH 0/4] PM: Hibernate: Add hibernation cancellation support Muhammad Usama Anjum
2025-11-07 18:44 ` [PATCH 1/4] PM: hibernate: export pm_sleep_transition_in_progress() Muhammad Usama Anjum
2025-11-24 17:01 ` Greg Kroah-Hartman
2025-11-25 16:44 ` Muhammad Usama Anjum
2025-11-07 18:44 ` [PATCH 2/4] ACPI: button: Cancel hibernation if button is pressed during hibernation Muhammad Usama Anjum
2025-11-24 17:03 ` Greg Kroah-Hartman
2025-11-25 11:12 ` Muhammad Usama Anjum
2025-11-25 11:47 ` Greg Kroah-Hartman
2025-11-25 16:41 ` Muhammad Usama Anjum
2025-11-26 7:38 ` Greg Kroah-Hartman
2025-11-26 12:55 ` Mario Limonciello
2025-11-26 13:26 ` Muhammad Usama Anjum [this message]
2025-11-26 13:41 ` Muhammad Usama Anjum
2025-11-24 18:42 ` Rafael J. Wysocki
2025-11-28 14:17 ` Muhammad Usama Anjum
2025-11-07 18:44 ` [PATCH 3/4] Input: Ignore the KEY_POWER events if hibernation is in progress Muhammad Usama Anjum
2025-11-24 18:50 ` Rafael J. Wysocki
2025-11-25 10:22 ` Muhammad Usama Anjum
2025-11-25 12:25 ` Rafael J. Wysocki
2025-11-25 16:05 ` Muhammad Usama Anjum
2025-11-28 17:00 ` Muhammad Usama Anjum
2025-11-07 18:44 ` [PATCH 4/4] PM: sleep: clear pm_abort_suspend at suspend Muhammad Usama Anjum
2025-11-24 18:54 ` Rafael J. Wysocki
2025-11-25 9:53 ` Muhammad Usama Anjum
2025-11-25 12:59 ` Rafael J. Wysocki
2025-11-24 13:03 ` [PATCH 0/4] PM: Hibernate: Add hibernation cancellation support Muhammad Usama Anjum
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=d4ae5b48-d437-494d-8253-0bb66fedd78e@collabora.com \
--to=usama.anjum@collabora.com \
--cc=dakr@kernel.org \
--cc=dmitry.torokhov@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=kernel@collabora.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=pavel@kernel.org \
--cc=peterz@infradead.org \
--cc=rafael@kernel.org \
--cc=safinaskar@gmail.com \
--cc=superm1@kernel.org \
--cc=tglx@linutronix.de \
/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 NNTP newsgroup(s).