diff options
| author | Keith Short <keithshort@chromium.org> | 2022-03-30 18:18:08 -0600 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-03-31 20:28:50 +0000 |
| commit | 1d844062cdf82d56cd4190f92c2fab592da7bc36 (patch) | |
| tree | bf2325e854778f2eeb69e49c992f30ece8be09c6 /zephyr/app/ec | |
| parent | 9198149037b783de361169e91793aba964c54ff9 (diff) | |
| download | chrome-ec-1d844062cdf82d56cd4190f92c2fab592da7bc36.tar.gz | |
zephyr: Fix the override of the system workqueue
After the conversion of HOOK_INIT to SYS_INIT within the Zephyr shim
code, the EC reported a false error that the sysworkq priority was
incorrect.
Move the code that sets the system workqueue thread priority to sooner
in the initialization order. Also move the check of the system
workqueue thread to occur after all other threads are started.
BUG=none
BRANCH=none
TEST=zmake testall
TEST=Run "kernel threads" on byra and verify system workqueue is set to
the lowest priority, but above the idle priority.
Signed-off-by: Keith Short <keithshort@chromium.org>
Change-Id: I14743cbb6643dce9e9b73584d75ec3298c8d9117
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3562002
Reviewed-by: Denis Brockus <dbrockus@chromium.org>
Diffstat (limited to 'zephyr/app/ec')
| -rw-r--r-- | zephyr/app/ec/ec_app_main.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/zephyr/app/ec/ec_app_main.c b/zephyr/app/ec/ec_app_main.c index 32ad941f6c..3e01cf04ee 100644 --- a/zephyr/app/ec/ec_app_main.c +++ b/zephyr/app/ec/ec_app_main.c @@ -74,14 +74,6 @@ void ec_app_main(void) } /* - * Hooks run from the system workqueue and must be the lowest priority - * thread. By default, the system workqueue is run at the lowest - * cooperative thread priority, blocking all preemptive threads until - * the deferred work is completed. - */ - k_thread_priority_set(&k_sys_work_q.thread, LOWEST_THREAD_PRIORITY); - - /* * Print the init time. Not completely accurate because it can't take * into account the time before timer_init(), but it'll at least catch * the majority of the time. |