public inbox for rcu@vger.kernel.org
 help / color / mirror / Atom feed
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>
> 

  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