From: Heiko Carstens <hca@linux.ibm.com>
To: Mete Durlu <meted@linux.ibm.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
Petr Mladek <pmladek@suse.com>, Vasily Gorbik <gor@linux.ibm.com>,
Alexander Gordeev <agordeev@linux.ibm.com>,
Christian Borntraeger <borntraeger@linux.ibm.com>,
Sven Schnelle <svens@linux.ibm.com>,
"David S. Miller" <davem@davemloft.net>,
Andreas Larsson <andreas@gaisler.com>,
Bradley Morgan <include@grrlz.net>,
linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org,
sparclinux@vger.kernel.org
Subject: Re: [PATCH v3 2/3] s390: Implement arch_do_panic
Date: Thu, 30 Jul 2026 16:23:44 +0200 [thread overview]
Message-ID: <20260730142344.70140A38-hca@linux.ibm.com> (raw)
In-Reply-To: <f11500d0-4605-41c6-836a-510801ec6293@linux.ibm.com>
On Thu, Jul 30, 2026 at 03:15:31PM +0200, Mete Durlu wrote:
> On 30/07/2026 13:54, Heiko Carstens wrote:
> > Question: why is it desirable that panic_timeout takes precedence? The result
> > of this change is quite surprising: if anybody (e.g. a distribution) sets
> > CONFIG_PANIC_TIMEOUT to a non-zero value this completely breaks "on_panic"
> > behaviour on s390.
>
> panic timeout can be set during boot or compile time as you said, so it
> can be used to determine what will happen to a system if it panics
> during boot along with after boot. Since panic timeout covers a larger
> area I thought it should get precedence.
> Being able to choose what will happen on panic before boot is a super
> power IMO and would help immensely if one would like to boot an untested
> kernel via kexec for example.
Wouldn't it then not make more sense to add an s390 specific kernel command
line parameter which _disables_ the notifier entirely, and then you get your
panic_timeout working?
Then you have to explicitely state what you want and don't get suprised by
subtle semantic changes.
> > I could understand if this change would result in a larger timeout and
> > additional information being printed, but not that it breaks existing and
> > actually designed and desired behaviour.
>
> For that to happen users have to "misconfigure" the system and try to
> use both panic_timeout and a custom "on_panic" action. The same goes for
> kdump, when kdump is configured "on_panic" actions are ignored silently
> and system always dumps on panic.
This is really not the same. If setting panic_timeout would disable something
else that would be surprising at least, especially since it never did before.
next prev parent reply other threads:[~2026-07-30 14:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-30 9:23 [PATCH v3 0/3] Introduce arch_do_panic Mete Durlu
2026-07-30 9:23 ` [PATCH v3 1/3] panic: " Mete Durlu
2026-07-30 9:45 ` sashiko-bot
2026-07-30 13:03 ` Bradley Morgan
2026-07-30 9:23 ` [PATCH v3 2/3] s390: Implement arch_do_panic Mete Durlu
2026-07-30 10:05 ` sashiko-bot
2026-07-30 11:54 ` Heiko Carstens
2026-07-30 13:15 ` Mete Durlu
2026-07-30 14:23 ` Heiko Carstens [this message]
2026-07-30 9:23 ` [PATCH v3 3/3] sparc: " Mete Durlu
2026-07-30 10:14 ` sashiko-bot
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=20260730142344.70140A38-hca@linux.ibm.com \
--to=hca@linux.ibm.com \
--cc=agordeev@linux.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=andreas@gaisler.com \
--cc=borntraeger@linux.ibm.com \
--cc=davem@davemloft.net \
--cc=gor@linux.ibm.com \
--cc=include@grrlz.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=meted@linux.ibm.com \
--cc=pmladek@suse.com \
--cc=sparclinux@vger.kernel.org \
--cc=svens@linux.ibm.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.