From: John Ogness <john.ogness@linutronix.de>
To: Sherry Sun <sherry.sun@nxp.com>,
"esben@geanix.com" <esben@geanix.com>,
"pmladek@suse.com" <pmladek@suse.com>,
"senozhatsky@chromium.org" <senozhatsky@chromium.org>,
"tglx@linutronix.de" <tglx@linutronix.de>,
Greg KH <gregkh@linuxfoundation.org>,
Jiri Slaby <jirislaby@kernel.org>,
Shawn Guo <shawnguo@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
"ryotkkr98@gmail.com" <ryotkkr98@gmail.com>,
"kkartik@nvidia.com" <kkartik@nvidia.com>,
"fj6611ie@aa.jp.fujitsu.com" <fj6611ie@aa.jp.fujitsu.com>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
linux-serial <linux-serial@vger.kernel.org>,
"imx@lists.linux.dev" <imx@lists.linux.dev>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>
Subject: RE: [BUG] Suspend failure with nbcon + pm_debug_messages (Linux 6.18-rc4)
Date: Wed, 05 Nov 2025 00:25:55 +0106 [thread overview]
Message-ID: <87bjlhz8pw.fsf@jogness.linutronix.de> (raw)
In-Reply-To: <DB9PR04MB8429F50811DDC648DD8A7B8792C4A@DB9PR04MB8429.eurprd04.prod.outlook.com>
Hi Sherry,
On 2025-11-04, Sherry Sun <sherry.sun@nxp.com> wrote:
>> Thanks for confirming the fix. I will make an official post on LKML
>> with the patch today. Since the i.MX nbcon-driver is already
>> mainline, I will CC stable. I will CC you as well.
>
> I'll keep an eye on any fixes you send later and verify them if
> necessary. Hope this fix can be applied to the mainline and the stable
> tree ASAP.
Even with this patch [1] I am able to reproduce the problem on one of my
test boards. I am using the following extra boot arguments:
no_console_suspend loglevel=8 initcall_debug
And then:
# echo 1 > /sys/power/pm_debug_messages
# echo mem > /sys/power/state
This generates a lot of output and leads to the same suspend
failure. This may be due to the out-of-tree nbcon-8250 driver I am
using, which introduces extra irq_work usage within the driver. Could
you please try the above boot arguments with your i.MX board?
I am wondering if blocking the queueing of irq_work during suspend
should be generally implemented rather than just printk avoiding it. It
also seems to only be a problem for ARM32 so perhaps it needs to be
addressed at the architecture level.
John Ogness
>> [1] https://github.com/Linutronix/linux/commit/ae173249d9028ef159fba040bdab260d80dda43f
next prev parent reply other threads:[~2025-11-04 23:19 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-04 7:44 [BUG] Suspend failure with nbcon + pm_debug_messages (Linux 6.18-rc4) Sherry Sun
2025-11-04 8:28 ` John Ogness
2025-11-04 9:46 ` Sherry Sun
2025-11-04 10:16 ` John Ogness
2025-11-04 10:43 ` Sherry Sun
2025-11-04 23:19 ` John Ogness [this message]
2025-11-05 3:43 ` Sherry Sun
2025-11-05 7:30 ` Petr Mladek
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=87bjlhz8pw.fsf@jogness.linutronix.de \
--to=john.ogness@linutronix.de \
--cc=esben@geanix.com \
--cc=fj6611ie@aa.jp.fujitsu.com \
--cc=gregkh@linuxfoundation.org \
--cc=imx@lists.linux.dev \
--cc=jirislaby@kernel.org \
--cc=kernel@pengutronix.de \
--cc=kkartik@nvidia.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=pmladek@suse.com \
--cc=ryotkkr98@gmail.com \
--cc=s.hauer@pengutronix.de \
--cc=senozhatsky@chromium.org \
--cc=shawnguo@kernel.org \
--cc=sherry.sun@nxp.com \
--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