Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitb5a22d8

Browse files
committed
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+). Thiscaused PG builds to fail with assembler syntax errors, because we'd tryto compile s_lock.h's gcc asm fragment for PPC, and that assembly coderelied on GNU-style local labels. We can't substitute normal labelsbecause it would fail in any file containing more than one inlined use oftas(). Fortunately, that code is stable enough, and the PPC ISA is simpleenough, that it doesn't seem like too much of a maintenance burden to justhand-code the branch offsets, removing the need for any labels.Note that the AIX assembler only accepts "$" for the location counterpseudo-symbol. The usual GNU convention is "."; but it appears that allversions of gas for PPC also accept "$", so in theory this patch will notbreak any other PPC platforms.This has been reported by a few people, but Steve Underwood gets the creditfor being the first to pursue the problem far enough to understand why itwas failing. Thanks also to Noah Misch for additional testing.
1 parentbbe6d23 commitb5a22d8

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

‎src/include/storage/s_lock.h

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -410,6 +410,12 @@ typedef unsigned int slock_t;
410410
/*
411411
* NOTE: per the Enhanced PowerPC Architecture manual, v1.0 dated 7-May-2002,
412412
* an isync is a sufficient synchronization barrier after a lwarx/stwcx loop.
413+
*
414+
* Ordinarily, we'd code the branches here using GNU-style local symbols, that
415+
* is "1f" referencing "1:" and so on. But some people run gcc on AIX with
416+
* IBM's assembler as backend, and IBM's assembler doesn't do local symbols.
417+
* So hand-code the branch offsets; fortunately, all PPC instructions are
418+
* exactly 4 bytes each, so it's not too hard to count.
413419
*/
414420
static __inline__int
415421
tas(volatileslock_t*lock)
@@ -420,16 +426,14 @@ tas(volatile slock_t *lock)
420426
__asm__ __volatile__(
421427
"lwarx %0,0,%3\n"
422428
"cmpwi %0,0\n"
423-
"bne1f\n"
429+
"bne$+16\n"/* branch to li %1,1 */
424430
"addi %0,%0,1\n"
425431
"stwcx. %0,0,%3\n"
426-
"beq 2f \n"
427-
"1:li %1,1\n"
428-
"b3f\n"
429-
"2:\n"
432+
"beq $+12\n"/* branch to isync */
433+
"li %1,1\n"
434+
"b $+12\n"/* branch to end of asm sequence */
430435
"isync\n"
431436
"li %1,0\n"
432-
"3:\n"
433437

434438
:"=&r"(_t),"=r"(_res),"+m"(*lock)
435439
:"r"(lock)

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp