Still further tweaking of s_lock assembler: do not assume that leading

whitespace is unimportant in assembly code.  Also, move VAX definition
of typedef slock_t to port header files to be like all the other ports.
Note that netbsd.h and openbsd.h are now identical, and I rather think
that freebsd.h is broken in the places where it doesn't agree --- but
I'll leave it to the freebsders to look at that.

src/include/port/freebsd.h

@@ -14,6 +14,7 @@ typedef long int slock_t;
 #endif
 
 #if defined(__vax__)
+typedef unsigned char slock_t;
 #define NEED_VAX_TAS_ASM
 #define HAS_TEST_AND_SET
 #endif

src/include/port/openbsd.h

@@ -1,43 +1,47 @@
 #if defined(__i386__)
 #define NEED_I386_TAS_ASM
 #define HAS_TEST_AND_SET
+typedef unsigned char slock_t;
 #endif
 
 #if defined(__sparc__)
 #define NEED_SPARC_TAS_ASM
 #define HAS_TEST_AND_SET
+typedef unsigned char slock_t;
 #endif
 
 #if defined(__vax__)
 #define NEED_VAX_TAS_ASM
 #define HAS_TEST_AND_SET
+typedef unsigned char slock_t;
 #endif
 
 #if defined(__ns32k__)
 #define NEED_NS32K_TAS_ASM
 #define HAS_TEST_AND_SET
+typedef unsigned char slock_t;
 #endif
 
 #if defined(__m68k__)
 #define HAS_TEST_AND_SET
+typedef unsigned char slock_t;
 #endif
 
 #if defined(__arm__)
 #define HAS_TEST_AND_SET
+typedef unsigned char slock_t;
 #endif
 
 #if defined(__mips__)
 /* #	undef HAS_TEST_AND_SET */
 #endif
 
+#if defined(__alpha__)
+#define HAS_TEST_AND_SET
+typedef unsigned long slock_t;
+#endif
+
 #if defined(__powerpc__)
 #define HAS_TEST_AND_SET
-#endif
-
-#if defined(__powerpc__)
 typedef unsigned int slock_t;
-
-#else
-typedef unsigned char slock_t;
-
 #endif

src/include/storage/s_lock.h

@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/include/storage/s_lock.h,v 1.84 2001/01/19 21:09:57 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/include/storage/s_lock.h,v 1.85 2001/01/20 00:03:54 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -100,15 +100,15 @@ extern void s_lock_sleep(unsigned spins, int microsec,
  */
 
 /*
- * Standard _asm format:
+ * Standard gcc asm format:
  *
- *	__asm__ __volatile__(
+	__asm__ __volatile__(
-			"command	\n"
+		"	command	\n"
-			"command	\n"
+		"	command	\n"
-			"command	\n"
+		"	command	\n"
-:			"=r"(_res)			return value, in register
+:		"=r"(_res)			return value, in register
-:			"r"(lock)			argument, 'lock pointer', in register
+:		"r"(lock)			argument, 'lock pointer', in register
-:			"r0");				inline code uses this register
+:		"r0");				inline code uses this register
  */
 
 
@@ -121,10 +121,10 @@ tas(volatile slock_t *lock)
 	register slock_t _res = 1;
 
 	__asm__ __volatile__(
-			"lock			\n"
+		"	lock			\n"
-			"xchgb	%0,%1	\n"
+		"	xchgb	%0,%1	\n"
-:			"=q"(_res), "=m"(*lock)
+:		"=q"(_res), "=m"(*lock)
-:			"0"(_res));
+:		"0"(_res));
 	return (int) _res;
 }
 
@@ -140,10 +140,10 @@ tas(volatile slock_t *lock)
 	long int	ret;
 
 	__asm__ __volatile__(
-			"xchg4 	%0=%1,%2	\n"
+		"	xchg4 	%0=%1,%2	\n"
-:			"=r"(ret), "=m"(*lock)
+:		"=r"(ret), "=m"(*lock)
-:			"r"(1), "1"(*lock)
+:		"r"(1), "1"(*lock)
-:			"memory");
+:		"memory");
 
 	return (int) ret;
 }
@@ -160,9 +160,9 @@ tas(volatile slock_t *lock)
 	register slock_t _res = 1;
 
 	__asm__ __volatile__(
-			"swpb 	%0, %0, [%3]	\n"
+		"	swpb 	%0, %0, [%3]	\n"
-:			"=r"(_res), "=m"(*lock)
+:		"=r"(_res), "=m"(*lock)
-:			"0"(_res), "r"(lock));
+:		"0"(_res), "r"(lock));
 	return (int) _res;
 }
 
