perf/ring_buffer: Use local_try_cmpxchg in __perf_output_begin
authorUros Bizjak <ubizjak@gmail.com>
Sat, 8 Jul 2023 09:00:37 +0000 (11:00 +0200)
committerPeter Zijlstra <peterz@infradead.org>
Mon, 10 Jul 2023 07:52:36 +0000 (09:52 +0200)
commit1af61adb3a23192023fec1733bd4c8500f53e546
treec30d7a1b15c81c6a191937315acb04f4231b9b2d
parentd6b45484c130f4095313ae3edeb4aae662c12fb1
perf/ring_buffer: Use local_try_cmpxchg in __perf_output_begin

Use local_try_cmpxchg instead of local_cmpxchg (*ptr, old, new) == old
in __perf_output_begin.  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, try_cmpxchg implicitly assigns old *ptr value to "old" when cmpxchg
fails. There is no need to re-read the value in the loop.

No functional change intended.

Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20230708090048.63046-2-ubizjak@gmail.com
kernel/events/ring_buffer.c