Re: [PATCH rcu 1/3] srcu: Remove smp_mb() from srcu_read_unlock_lite()

[Neeraj Upadhyay]

On 11/12/2024 7:01 AM, Paul E. McKenney wrote:
> The srcu_read_unlock_lite() function invokes __srcu_read_unlock() instead
> of __srcu_read_unlock_lite(), which means that it is doing an unnecessary
> smp_mb().  This is harmless other than the performance degradation.
> 
> This commit therefore switches to __srcu_read_unlock_lite().
> 
> Reported-by: Neeraj Upadhyay <neeraj.upadh...@amd.com>
> Closes: 
> https://lore.kernel.org/all/d07e8f4a-d5ff-4c8e-8e61-50db285c5...@amd.com/
> Fixes: c0f08d6b5a61 ("srcu: Add srcu_read_lock_lite() and 
> srcu_read_unlock_lite()")
> Signed-off-by: Paul E. McKenney <paul...@kernel.org>
> Cc: Frederic Weisbecker <frede...@kernel.org>
> ---

Reviewed-by: Neeraj Upadhyay <neeraj.upadh...@amd.com>


- Neeraj