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] [thread-next>] [day] [month] [year] [list]
Message-ID: <f5a833c4-6346-4ec4-8360-4a818342fb52@paulmck-laptop>
Date: Tue, 15 Oct 2024 16:02:37 -0700
From: "Paul E. McKenney" <paulmck@...nel.org>
To: Joel Fernandes <joel@...lfernandes.org>
Cc: frederic@...nel.org, rcu@...r.kernel.org, linux-kernel@...r.kernel.org,
	kernel-team@...a.com, rostedt@...dmis.org
Subject: Re: [PATCH rcu 0/3] RCU CPU stall-warning changes for v6.13

On Tue, Oct 15, 2024 at 02:49:06PM -0400, Joel Fernandes wrote:
> On Wed, Oct 09, 2024 at 11:05:02AM -0700, Paul E. McKenney wrote:
> > Hello!
> > 
> > This series contains RCU CPU stall-warning changes for v6.13:
> > 
> > 1.	Delete unused rcu_gp_might_be_stalled() function.
> > 
> > 2.	Stop stall warning from dumping stacks if grace period ends.
> > 
> > 3.	Finer-grained grace-period-end checks in rcu_dump_cpu_stacks().
> > 
> 
> Other than small nit in 2/3, 
> 
> Reviewed-by: Joel Fernandes (Google) <joel@...lfernandes.org>

Applied, thank you, and I also added the data_race() in 2/3.

> I was curious if you're seeing perf or other improvements with the
> finer-grained rnp locking.

This is about robustness rather that performance, though I suppose you
could argue that lack of robustness is an extreme form of bad performance.
Holding the leaf rcu_node locks for too long while dumping stacks can
result in things like CSD-lock timeouts due to the dumping CPU having
interrupts disabled for an extended period.

And earlier commit, 1ecd9d68eb44 ("rcu: Defer printing stall-warning
backtrace when holding rcu_node lock"), took care of most of the issue
by deferring the actual output.  But while in the area, it seemed wise
to avoid up to 64 dumps being generated (but no longer printed) while
holding a leaf rcu_node lock.

Hence this commit.

							Thanx, Paul

> thanks,
> 
>  - Joel
> 
> 
> > 						Thanx, Paul
> > 
> > ------------------------------------------------------------------------
> > 
> >  b/include/linux/rcutiny.h |    1 -
> >  b/include/linux/rcutree.h |    1 -
> >  b/kernel/rcu/tree_stall.h |   30 ------------------------------
> >  kernel/rcu/tree_stall.h   |   35 ++++++++++++++++++++++-------------
> >  4 files changed, 22 insertions(+), 45 deletions(-)
> > 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