@@ -180,14 +180,14 @@ tas(volatile slock_t *lock)
 	int			_res;
 
 	__asm__	__volatile__(
-			"la		1,1		\n"
+		"	la	1,1			\n"
-			"l 		2,%2	\n"
+		"	l 	2,%2		\n"
-			"slr 	0,0		\n"
+		"	slr 0,0			\n"
-			"cs 	0,1,0(2)\n"
+		"	cs 	0,1,0(2)	\n"
-			"lr 	%1,0	\n"
+		"	lr 	%1,0		\n"
-:			"=m"(lock), "=d"(_res)
+:		"=m"(lock), "=d"(_res)
-:			"m"(lock)
+:		"m"(lock)
-:			"0", "1", "2");
+:		"0", "1", "2");
 
 	return (_res);
 }
@@ -204,9 +204,9 @@ tas(volatile slock_t *lock)
 	register slock_t _res = 1;
 
 	__asm__ __volatile__(
-			"ldstub	[%2], %0	\n"
+		"	ldstub	[%2], %0	\n"
-:			"=r"(_res), "=m"(*lock)
+:		"=r"(_res), "=m"(*lock)
-:			"r"(lock));
+:		"r"(lock));
 	return (int) _res;
 }
 
@@ -222,11 +222,11 @@ tas(volatile slock_t *lock)
 	register int rv;
 
 	__asm__	__volatile__(
-			"tas	%1		\n"
+		"	tas	%1		\n"
-			"sne	%0		\n"
+		"	sne	%0		\n"
-:			"=d"(rv), "=m"(*lock)
+:		"=d"(rv), "=m"(*lock)
-:			"1"(*lock)
+:		"1"(*lock)
-:			"cc");
+:		"cc");
 
 	return rv;
 }
@@ -241,21 +241,19 @@ tas(volatile slock_t *lock)
  */
 #define TAS(lock) tas(lock)
 
-typedef unsigned char slock_t;
-
 static __inline__ int
 tas(volatile slock_t *lock)
 {
 	register	_res;
 
 	__asm__ __volatile__(
-			"movl 	$1, r0			\n"
+		"	movl 	$1, r0			\n"
-			"bbssi	$0, (%1), 1f	\n"
+		"	bbssi	$0, (%1), 1f	\n"
-			"clrl	r0				\n"
+		"	clrl	r0				\n"
-			"1: movl r0, %0			\n"
+		"1:	movl	r0, %0			\n"
-:			"=r"(_res)
+:		"=r"(_res)
-:			"r"(lock)
+:		"r"(lock)
-:			"r0");
+:		"r0");
 	return (int) _res;
 }
 
@@ -271,9 +269,9 @@ tas(volatile slock_t *lock)
 	register	_res;
 
 	__asm__ __volatile__(
-			"sbitb	0, %0	\n"
+		"	sbitb	0, %0	\n"
-			"sfsd	%1		\n"
+		"	sfsd	%1		\n"
-:			"=m"(*lock), "=r"(_res));
+:		"=m"(*lock), "=r"(_res));
 	return (int) _res;
 }
 
@@ -329,7 +327,7 @@ tas(volatile slock_t *s_lock)
 #define S_UNLOCK(lock)	\
 do \
 {\
-	__asm__ __volatile__ ("mb"); \
+	__asm__ __volatile__ ("	mb \n"); \
 	*(lock) = 0; \
 } while (0)
 
@@ -339,20 +337,20 @@ tas(volatile slock_t *lock)
 	register slock_t _res;
 
 	__asm__	__volatile__(
-			"ldq	$0, %0	\n"
+		"	ldq		$0, %0	\n"
-			"bne	$0, 2f	\n"
+		"	bne		$0, 2f	\n"
-			"ldq_l	%1, %0	\n"
+		"	ldq_l	%1, %0	\n"
-			"bne	%1, 2f	\n"
+		"	bne		%1, 2f	\n"
-			"mov	1,  $0	\n"
+		"	mov		1,  $0	\n"
-			"stq_c	$0, %0	\n"
+		"	stq_c	$0, %0	\n"
-			"beq	$0, 2f	\n"
+		"	beq		$0, 2f	\n"
-			"mb				\n"
+		"	mb				\n"
-			"br		3f		\n"
+		"	br		3f		\n"
-			"2: mov 1, %1	\n"
+		"2:	mov		1, %1	\n"
-			"3:				\n"
+		"3:					\n"
-:			"=m"(*lock), "=r"(_res)
+:		"=m"(*lock), "=r"(_res)
 :
-:			"0");
+:		"0");
 
 	return (int) _res;
 }