Improve handling of tasks on BSP and AP CPUs #278
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wipawel
added
the
improvement
Signed-off-by: Pawel Wieczorkiewicz <[email protected]>
Running test_main() as a task on BSP blocks scheduling and execution of other tasks on BSP (chicken-egg problem), which get created and scheduled from within the test_main() itself. Signed-off-by: Pawel Wieczorkiewicz <[email protected]>
In the wait_for_all_cpus() wait for all unfinished CPUs and/or CPUs without empty task_queue. But do not wait for BSP as it typically is the expected caller of this function. Also, do not unlink CPU struct from the cpus list, there might be more task upcoming. Signed-off-by: Pawel Wieczorkiewicz <[email protected]>
All AP CPUs after empting their task_queue get into blocked state, which makes run_tasks() wait on them until they get unblocked. The BSP is never waited on as it is supposed to unblock the AP CPUs. After an unblock, run_tasks() sets the CPU into unfinished state, before executing any tasks from the task_queue. That way BSP controls when all APs start executing their tasks and when they are finished with it. Signed-off-by: Pawel Wieczorkiewicz <[email protected]>
The execute_tasks() helper is responsible for collectively unblocking all CPUs, calling run_tasks() on BSP CPU and waiting for all AP CPUs untile they are finished. In kernel_main the execute_tasks() is called before test_main() just in case. Currently there are no tasks scheduled before the test_main(). The execute_tasks() called after the test_main() makes sure all APs and the BSP get unblocked and their tasks executed or the states of the CPUs are set to finished. Signed-off-by: Pawel Wieczorkiewicz <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.