rwlock: remove unneeded subtraction
authorRoger Pau Monné <roger.pau@citrix.com>
Fri, 18 Feb 2022 08:01:27 +0000 (09:01 +0100)
committerJan Beulich <jbeulich@suse.com>
Fri, 18 Feb 2022 08:01:27 +0000 (09:01 +0100)
commit26f8eead2b8d19396bb57d00144bb19ae25f8f27
tree9b77ebbd6748c8cf15494adcfd465dc74d4e6bb6
parent3e548a6379bc0b1b79a20d12aeb4faed2833a4a7
rwlock: remove unneeded subtraction

There's no need to subtract _QR_BIAS from the lock value for storing
in the local cnts variable in the read lock slow path: the users of
the value in cnts only care about the writer-related bits and use a
mask to get the value.

Note that further setting of cnts in rspin_until_writer_unlock already
do not subtract _QR_BIAS.

Originally _QR_BIAS was subtracted from the result of
atomic_add_return_acquire in order to prevent GCC from emitting an
unneeded ADD instruction. This being in the lock slow path such
optimizations don't seem likely to make any relevant performance
difference. Also modern GCC and CLANG versions will already avoid
emitting the ADD instruction.

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Luca Fancellu <luca.fancellu@arm.com>
xen/common/rwlock.c