hppa: Align __clone stack argument to 8 bytes (Bug 25066)

The hppa architecture requires strict alignment for loads and stores.
As a result, the minimum stack alignment that will work is 8 bytes.
This patch adjusts __clone() to align the stack argument passed to it.
It also adjusts slightly some formatting.

This fixes the nptl/tst-tls1 test.

diff --git a/sysdeps/unix/sysv/linux/hppa/clone.S b/sysdeps/unix/sysv/linux/hppa/clone.S
index 6db2cb5dd5979da17cf353ebf6bb14dde6cc2790..372d29a83858b988c2869c7eba8752a361ce9c82 100644 (file)
--- a/sysdeps/unix/sysv/linux/hppa/clone.S
+++ b/sysdeps/unix/sysv/linux/hppa/clone.S
@@ -73,13 +73,18 @@ ENTRY(__clone)
 #endif
 
        /* Sanity check arguments.  */
-       comib,=,n  0, %arg0, .LerrorSanity        /* no NULL function pointers */
-       comib,=,n  0, %arg1, .LerrorSanity        /* no NULL stack pointers */
+       comib,=,n       0,%arg0,.LerrorSanity   /* no NULL function pointers */
+       comib,=,n       0,%arg1,.LerrorSanity   /* no NULL stack pointers */
+
+       /* Ensure stack argument is 8-byte aligned.  */
+       ldo             7(%r25),%r25
+       depi            0,31,3,%r25
 
        /* Save the function pointer, arg, and flags on the new stack.  */
        stwm    %r26, 64(%r25)
        stw     %r23, -60(%r25)
        stw     %r24, -56(%r25)
+
        /* Clone arguments are (int flags, void * child_stack) */
        copy    %r24, %r26              /* flags are first */
        /* User stack pointer is in the correct register already */