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

scx_lavd: automatically determine power mode and more #584

Merged
merged 4 commits into from
Aug 29, 2024

Conversation

multics69
Copy link
Contributor

The main purpose of this PR is to provide the "--auto" option, which automatically determines power mode selection (performance, balanced, powersave) based on a user's energy profile selection. It periodically checks the EPP and changes LAVD's core selection preference, etc., according to the EPP value.

Besides this main change, it includes the following minor bug fixes and optimizations:

  • minor optimization for consume_task()
  • bug fix for improper handling of ctrl-c in the monitoring mode
  • removed unused functions in the rust code

Changwoo Min added 4 commits August 28, 2024 10:02
Ctrl-c wasn't properly handled in the monitoring mode
(`--monitor-sched-samples`), so the scheduler could not be terminated by
pressing ctrl-c. The missing ctrl-c handling is added to the monitor
thread.

Signed-off-by: Changwoo Min <[email protected]>
When iterating neighbors, the existing code unnecessarily iterates all
the neighbors to the maximum even if there is no neighors. So the fix
escapes early when there is no neighbors.

Signed-off-by: Changwoo Min <[email protected]>
It checkes the EPP (energy performance preference) peirodically and sets
the power profile of the scheduler during runtiime as a user changes its
EPP profile (from her desktop UI).

Signed-off-by: Changwoo Min <[email protected]>
@multics69 multics69 requested a review from htejun August 29, 2024 10:28
@multics69 multics69 merged commit 3e2e78a into sched-ext:main Aug 29, 2024
2 checks passed
@multics69 multics69 deleted the lavd-turbo2 branch August 29, 2024 23:56
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