Fix the CAS spinlock implementation
authorSoby Mathew <soby.mathew@arm.com>
Wed, 25 Sep 2019 13:03:41 +0000 (14:03 +0100)
committerOlivier Deprez <olivier.deprez@arm.com>
Fri, 4 Oct 2019 08:19:35 +0000 (10:19 +0200)
commitc97cba4ea44910df1f7b1af5dba79013fb44c383
tree9d2965a2b9610413dea548a95e2f5094fc84b6e8
parentace23683beb81354d6edbc61c087ab8c384d0631
Fix the CAS spinlock implementation

Make the spinlock implementation use ARMv8.1-LSE CAS instruction based
on a platform build option. The CAS-based implementation used to be
unconditionally selected for all ARM8.1+ platforms.

The previous CAS spinlock implementation had a bug wherein the spin_unlock()
implementation had an `sev` after `stlr` which is not sufficient. A dsb is
needed to ensure that the stlr completes prior to the sev. Having a dsb is
heavyweight and a better solution would be to use load exclusive semantics
to monitor the lock and wake up from wfe when a store happens to the lock.
The patch implements the same.

Change-Id: I5283ce4a889376e4cc01d1b9d09afa8229a2e522
Signed-off-by: Soby Mathew <soby.mathew@arm.com>
Signed-off-by: Olivier Deprez <olivier.deprez@arm.com>
Makefile
docs/design/firmware-design.rst
docs/getting_started/user-guide.rst
lib/locks/exclusive/aarch64/spinlock.S
make_helpers/defaults.mk