summaryrefslogtreecommitdiff
path: root/include/panic.h
diff options
context:
space:
mode:
authorRandall Spangler <rspangler@chromium.org>2013-11-07 10:53:12 -0800
committerchrome-internal-fetch <chrome-internal-fetch@google.com>2013-11-07 22:35:56 +0000
commite2f851aae244c0c233064192c79f46fe0744e58f (patch)
treebb06eb8551f7affc6c2ee3352c11cdc7679b31ff /include/panic.h
parenta5d2fa9fb4fde97bced002c1ab886612a5b8f84a (diff)
downloadchrome-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.h7
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