ucount: use atomic_long_try_cmpxchg() in atomic_long_inc_below()
authorUros Bizjak <ubizjak@gmail.com>
Mon, 21 Jul 2025 17:45:58 +0000 (19:45 +0200)
committerAndrew Morton <akpm@linux-foundation.org>
Sat, 2 Aug 2025 19:01:38 +0000 (12:01 -0700)
commit58b4fba81a2e400a47ddbe7c1dc0a2bc038313b7
tree6f8e874cbcb17856c60ab3a60ae8b9151a279248
parentf8cd9193b62e92ad25def5370ca8ea2bc7585381
ucount: use atomic_long_try_cmpxchg() in atomic_long_inc_below()

Use atomic_long_try_cmpxchg() instead of
atomic_long_cmpxchg (*ptr, old, new) == old in atomic_long_inc_below().
x86 CMPXCHG instruction returns success in ZF flag, so this change saves
a compare after cmpxchg (and related move instruction in front of cmpxchg).

Also, atomic_long_try_cmpxchg implicitly assigns old *ptr value to "old"
when cmpxchg fails, enabling further code simplifications.

No functional change intended.

Link: https://lkml.kernel.org/r/20250721174610.28361-2-ubizjak@gmail.com
Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Reviewed-by: Alexey Gladkov <legion@kernel.org>
Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: "Paul E. McKenney" <paulmck@kernel.org>
Cc: Alexey Gladkov <legion@kernel.org>
Cc: Roman Gushchin <roman.gushchin@linux.dev>
Cc: MengEn Sun <mengensun@tencent.com>
Cc: "Thomas Weißschuh" <linux@weissschuh.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
kernel/ucount.c