From: Paul E. McKenney Date: Mon, 11 Nov 2024 17:38:26 +0000 (-0800) Subject: srcu: Remove smp_mb() from srcu_read_unlock_lite() X-Git-Tag: v6.13-rc1~190^2^5~2 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=174dd22a781b97cb355b1037f0931436a254d3be;p=thirdparty%2Fkernel%2Flinux.git srcu: Remove smp_mb() from srcu_read_unlock_lite() 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 Closes: https://lore.kernel.org/all/d07e8f4a-d5ff-4c8e-8e61-50db285c57e9@amd.com/ Fixes: c0f08d6b5a61 ("srcu: Add srcu_read_lock_lite() and srcu_read_unlock_lite()") Signed-off-by: Paul E. McKenney Cc: Frederic Weisbecker Reviewed-by: Neeraj Upadhyay Signed-off-by: Frederic Weisbecker --- diff --git a/include/linux/srcu.h b/include/linux/srcu.h index bab1dae3f69e6..56f83237de4d4 100644 --- a/include/linux/srcu.h +++ b/include/linux/srcu.h @@ -372,7 +372,7 @@ static inline void srcu_read_unlock_lite(struct srcu_struct *ssp, int idx) WARN_ON_ONCE(idx & ~0x1); srcu_check_read_flavor(ssp, SRCU_READ_FLAVOR_LITE); srcu_lock_release(&ssp->dep_map); - __srcu_read_unlock(ssp, idx); + __srcu_read_unlock_lite(ssp, idx); } /**