diff options
| author | Raul E Rangel <rrangel@chromium.org> | 2018-06-25 14:22:27 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-06-27 22:12:28 +0000 |
| commit | e18e5ab5fa5002f5835a2b83be0fcd6c5739cd8a (patch) | |
| tree | 7b0064fc8abc7af967cf3a1db19e20d77629efdf | |
| parent | 871156898c874c5247d4024309af348f8bc1e518 (diff) | |
| download | coreboot-e18e5ab5fa5002f5835a2b83be0fcd6c5739cd8a.tar.xz | |
smm: Switch from %edx to %ebx
%edx gets clobbered before the c handler is invoked. This is just a
cleanup cl to make the next cl look clean.
BUG=b:80539294
TEST=verified SMI still works on grunt.
Change-Id: I21bf41ed4fdeaaa8737c883f202a39cb57c2b517
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Reviewed-on: https://review.coreboot.org/27228
Reviewed-by: Martin Roth <martinroth@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/cpu/x86/smm/smm_stub.S | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/cpu/x86/smm/smm_stub.S b/src/cpu/x86/smm/smm_stub.S index 0c05032577..ba66db9179 100644 --- a/src/cpu/x86/smm/smm_stub.S +++ b/src/cpu/x86/smm/smm_stub.S @@ -132,9 +132,9 @@ smm_trampoline32: 1: movl stack_size, %eax mul %ecx /* %eax(stack_size) * %ecx(cpu) = %eax(offset) */ - movl stack_top, %edx - subl %eax, %edx /* global_stack_top - offset = stack_top */ - mov %edx, %esp + movl stack_top, %ebx + subl %eax, %ebx /* global_stack_top - offset = stack_top */ + mov %ebx, %esp /* Create stack frame by pushing a NULL stack base pointer */ pushl $0x0 |