[IA64] fix stack alignment for ia32 signal handlers
authorMarkus F.X.J. Oberhumer <markus@oberhumer.com>
Mon, 23 Apr 2007 10:00:05 +0000 (12:00 +0200)
committerTony Luck <tony.luck@intel.com>
Tue, 8 May 2007 18:22:59 +0000 (11:22 -0700)
This fixes the setup of the alignment of the signal frame, so that all
signal handlers are run with a properly aligned stack frame.

The current code "over-aligns" the stack pointer so that the stack frame
is effectively always mis-aligned by 4 bytes.  But what we really want
is that on function entry ((sp + 4) & 15) == 0, which matches what would
happen if the stack were aligned before a "call" instruction.

i386 and x86_64 are already fixed by d347f372273c2b3d86a66e2e1c94c790c208e166

Signed-off-by: Markus F.X.J. Oberhumer <markus@oberhumer.com>
Signed-off-by: Tony Luck <tony.luck@intel.com>
arch/ia64/ia32/ia32_signal.c

index b3355a9ca2c3fb3b3b654dae5f7817936758b47c..7b38b73e7827f970aaf037eb5ac20d3530317629 100644 (file)
@@ -811,7 +811,11 @@ get_sigframe (struct k_sigaction *ka, struct pt_regs * regs, size_t frame_size)
        }
        /* Legacy stack switching not supported */
 
-       return (void __user *)((esp - frame_size) & -8ul);
+       esp -= frame_size;
+       /* Align the stack pointer according to the i386 ABI,
+        * i.e. so that on function entry ((sp + 4) & 15) == 0. */
+       esp = ((esp + 4) & -16ul) - 4;
+       return (void __user *) esp;
 }
 
 static int