pgsql: Fix s_lock.h PPC assembly code to be compatible with native AIX - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix s_lock.h PPC assembly code to be compatible with native AIX
Date
Msg-id E1ZVmRk-0004i3-32@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix s_lock.h PPC assembly code to be compatible with native AIX assembler.

On recent AIX it's necessary to configure gcc to use the native assembler
(because the GNU assembler hasn't been updated to handle AIX 6+).  This
caused PG builds to fail with assembler syntax errors, because we'd try
to compile s_lock.h's gcc asm fragment for PPC, and that assembly code
relied on GNU-style local labels.  We can't substitute normal labels
because it would fail in any file containing more than one inlined use of
tas().  Fortunately, that code is stable enough, and the PPC ISA is simple
enough, that it doesn't seem like too much of a maintenance burden to just
hand-code the branch offsets, removing the need for any labels.

Note that the AIX assembler only accepts "$" for the location counter
pseudo-symbol.  The usual GNU convention is "."; but it appears that all
versions of gas for PPC also accept "$", so in theory this patch will not
break any other PPC platforms.

This has been reported by a few people, but Steve Underwood gets the credit
for being the first to pursue the problem far enough to understand why it
was failing.  Thanks also to Noah Misch for additional testing.

Branch
------
REL9_4_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/3da9c060fc775f17e43ae8608b818079a0099516

Modified Files
--------------
src/include/storage/s_lock.h |   16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)


pgsql-committers by date:

Previous
From: Stephen Frost
Date:
Subject: pgsql: Ensure locks are acquired on RLS-added relations
Next
From: Tom Lane
Date:
Subject: pgsql: Fix s_lock.h PPC assembly code to be compatible with native AIX