linux-kernel - RE: [PATCH printk v2 0/2] Fix reported suspend failures

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <DB9PR04MB8429BD6BD32FD6A9A90F7E2792C8A@DB9PR04MB8429.eurprd04.prod.outlook.com>
Date: Sun, 16 Nov 2025 12:14:31 +0000
From: Sherry Sun <sherry.sun@....com>
To: John Ogness <john.ogness@...utronix.de>, Petr Mladek <pmladek@...e.com>
CC: Sergey Senozhatsky <senozhatsky@...omium.org>, Steven Rostedt
	<rostedt@...dmis.org>, Jacky Bai <ping.bai@....com>, Jon Hunter
	<jonathanh@...dia.com>, Thierry Reding <thierry.reding@...il.com>, Derek
 Barbosa <debarbos@...hat.com>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "stable@...r.kernel.org"
	<stable@...r.kernel.org>
Subject: RE: [PATCH printk v2 0/2] Fix reported suspend failures



> -----Original Message-----
> From: John Ogness <john.ogness@...utronix.de>
> Sent: Friday, November 14, 2025 12:04 AM
> To: Petr Mladek <pmladek@...e.com>
> Cc: Sergey Senozhatsky <senozhatsky@...omium.org>; Steven Rostedt
> <rostedt@...dmis.org>; Sherry Sun <sherry.sun@....com>; Jacky Bai
> <ping.bai@....com>; Jon Hunter <jonathanh@...dia.com>; Thierry Reding
> <thierry.reding@...il.com>; Derek Barbosa <debarbos@...hat.com>; linux-
> kernel@...r.kernel.org; stable@...r.kernel.org
> Subject: [PATCH printk v2 0/2] Fix reported suspend failures
>
> This is v2 of a series to address multiple reports [0][1] (+ 2 offlist) of suspend
> failing when NBCON console drivers are in use. With the help of NXP and
> NVIDIA we were able to isolate the problem and verify the fix.
>
> v1 is here [2].
>
> The first NBCON drivers appeared in 6.13, so currently there is no LTS kernel
> that requires this series. But it should go into 6.17.x and 6.18.
>
> The changes since v1:
>
> - For printk_trigger_flush() add support for all flush types
>   that are available. This will prevent printk_trigger_flush()
>   from trying to inappropriately queue irq_work after this
>   series is applied.
>
> - Add WARN_ON_ONCE() to the printk irq_work queueing functions
>   in case they are called when irq_work is blocked. There
>   should never be (and currently are no) such callers, but
>   these functions are externally available.
>

For this V2 patch set, I have verified it works on i.MX platforms, thanks for the fix.
Tested-by: Sherry Sun <sherry.sun@....com>

Best Regards
Sherry

> John Ogness
>
> [0]
> https://lore.ke/
> rnel.org%2Flkml%2F80b020fc-c18a-4da4-b222-
> 16da1cab2f4c%40nvidia.com&data=05%7C02%7Csherry.sun%40nxp.com%7C
> 9cfc62ea070640d33aaa08de22ce3d98%7C686ea1d3bc2b4c6fa92cd99c5c3016
> 35%7C0%7C0%7C638986466358701215%7CUnknown%7CTWFpbGZsb3d8eyJ
> FbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiT
> WFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=SvtsyYrKyzA4syX%2Fju
> hhKS6vFf8kVPLgR%2FaeMZEfmDQ%3D&reserved=0
> [1]
> https://lore.ke/
> rnel.org%2Flkml%2FDB9PR04MB8429E7DDF2D93C2695DE401D92C4A%40DB
> 9PR04MB8429.eurprd04.prod.outlook.com&data=05%7C02%7Csherry.sun%4
> 0nxp.com%7C9cfc62ea070640d33aaa08de22ce3d98%7C686ea1d3bc2b4c6fa9
> 2cd99c5c301635%7C0%7C0%7C638986466358740800%7CUnknown%7CTWFp
> bGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4z
> MiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=aw9PKics
> 81DBClwWFsavyPS4XcHqGvxC53rUtd%2Fu7yE%3D&reserved=0
> [2]
> https://lore.ke/
> rnel.org%2Flkml%2F20251111144328.887159-1-
> john.ogness%40linutronix.de&data=05%7C02%7Csherry.sun%40nxp.com%7C
> 9cfc62ea070640d33aaa08de22ce3d98%7C686ea1d3bc2b4c6fa92cd99c5c3016
> 35%7C0%7C0%7C638986466358781709%7CUnknown%7CTWFpbGZsb3d8eyJ
> FbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiT
> WFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=JXLraqSBF2HnRs8FmN
> wbBPu9nlyvnDSzx%2BU0AOML0Do%3D&reserved=0
>
> John Ogness (2):
>   printk: Allow printk_trigger_flush() to flush all types
>   printk: Avoid scheduling irq_work on suspend
>
>  kernel/printk/internal.h |  8 ++--
>  kernel/printk/nbcon.c    |  9 ++++-
>  kernel/printk/printk.c   | 81 ++++++++++++++++++++++++++++++++--------
>  3 files changed, 78 insertions(+), 20 deletions(-)
>
>
> base-commit: e9a6fb0bcdd7609be6969112f3fbfcce3b1d4a7c
> --
> 2.47.3