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
linux-common: Add CFS_BANDWIDTH #35043
Conversation
The option is needed to limit CPU usage of processes. It seems to have no negative impact if left unused. See https://lwn.net/Articles/428230/
@GrahamcOfBorg test kernel-lts kernel-latest |
I may also need to undo #19251, so there may be more to it. I am currently testing this. |
Success on aarch64-linux (full log) Partial log (click to expand)
|
Failure on x86_64-linux (full log) Partial log (click to expand)
|
Does |
@Mic92 Both |
@NeQuissimus they probably still function, but does it impact userland realtime scheduling? |
Not sure how to make sure of that. |
@NeQuissimus the option should be fine. Overhead is nearly unmeasurable even in synthetic benchmarks. |
The option is needed to limit CPU usage of processes.
It seems to have no negative impact if left unused.
See https://lwn.net/Articles/428230/
Motivation for this change
I need this to limit Docker container resources:
https://docs.docker.com/config/containers/resource_constraints/#configure-the-default-cfs-scheduler
Since there seems to be no negative impact to people not using the functionality, I propose this be added to the common configuration.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)