From: "Paul E. McKenney" <paulmck@kernel.org>
To: Gustavo Luiz Duarte <gustavold@gmail.com>
Cc: Breno Leitao <leitao@debian.org>,
Frederic Weisbecker <frederic@kernel.org>,
Neeraj Upadhyay <neeraj.upadhyay@kernel.org>,
Joel Fernandes <joelagnelf@nvidia.com>,
Josh Triplett <josh@joshtriplett.org>,
Boqun Feng <boqun@kernel.org>,
Uladzislau Rezki <urezki@gmail.com>,
Steven Rostedt <rostedt@goodmis.org>,
Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
Lai Jiangshan <jiangshanlai@gmail.com>,
Zqiang <qiang.zhang@linux.dev>,
rcu@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] rcu: Add BOOTPARAM_RCU_STALL_PANIC Kconfig option
Date: Fri, 13 Mar 2026 10:07:33 -0700 [thread overview]
Message-ID: <3dbfa497-091f-4538-8926-13f33fed3603@paulmck-laptop> (raw)
In-Reply-To: <20260313-panic_on_rcustall-v1-1-a4f8fe54a4bd@gmail.com>
On Fri, Mar 13, 2026 at 09:56:49AM -0700, Gustavo Luiz Duarte wrote:
> Add a Kconfig option to set the default value of the
> kernel.panic_on_rcu_stall sysctl, allowing the kernel to be built
> with panic-on-RCU-stall enabled by default.
>
> This is useful for high-availability systems that require automatic
> recovery (via panic_timeout) when a CPU stall is detected, without
> needing userspace to configure the sysctl at boot.
>
> This follows the pattern established by BOOTPARAM_SOFTLOCKUP_PANIC
> and BOOTPARAM_HUNG_TASK_PANIC. The runtime sysctl can still override
> the Kconfig default.
>
> Signed-off-by: Gustavo Luiz Duarte <gustavold@gmail.com>
Reviewed-by: Paul E. McKenney <paulmck@kernel.org>
> ---
> kernel/rcu/Kconfig.debug | 24 ++++++++++++++++++++++++
> kernel/rcu/tree_stall.h | 2 +-
> 2 files changed, 25 insertions(+), 1 deletion(-)
>
> diff --git a/kernel/rcu/Kconfig.debug b/kernel/rcu/Kconfig.debug
> index 625d75392647..fc2c775eeefc 100644
> --- a/kernel/rcu/Kconfig.debug
> +++ b/kernel/rcu/Kconfig.debug
> @@ -175,6 +175,30 @@ config RCU_CPU_STALL_NOTIFIER
> Say Y here if you want RCU CPU stall notifiers (you don't want them)
> Say N if you are unsure.
>
> +config BOOTPARAM_RCU_STALL_PANIC
> + bool "Panic (reboot) on RCU CPU stall"
> + depends on RCU_STALL_COMMON
> + default n
> + help
> + Say Y here to enable the kernel to panic when an RCU CPU stall
> + is detected.
> +
> + The panic can be used in combination with panic_timeout,
> + to cause the system to reboot automatically after an
> + RCU CPU stall has been detected. This feature is useful for
> + high-availability systems that have uptime guarantees and
> + where a CPU stall must be resolved ASAP.
> +
> + The kernel.max_rcu_stall_to_panic sysctl can be used to set
> + a minimum number of stalls before panicking, allowing the
> + system to tolerate a given number of RCU CPU stalls before
> + triggering a panic.
> +
> + This setting can be overridden at runtime via the
> + kernel.panic_on_rcu_stall sysctl parameter.
> +
> + Say N if unsure.
> +
> config RCU_TRACE
> bool "Enable tracing for RCU"
> depends on DEBUG_KERNEL
> diff --git a/kernel/rcu/tree_stall.h b/kernel/rcu/tree_stall.h
> index b67532cb8770..43ddabf46b5e 100644
> --- a/kernel/rcu/tree_stall.h
> +++ b/kernel/rcu/tree_stall.h
> @@ -17,7 +17,7 @@
> // Controlling CPU stall warnings, including delay calculation.
>
> /* panic() on RCU Stall sysctl. */
> -static int sysctl_panic_on_rcu_stall __read_mostly;
> +static int sysctl_panic_on_rcu_stall __read_mostly = IS_ENABLED(CONFIG_BOOTPARAM_RCU_STALL_PANIC);
> static int sysctl_max_rcu_stall_to_panic __read_mostly;
>
> static const struct ctl_table rcu_stall_sysctl_table[] = {
>
> ---
> base-commit: 1f318b96cc84d7c2ab792fcc0bfd42a7ca890681
> change-id: 20260313-panic_on_rcustall-95709d1abf2f
>
> Best regards,
> --
> Gustavo Luiz Duarte <gustavold@gmail.com>
>
next prev parent reply other threads:[~2026-03-13 17:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-13 16:56 [PATCH] rcu: Add BOOTPARAM_RCU_STALL_PANIC Kconfig option Gustavo Luiz Duarte
2026-03-13 17:07 ` Paul E. McKenney [this message]
2026-03-17 21:39 ` Joel Fernandes
2026-03-13 17:43 ` Breno Leitao
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=3dbfa497-091f-4538-8926-13f33fed3603@paulmck-laptop \
--to=paulmck@kernel.org \
--cc=boqun@kernel.org \
--cc=frederic@kernel.org \
--cc=gustavold@gmail.com \
--cc=jiangshanlai@gmail.com \
--cc=joelagnelf@nvidia.com \
--cc=josh@joshtriplett.org \
--cc=leitao@debian.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@efficios.com \
--cc=neeraj.upadhyay@kernel.org \
--cc=qiang.zhang@linux.dev \
--cc=rcu@vger.kernel.org \
--cc=rostedt@goodmis.org \
--cc=urezki@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox