]> granicus.if.org Git - postgresql/commit
Remove swpb-based spinlock implementation for ARMv5 and earlier.
authorRobert Haas <rhaas@postgresql.org>
Sun, 6 Jul 2014 18:52:25 +0000 (14:52 -0400)
committerRobert Haas <rhaas@postgresql.org>
Sun, 6 Jul 2014 18:56:36 +0000 (14:56 -0400)
commit4893ccd02459fdb444f4f4b34c441eb899592879
tree25835919fb65a8cf99b9e298201d4097911f02a6
parent1b86c81d2d255d3fb665ddc77c2bc3dfd751a1df
Remove swpb-based spinlock implementation for ARMv5 and earlier.

Per recent analysis by Andres Freund, this implementation is in fact
unsafe, because ARMv5 has weak memory ordering, which means tha the
CPU could move loads or stores across the volatile store performed by
the default S_UNLOCK.  We could try to fix this, but have no ARMv5
hardware to test on, so removing support seems better.  We can still
support ARMv5 systems on GCC versions new enough to have built-in
atomics support for this platform, and can also re-add support for
the old way if someone has hardware that can be used to test a fix.
However, since the requirement to use a relatively-new GCC hasn't
been an issue for ARMv6 or ARMv7, which lack the swpb instruction
altogether, perhaps it won't be an issue for ARMv5 either.
src/include/storage/s_lock.h