MINOR: plock: use an ARMv8 instruction barrier for the pause instruction
authorYour Name <you@example.com>
Sat, 28 Nov 2020 15:37:14 +0000 (15:37 +0000)
committerWilly Tarreau <w@1wt.eu>
Sun, 29 Nov 2020 13:53:33 +0000 (14:53 +0100)
commit1e237d037b3a45ec92d1dfa80dfd2c6bd7fc3af9
tree2aa9926379b0fafbd31ece2e561d76693774504f
parenta9ffc416377e0df9859526dc3c1d769c6a68636f
MINOR: plock: use an ARMv8 instruction barrier for the pause instruction

As suggested by @AGSaidi in issue #958, on ARMv8 its convenient to use
an "isb" instruction in pl_cpu_relax() to improve fairness. Without it
I've met a few watchdog conditions on valid locks with 16 threads,
indicating that some threads couldn't manage to get it in 2 seconds. I
never happened again with it. In addition, the performance increased
by slightly more than 5% thanks to the reduced contention.

This should be backported as far as 2.2, possibly even 2.0.
include/import/atomic-ops.h