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.

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

i#6831 sched refactor, step 7: Split eof_or_idle #7083

Merged
merged 7 commits into from
Nov 15, 2024
Merged

Conversation

derekbruening
Copy link
Contributor

Splits eof_or_idle() into separate overrides in the 3 scheduler subclasses. This then allows moving pop_from_ready_queue*() into the dynamic subclass.

Moves process_markers(), ready_queue_empty(), and
syscall_incurs_switch() into the dynamic subclass.

Issue: #6831

Splits set_initial_schedule() into separate overrides in the 3
scheduler subclasses, sharing the initial content reading in init()
prior to calling set_initial_schedule().

Splits set_output_active(): it is only valid in dynamic.

Moves rebalance_queues() and two rebalance vars to dynamic.

Moves read_recorded_schedule() and
read_and_instantiate_traced_schedule() to replay.

Adds more using statements to remove the need for "this->" prefixes on
frequent base class member field references.

Issue: #6831
Splits eof_or_idle() into separate overrides in the 3 scheduler
subclasses.  This then allows moving pop_from_ready_queue*() into the
dynamic subclass.

Moves process_markers(), ready_queue_empty(), and
syscall_incurs_switch() into the dynamic subclass.

Issue: #6831
Base automatically changed from i6831-split-init to master November 14, 2024 20:01
@derekbruening
Copy link
Contributor Author

a64 failure is the thread exit before a bundle #3320

@derekbruening derekbruening merged commit 336bac6 into master Nov 15, 2024
16 of 17 checks passed
@derekbruening derekbruening deleted the i6831-split-eof branch November 15, 2024 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants