mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: John Ogness <john.ogness@linutronix.de>
To: Breno Leitao <leitao@debian.org>
Cc: Petr Mladek <pmladek@suse.com>,
	Ryan Roberts <ryan.roberts@arm.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Steven Rostedt <rostedt@goodmis.org>,
	Sergey Senozhatsky <senozhatsky@chromium.org>,
	Rio <rioo.tsukatsukii@gmail.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH v1] panic: Flush unsafe consoles before panic reboot
Date: Thu, 29 Oct 2026 14:59:59 +0106	[thread overview]
Message-ID: <874ie4y5mw.fsf@jogness.linutronix.de> (raw)
In-Reply-To: <auMmQjcy1A6E0eUm@gmail.com>

On 2026-10-29, Breno Leitao <leitao@debian.org> wrote:
> Here is my view right now:
>
> 1) Converting netconsole to NBCON caused an undesired "regression" for
>    users who would prefer the messages go out, even if unsafely. (as
>    Ryan and Meta)

Legacy consoles were never truly synchronous. But, generally speaking,
they were mostly synchronous in normal situations, which was good enough
(and certainly more synchronous than the nbcon netconsole implementation
today).

> 2) It seems the proper solution for this will take a few cycles.

Based on past experience, I agree.

> 3) I would hate to leave users with this "regression" we have today.

Agreed.

> Do you have any suggestion for something doable in the short term?

It depends on what is deemed acceptable. For example, we could introduce
a Kconfig to always unsafe-flush when the kernel ends. This would be a
fairly straight-forward change and require no changes from the user
environment. Later we could add boottime/runtime configuration for this
policy and make it per-console.

If Kconfig changes are not acceptable, we could hardwire unsafe flushing
only for the netconsole (rather than all consoles). This would be a bit
ugly, but it would still help us to identify all the correct points to
enforce a later boottime/runtime policy.

I am open to both of these suggestions.

> Would reverting netconsole's NBCON conversion, while we address the
> dependnecies above, be an option?

That would be unfortunate since the nbcon consoles provide a real
benefit during runtime (full offloading of console printing). Also, the
nbcon callbacks allowed the netconsole code to simplify due to the known
in_task() context.

And I would like to remind that our ultimate goal would be to establish
safe atomic printing for the netconsole. So any workarounds (such as
CON_NBCON_ATOMIC_UNSAFE) are only meant to be temporary (even if
temporary will be measured in years).

John

  reply	other threads:[~2026-10-29 13:54 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-15 10:01 Ryan Roberts
2026-09-15 10:06 ` Ryan Roberts
2026-09-15 18:49   ` Bradley Morgan
2026-09-16  7:44     ` Ryan Roberts
2026-09-16 14:58       ` Bradley Morgan
2026-09-16 13:08 ` Petr Mladek
2026-10-22 20:35   ` Gustavo Luiz Duarte
2026-10-23  7:57   ` John Ogness
2026-10-23 10:36     ` Petr Mladek
2026-10-27 16:12       ` Breno Leitao
2026-10-27 17:43         ` Breno Leitao
2026-10-27 18:01           ` John Ogness
2026-10-28 11:24             ` Breno Leitao
2026-10-28 10:52     ` Breno Leitao
2026-10-28 14:37       ` John Ogness
2026-10-28 21:15         ` Breno Leitao
2026-10-29  9:20           ` John Ogness
2026-10-29 10:58             ` Breno Leitao
2026-10-29 13:53               ` John Ogness [this message]
2026-10-29 14:37                 ` 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=874ie4y5mw.fsf@jogness.linutronix.de \
    --to=john.ogness@linutronix.de \
    --cc=akpm@linux-foundation.org \
    --cc=leitao@debian.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pmladek@suse.com \
    --cc=rioo.tsukatsukii@gmail.com \
    --cc=rostedt@goodmis.org \
    --cc=ryan.roberts@arm.com \
    --cc=senozhatsky@chromium.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®