diff options
author | Randall Spangler <rspangler@chromium.org> | 2013-11-07 10:53:12 -0800 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-11-07 22:35:56 +0000 |
commit | e2f851aae244c0c233064192c79f46fe0744e58f (patch) | |
tree | bb06eb8551f7affc6c2ee3352c11cdc7679b31ff /include | |
parent | a5d2fa9fb4fde97bced002c1ab886612a5b8f84a (diff) | |
download | chrome-ec-e2f851aae244c0c233064192c79f46fe0744e58f.tar.gz |
Enable stack overflow checking on all context switches
Changes somewhere in the recent past have caused I2C operations to
consume more stack space. The current failure mode is that after some
debug command or infrequent battery operation, the system fails.
Clean up and enable stack overflow detection by default, and add a
debug command (disabled by default) to verify overflow detection
works.
This adds several instructions to each context switch, but it's still
fairly inexpensive, and represents only a few percent increase in the
size of svc_handler(). That's better than silent failures.
BUG=chrome-os-partner:23938
BRANCH=none
TEST=Enable CONFIG_CMD_STACKOVERFLOW, then run the 'stackoverflow' command.
This should cause a stack overflow to be detected in the CONSOLE task.
Change-Id: I9303aee5bd9318f1d92838b399d15fb8f6a2bbf9
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/176113
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/config.h | 7 | ||||
-rw-r--r-- | include/panic.h | 7 |
2 files changed, 10 insertions, 4 deletions
diff --git a/include/config.h b/include/config.h index f9e2a806a3..c6e151158e 100644 --- a/include/config.h +++ b/include/config.h @@ -215,6 +215,7 @@ #undef CONFIG_CMD_RTC_ALARM #undef CONFIG_CMD_SCRATCHPAD #undef CONFIG_CMD_SLEEP +#undef CONFIG_CMD_STACKOVERFLOW /*****************************************************************************/ @@ -296,6 +297,9 @@ */ #define CONFIG_DEBUG_EXCEPTIONS +/* Check for stack overflows on every context switch */ +#define CONFIG_DEBUG_STACK_OVERFLOW + /*****************************************************************************/ /* Support DMA transfers inside the EC */ @@ -502,9 +506,6 @@ /* Support one-wire interface */ #undef CONFIG_ONEWIRE -/* Check for stack overflows on every context switch */ -#undef CONFIG_OVERFLOW_DETECT - /* Support PECI interface to x86 processor */ #undef CONFIG_PECI diff --git a/include/panic.h b/include/panic.h index 256ce84e87..2647592448 100644 --- a/include/panic.h +++ b/include/panic.h @@ -82,13 +82,18 @@ void panic_assert_fail(const char *msg, const char *func, const char *fname, int linenum); /** - * Display a panic message and reset + * Display a custom panic message and reset * * @param msg Panic message */ void panic(const char *msg); /** + * Display a default message and reset + */ +void panic_reboot(void); + +/** * Enable/disable bus fault handler * * @param ignored Non-zero if ignoring bus fault |