summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2019-12-03 23:04:41 -0800
committerPatrick Georgi <pgeorgi@google.com>2019-12-05 17:58:05 +0000
commit2e0bca011af0518f7335a700ea04082611b3ddad (patch)
tree96511b75c71b6e8d7f59a2f1d72ed1893b6271cf /src
parentbb345abbfc999f70e3f0f9739f13e1f45d5a0fe9 (diff)
downloadcoreboot-2e0bca011af0518f7335a700ea04082611b3ddad.tar.xz
arm64: Bump exception stack size to 2KB
To avoid trampling over interesting exception artifacts on the real stack, our arm64 systems switch to a separate exception stack when entering an exception handler. We don't want that to use up too much SRAM so we just set it to 512 bytes. I mean it just prints a bunch of registers, how much stack could it need, right? Quite a bit it turns out. The whole vtxprintf() call stack goes pretty deep, and aarch64 generally seems to be very generous with stack space. Just the varargs handling seems to require 128 bytes for some reason, and the other stuff adds up too. In the end the current implementation takes 1008 bytes, so bump the exception stack size to 2K to make sure it fits. Change-Id: I910be4c5f6b29fae35eb53929c733a1bd4585377 Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/37464 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org>
Diffstat (limited to 'src')
-rw-r--r--src/arch/arm64/armv8/exception.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/arch/arm64/armv8/exception.c b/src/arch/arm64/armv8/exception.c
index 579e1040b4..13f456da26 100644
--- a/src/arch/arm64/armv8/exception.c
+++ b/src/arch/arm64/armv8/exception.c
@@ -36,7 +36,7 @@
#include <console/console.h>
#include <arch/lib_helpers.h>
-uint8_t exception_stack[0x200] __attribute__((aligned(16)));
+uint8_t exception_stack[2*KiB] __attribute__((aligned(16)));
static const char *exception_names[NUM_EXC_VIDS] = {
[EXC_VID_CUR_SP_EL0_SYNC] = "_sync_sp_el0",