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#6938 sched migrate: Add migration count to scheduler #6950

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

derekbruening
Copy link
Contributor

Adds the migration count to the new scheduler stats. Adds checks of the count to various unit tests.

Issue: #6938

Adds the migration count to the new scheduler stats.
Adds checks of the count to various unit tests.

Issue: #6938
@derekbruening
Copy link
Contributor Author

a64 failures seem to happen every time now: #6733. Re-running; expect to go green.

clients/drcachesim/common/memtrace_stream.h Outdated Show resolved Hide resolved
@derekbruening
Copy link
Contributor Author

x64 failure is sim.invariants #6733

@derekbruening derekbruening merged commit dbd993e into master Aug 29, 2024
16 of 17 checks passed
@derekbruening derekbruening deleted the i6938-migration-stats branch August 29, 2024 16:32
derekbruening added a commit that referenced this pull request Sep 9, 2024
Direct thread switches were not being counted as migrations in the
original migration stats in PR #6950: we address that here.  A test
with multiple outputs will be added later as part of the move to
separate runqueues, which is where this bug was discovered.

Issue: #6938
derekbruening added a commit that referenced this pull request Sep 10, 2024
Direct thread switches were not being counted as migrations in the
original migration stats in PR #6950: we address that here. A test with
multiple outputs will be added later as part of the move to separate
runqueues, which is where this bug was discovered.

Issue: #6938
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