diff options
author | Vadim Sukhomlinov <sukhomlinov@google.com> | 2020-06-17 22:13:52 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-06-18 19:09:13 +0000 |
commit | 08b1e22d83bd1d6c07619065fded9d0496f90e58 (patch) | |
tree | 5354ef29b77884c35ec11680b63427bb85b1e194 | |
parent | 32730b21cfd504438d6a711834b445c68ec19ae5 (diff) | |
download | chrome-ec-08b1e22d83bd1d6c07619065fded9d0496f90e58.tar.gz |
ec.tasklist: increase stack for console task for development mode
Due to changes in internal structures, stack consumption for SHA2,
HMAC, HMAC_DRBG grew up, and when combined with stack growth due to
changes in cprintf cause crash for some development console commands.
This patch increase console stack for CR50_DEV and CRYPTO_TEST modes.
BUG=none
TEST=manual, build with CR50_DEV or CRYPTO_TEST, run taskinfo
Signed-off-by: Vadim Sukhomlinov <sukhomlinov@google.com>
Change-Id: I7170c65e4b8092d165c478f505a435f834744ed9
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2251382
Reviewed-by: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Commit-Queue: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Commit-Queue: Vadim Bendebury <vbendeb@chromium.org>
Tested-by: Vadim Sukhomlinov <sukhomlinov@chromium.org>
Auto-Submit: Vadim Sukhomlinov <sukhomlinov@chromium.org>
-rw-r--r-- | board/cr50/ec.tasklist | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/board/cr50/ec.tasklist b/board/cr50/ec.tasklist index 43641c6c38..fd72cafaf9 100644 --- a/board/cr50/ec.tasklist +++ b/board/cr50/ec.tasklist @@ -6,7 +6,15 @@ /** * See CONFIG_TASK_LIST in config.h for details. */ + +#ifdef CRYPTO_TEST_SETUP +/* some crypto testing console commands require larger stack */ +#define CONSOLE_STACK_SIZE (CONFIG_STACK_SIZE + 512) +#else +#define CONSOLE_STACK_SIZE CONFIG_STACK_SIZE +#endif + #define CONFIG_TASK_LIST \ TASK_ALWAYS(HOOKS, hook_task, NULL, CONFIG_STACK_SIZE) \ TASK_NOTEST(TPM, tpm_task, NULL, 8192) \ - TASK_ALWAYS(CONSOLE, console_task, NULL, CONFIG_STACK_SIZE) + TASK_ALWAYS(CONSOLE, console_task, NULL, CONSOLE_STACK_SIZE) |