<p>Marshall Dawson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21768">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/x86: Align stack in SIPI handler<br><br>Ensure the stack is properly aligned in the SIPI handler.<br><br>BUG=b:66003093<br>TEST=boot kahlee built with gcc 6.3<br><br>Change-Id: Ibdd8242494c6a2bc0c6ead7ac98be55149219d7c<br>Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com><br>---<br>M src/cpu/x86/sipi_vector.S<br>1 file changed, 4 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/21768/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/cpu/x86/sipi_vector.S b/src/cpu/x86/sipi_vector.S<br>index b40ea76..bd60c65 100644<br>--- a/src/cpu/x86/sipi_vector.S<br>+++ b/src/cpu/x86/sipi_vector.S<br>@@ -114,6 +114,8 @@<br>         movl    stack_top, %edx<br>       subl    %eax, %edx<br>    mov     %edx, %esp<br>+   andl    $0xfffffff0, %esp /* ensure stack alignment */<br>+<br>     /* Save CPU number. */<br>        mov     %ecx, %esi<br> <br>@@ -192,7 +194,8 @@<br>    mov     %eax, %cr4<br> #endif<br> <br>-       /* c_handler(cpu_num) */<br>+     /* c_handler(cpu_num), preserve proper stack alignment */<br>+    sub     $12, %esp<br>     push    %esi    /* cpu_num */<br>         mov     c_handler, %eax<br>       call    *%eax<br></pre><p>To view, visit <a href="https://review.coreboot.org/21768">change 21768</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/21768"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ibdd8242494c6a2bc0c6ead7ac98be55149219d7c </div>
<div style="display:none"> Gerrit-Change-Number: 21768 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marshall Dawson <marshalldawson3rd@gmail.com> </div>