]> granicus.if.org Git - postgresql/commit
Fix spinlock assembly code for MIPS so it works on MIPS r6.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 23 Jun 2019 00:31:50 +0000 (20:31 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 23 Jun 2019 00:31:50 +0000 (20:31 -0400)
commit1323bfce55109dd54ee164828aab7983d3020a25
tree5d37e479699940d24f53f61c5facace391d1d2e7
parent660a2b19038b2f6b9f6bcb2c3297a47d5e3557a8
Fix spinlock assembly code for MIPS so it works on MIPS r6.

Original MIPS-I processors didn't have the LL/SC instructions (nor any
other userland synchronization primitive).  If the build toolchain
targets that ISA variant by default, as an astonishingly large fraction
of MIPS platforms still do, the assembler won't take LL/SC without
coercion in the form of a ".set mips2" instruction.  But we issued that
unconditionally, making it an ISA downgrade for chips later than MIPS2.
That breaks things for the latest MIPS r6 ISA, which encodes these
instructions differently.  Adjust the code so we don't change ISA level
if it's >= 2.

Note that this patch doesn't change what happens on an actual MIPS-I
processor: either the kernel will emulate these instructions
transparently, or you'll get a SIGILL failure.  That tradeoff seemed
fine in 2002 when this code was added (cf 3cbe6b247), and it's even
more so today when MIPS-I is basically extinct.  But let's add a
comment about that.

YunQiang Su (with cosmetic adjustments by me).  Back-patch to all
supported branches.

Discussion: https://postgr.es/m/15844-8f62fe7e163939b3@postgresql.org
src/include/storage/s_lock.h