summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2018-05-23 01:13:27 +0300
committerStefan Reinauer <stefan.reinauer@coreboot.org>2019-01-11 00:53:51 +0000
commita706ad54446afc418c60776c06e699943ce4bdd5 (patch)
treec03e6564f53b5163e9baa8b6714000a6b785d906
parent2dbe51a17c3b7672a99965b160204ffceefccb6d (diff)
downloadcoreboot-a706ad54446afc418c60776c06e699943ce4bdd5.tar.xz
arch/x86/lapic: Remove second stack poisoning
It was already done once in c_start.S. Change-Id: I1cb0ea25251644dbd1127d177247a02ba52bb550 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/30796 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/cpu/x86/lapic/lapic_cpu_init.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c
index 0a51a08477..4498b97b1c 100644
--- a/src/cpu/x86/lapic/lapic_cpu_init.c
+++ b/src/cpu/x86/lapic/lapic_cpu_init.c
@@ -265,11 +265,9 @@ int start_cpu(struct device *cpu)
struct cpu_info *info;
unsigned long stack_end;
unsigned long stack_base;
- unsigned long *stack;
unsigned long apicid;
unsigned int index;
unsigned long count;
- int i;
int result;
spin_lock(&start_cpu_lock);
@@ -287,10 +285,8 @@ int start_cpu(struct device *cpu)
stack_base = ((unsigned long)_estack) - (CONFIG_STACK_SIZE*(index+1));
printk(BIOS_SPEW, "CPU%d: stack_base %p, stack_end %p\n", index,
(void *)stack_base, (void *)stack_end);
- /* poison the stack */
- for (stack = (void *)stack_base, i = 0; i < CONFIG_STACK_SIZE; i++)
- stack[i/sizeof(*stack)] = 0xDEADBEEF;
- stacks[index] = stack;
+ stacks[index] = (void *)stack_base;
+
/* Record the index and which CPU structure we are using */
info = (struct cpu_info *)stack_end;
info->index = index;