Commit 5fcb3a5f authored by Paul E. McKenney's avatar Paul E. McKenney
Browse files

rcu: Mark accesses to ->rcu_read_lock_nesting 



KCSAN flags accesses to ->rcu_read_lock_nesting as data races, but
in the past, the overhead of marked accesses was excessive.  However,
that was long ago, and much has changed since then, both in terms of
hardware and of compilers.  Here is data taken on an eight-core laptop
using Intel(R) Core(TM) i9-10885H CPU @ 2.40GHz with a kernel built
using gcc version 9.3.0, with all data in nanoseconds.

Unmarked accesses (status quo), measured by three refscale runs:

	Minimum reader duration:  3.286  2.851  3.395
	Median reader duration:   3.698  3.531  3.4695
	Maximum reader duration:  4.481  5.215  5.157

Marked accesses, also measured by three refscale runs:

	Minimum reader duration:  3.501  3.677  3.580
	Median reader duration:   4.053  3.723  3.895
	Maximum reader duration:  7.307  4.999  5.511

This focused microbenhmark shows only sub-nanosecond differences which
are unlikely to be visible at the system level.  This commit therefore
marks data-racing accesses to ->rcu_read_lock_nesting.

Signed-off-by: default avatarPaul E. McKenney <paulmck@kernel.org>
parent 2be57f73

include/linux/rcupdate.h

+1 −1
Original line number Diff line number Diff line
@@ -53,7 +53,7 @@ void __rcu_read_unlock(void); 
 * nesting depth, but makes sense only if CONFIG_PREEMPT_RCU -- in other

 * types of kernel builds, the rcu_read_lock() nesting depth is unknowable.

 */

#define rcu_preempt_depth() (current->rcu_read_lock_nesting)

#define rcu_preempt_depth() READ_ONCE(current->rcu_read_lock_nesting)



#else /* #ifdef CONFIG_PREEMPT_RCU */

kernel/rcu/tree_plugin.h

+6 −3
Original line number Diff line number Diff line
@@ -405,17 +405,20 @@ static int rcu_preempt_blocked_readers_cgp(struct rcu_node *rnp) 


static void rcu_preempt_read_enter(void)

{

	current->rcu_read_lock_nesting++;

	WRITE_ONCE(current->rcu_read_lock_nesting, READ_ONCE(current->rcu_read_lock_nesting) + 1);

}



static int rcu_preempt_read_exit(void)

{

	return --current->rcu_read_lock_nesting;

	int ret = READ_ONCE(current->rcu_read_lock_nesting) - 1;



	WRITE_ONCE(current->rcu_read_lock_nesting, ret);

	return ret;

}



static void rcu_preempt_depth_set(int val)

{

	current->rcu_read_lock_nesting = val;

	WRITE_ONCE(current->rcu_read_lock_nesting, val);

}



/*