diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-03-15 08:07:22 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-03-19 21:38:53 +0100 |
| commit | 4796c32ad60933bd8c23729684414121097b0445 (patch) | |
| tree | d4a717d13010eed377aa37be1ee4e7da399922e5 | |
| parent | 85e81dfa6dffde15670a8bd142ccac5ca27aa36d (diff) | |
| download | coreboot-4796c32ad60933bd8c23729684414121097b0445.tar.xz | |
ramstage: Align stack to 16 bytes
Some SSE instructions could take 128bit memory operands from
stack.
AGESA vendorcode was always built with SSE enabled, but until
now stack alignment was not known to cause major issues. Seems
like GCC-6.3 more likely emits instructions that depend on the
16 byte alignment of stack.
Change-Id: Iea3de54f20ff242105bce5a5edbbd76b04c0116c
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/18823
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
| -rw-r--r-- | src/arch/x86/c_start.S | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/arch/x86/c_start.S b/src/arch/x86/c_start.S index 29d3a538b4..9ad2698483 100644 --- a/src/arch/x86/c_start.S +++ b/src/arch/x86/c_start.S @@ -110,6 +110,8 @@ _start: */ post_code(POST_PRE_HARDWAREMAIN) /* post fe */ + andl $0xFFFFFFF0, %esp + #if CONFIG_GDB_WAIT call gdb_hw_init call gdb_stub_breakpoint |