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/panic.h | |
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/panic.h')
-rw-r--r-- | include/panic.h | 7 |
1 files changed, 6 insertions, 1 deletions
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 |