diff options
| author | Randall Spangler <rspangler@chromium.org> | 2012-09-07 13:48:23 -0700 |
|---|---|---|
| committer | Gerrit <chrome-bot@google.com> | 2012-09-09 11:00:34 -0700 |
| commit | a3d62a3700206b9cd34e129f6a04967bed5e46e4 (patch) | |
| tree | 7961d109a7bf74d000f283a678ed757b0bf0989b /board/bds | |
| parent | 22d13781dcd006f9305956d33bf1ce1581454d2e (diff) | |
| download | chrome-ec-a3d62a3700206b9cd34e129f6a04967bed5e46e4.tar.gz | |
Switch to variable-size stacks
Increase stack size slightly for vboot hash task since the vboot
SHA256 function allocates ~300 bytes of stack data. Reduce stack size
for watchdog, power LED, and a few other tasks with simple call trees
where we can be sure an error path isn't going to blow past the
reduced stack.
This frees up ~1KB of RAM on STM32.
BUG=chrome-os-partner:13814
BRANCH=all
TEST=boot system; shmem should show more unused RAM; taskinfo should show
tasks still have unused stack
Change-Id: I47d6b77564a0180d15d86667cc0566a8919b776e
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/32608
Reviewed-by: Vincent Palatin <vpalatin@chromium.org>
Diffstat (limited to 'board/bds')
| -rw-r--r-- | board/bds/ec.tasklist | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/board/bds/ec.tasklist b/board/bds/ec.tasklist index beaaec5552..2e5408dd5b 100644 --- a/board/bds/ec.tasklist +++ b/board/bds/ec.tasklist @@ -8,12 +8,13 @@ * * The first one has the lowest priority. * - * For each task, use the macro TASK(n, r, d) where : + * For each task, use the macro TASK(n, r, d, s) where : * 'n' in the name of the task * 'r' in the main routine of the task * 'd' in an opaque parameter passed to the routine at startup + * 's' is the stack size in bytes; must be a multiple of 8 */ #define CONFIG_TASK_LIST \ - TASK(WATCHDOG, watchdog_task, NULL) \ - TASK(LIGHTBAR, lightbar_task, NULL) \ - TASK(CONSOLE, console_task, NULL) + TASK(WATCHDOG, watchdog_task, NULL, TASK_STACK_SIZE) \ + TASK(LIGHTBAR, lightbar_task, NULL, TASK_STACK_SIZE) \ + TASK(CONSOLE, console_task, NULL, TASK_STACK_SIZE) |