diff options
| author | Raul E Rangel <rrangel@chromium.org> | 2021-07-15 13:20:58 -0600 |
|---|---|---|
| committer | Raul Rangel <rrangel@chromium.org> | 2021-07-18 15:16:13 +0000 |
| commit | 4aec58dce7e7c17ddb943f995c794487660a4f2e (patch) | |
| tree | 8c2944d79aea82d7fc3d46bd9ca15a83241dfd19 /src/include | |
| parent | cc01da50b72e34859b5507ed0f4ae367da4ffaea (diff) | |
lib/thread: Make thread_run not block the current state
If a thread wants to block a state transition it can use
thread_run_until. Otherwise just let the thread run. `thread_join` can
be used to block on the thread. Boot states are also a ramstage concept.
If we want to use this API in any other stage, we need a way of starting
a thread without talking about stages.
BUG=b:179699789
TEST=verify thread_run no longer blocks the current state
Signed-off-by: Raul E Rangel <rrangel@chromium.org>
Change-Id: I3e5b0aed70385ddcd23ffcf7b063f8ccb547fc05
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56351
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/thread.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/include/thread.h b/src/include/thread.h index f9591c949f..7969f575fe 100644 --- a/src/include/thread.h +++ b/src/include/thread.h @@ -46,9 +46,8 @@ void threads_initialize(void); */ void *arch_get_thread_stackbase(void); /* Run func(arrg) on a new thread. Return 0 on successful start of thread, < 0 - * when thread could not be started. Note that the thread will block the - * current state in the boot state machine until it is complete. The thread - * handle if populated, will reflect the state and return code of the thread. + * when thread could not be started. The thread handle if populated, will + * reflect the state and return code of the thread. */ int thread_run(struct thread_handle *handle, enum cb_err (*func)(void *), void *arg); /* thread_run_until is the same as thread_run() except that it blocks state |