From decc9452e981a7c8ce5c6b8983061a054057c615 Mon Sep 17 00:00:00 2001 From: Hyungwoo Yang Date: Wed, 30 Jan 2019 12:30:47 -0800 Subject: ish: save/restore FPU context only for the task uses FPU Currently we save/retore FPU H/W context for every task on every contxt switch. This hurts overall performance of ISH. This patch allows save and restore FPU H/W context only for a task that declares it uses FPU. BRANCH=none BUG=none TEST=verified in Atlas platform Change-Id: Ic2f0bbf59f655661e2dd788c688edc4e83068c1c Signed-off-by: Hyungwoo Yang Reviewed-on: https://chromium-review.googlesource.com/1448818 Commit-Ready: ChromeOS CL Exonerator Bot Tested-by: Hyungwoo Yang Reviewed-by: Jett Rink --- board/arcada_ish/ec.tasklist | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) (limited to 'board/arcada_ish/ec.tasklist') diff --git a/board/arcada_ish/ec.tasklist b/board/arcada_ish/ec.tasklist index f384af4d8b..b6e2cf28d5 100644 --- a/board/arcada_ish/ec.tasklist +++ b/board/arcada_ish/ec.tasklist @@ -8,15 +8,17 @@ * * The first one has the lowest priority. * - * For each task, use the macro TASK_ALWAYS(n, r, d, s) for base tasks and - * TASK_NOTEST(n, r, d, s) for tasks that can be excluded in test binaries, + * For each task, use the macro TASK_ALWAYS(n, r, d, s, f) for base tasks and + * TASK_NOTEST(n, r, d, s, f) for tasks that can be excluded in test binaries, * 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 + * 'f' is the bit flags for the platform specific information + * - MIA_TASK_FLAG_USE_FPU : bit 0, task uses FPU H/W */ #define CONFIG_TASK_LIST \ - TASK_ALWAYS(HOOKS, hook_task, NULL, HUGE_TASK_STACK_SIZE) \ - TASK_ALWAYS(CONSOLE, console_task, NULL, LARGER_TASK_STACK_SIZE) + TASK_ALWAYS(HOOKS, hook_task, NULL, HUGE_TASK_STACK_SIZE, 0) \ + TASK_ALWAYS(CONSOLE, console_task, NULL, LARGER_TASK_STACK_SIZE, 0) -- cgit v1.2.1