Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make core_scheduler sound #912

Merged
merged 7 commits into from
Sep 19, 2023
Merged

Make core_scheduler sound #912

merged 7 commits into from
Sep 19, 2023

Conversation

mkroening
Copy link
Member

Fixes #622

@mkroening mkroening force-pushed the core-scheduler branch 11 times, most recently from fad2688 to 923caac Compare September 14, 2023 08:52
@mkroening mkroening changed the title Rework core_scheduler Make core_scheduler sound Sep 18, 2023
@mkroening mkroening force-pushed the core-scheduler branch 4 times, most recently from 38a482f to 8d65c0f Compare September 18, 2023 16:39
@mkroening
style(scheduler): inline closure
ceeb57d 
Signed-off-by: Martin Kröning <[email protected]>
@mkroening
refactor(scheduler): move scheduler input to core-local storage
64a9620 
Signed-off-by: Martin Kröning <[email protected]>
@mkroening
fix(task): assert task found in custom_wakeup
4deb95d 
Signed-off-by: Martin Kröning <[email protected]>
@mkroening
fix(task): don't access scheduler from BlockedTaskQueue::wakeup_task
a80bc91 
Signed-off-by: Martin Kröning <[email protected]>
@mkroening
refactor(scheduler): move reschedule, add_network_timer, and exit to …
a434f0e 
…extension trait

Signed-off-by: Martin Kröning <[email protected]>
@mkroening
refactor(scheduler): make run take no arguments
ab83ae1 
Signed-off-by: Martin Kröning <[email protected]>
@mkroening
fix(core_local): put scheduler into RefCell
8cbf7f6 
Signed-off-by: Martin Kröning <[email protected]>
@mkroening mkroening marked this pull request as ready for review September 18, 2023 17:59
@mkroening mkroening mentioned this pull request Sep 18, 2023
Closed
stlankes
stlankes approved these changes Sep 19, 2023
View reviewed changes
Copy link
Contributor

@stlankes stlankes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mkroening mkroening added this pull request to the merge queue Sep 19, 2023
Merged via the queue into hermit-os:main with commit 1872cc0 Sep 19, 2023
9 checks passed
@mkroening mkroening deleted the core-scheduler branch October 5, 2023 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stlankes stlankes stlankes approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Safety: Rework core_scheduler
2 participants
@mkroening @stlankes