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
cpu-freq: Try powersave if ondemand is not available #21423
Conversation
@gnidorah, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edolstra, @peti and @Mathnerd314 to be potential reviewers. |
Probably figuring correct governor on install stage and setting it at hardware-configuration.nix should be a better approach (if that's possible). |
See e.g., |
This reverts commit 4dc56db.
@joachifm Okay, let's try this 2c8253c325834e16251936e994c100e893d79253 |
@@ -69,9 +69,6 @@ in | |||
|
|||
config = mkIf cfg.enable { | |||
|
|||
# FIXME: Implement powersave governor for sandy bridge or later Intel CPUs | |||
powerManagement.cpuFreqGovernor = mkDefault "ondemand"; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably should keep this, since almost nobody reruns the config generator.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@grahamc Fixed, thanks!
Motivation for this change
This should fix default governor picking on Sandy Bridge+ chips and close #9611
Not all people monitor their CPUs at least eventually. And we don't want them to shorten lives of their machines.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)