michaelkuc6/blog_os
1
0
Fork 0
mirror of https://github.com/phil-opp/blog_os.git synced 2025-12-17 14:57:49 +00:00
Code Issues Projects Releases Wiki Activity

Update x86_64 dependency to version 0.11.0

Browse Source
This commit is contained in:
Philipp Oppermann
2020-05-20 14:35:06 +02:00
parent a8c85afeb0
commit ee09a70d40
8 changed files with 65 additions and 78 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
blog/content/second-edition/posts/12-async-await/index.md
View File

@@ -1744,8 +1744,8 @@ impl Executor {
Since we call `sleep_if_idle` directly after `run_ready_tasks`, which loops until the `task_queue` becomes empty, checking the queue again might seem unnecessary. However, a hardware interrupt might occur directly after `run_ready_tasks` returns, so there might be a new task in the queue at the time the `sleep_if_idle` function is called. Only if the queue is still empty, we put the CPU to sleep by executing the `hlt` instruction through the [`instructions::hlt`] wrapper function provided by the [`x86_64`] crate.
[`instructions::hlt`]: https://docs.rs/x86_64/0.9.6/x86_64/instructions/fn.hlt.html
[`x86_64`]: https://docs.rs/x86_64/0.9.6/x86_64/index.html
[`instructions::hlt`]: https://docs.rs/x86_64/0.11.0/x86_64/instructions/fn.hlt.html
[`x86_64`]: https://docs.rs/x86_64/0.11.0/x86_64/index.html
Unfortunately, there is still a subtle race condition in this implementation. Since interrupts are asynchronous and can happen at any time, it is possible that an interrupt happens right between the `is_empty` check and the call to `hlt`:
@@ -1760,7 +1760,7 @@ In case this interrupt pushes to the `task_queue`, we put the CPU to sleep even
The answer is to disable interrupts on the CPU before the check and atomically enable them again together with the `hlt` instruction. This way, all interrupts that happen in between are delayed after the `hlt` instruction so that no wake-ups are missed. To implement this approach, we can use the [`enable_interrupts_and_hlt`] function provided by the [`x86_64`] crate. This function is only available since version 0.9.6, so you might need to update your `x86_64` dependency to use it.
[`enable_interrupts_and_hlt`]: https://docs.rs/x86_64/0.9.6/x86_64/instructions/interrupts/fn.enable_interrupts_and_hlt.html
[`enable_interrupts_and_hlt`]: https://docs.rs/x86_64/0.11.0/x86_64/instructions/interrupts/fn.enable_interrupts_and_hlt.html
The updated implementation of our `sleep_if_idle` function looks like this: