Skip to content

systemd: Delegate cgroup management without turning on controllers.#14965

Merged
edolstra merged 1 commit intoNixOS:masterfrom
shlevy:controllerless-delegation
Jan 16, 2026
Merged

systemd: Delegate cgroup management without turning on controllers.#14965
edolstra merged 1 commit intoNixOS:masterfrom
shlevy:controllerless-delegation

Conversation

@shlevy
Copy link
Member

@shlevy shlevy commented Jan 10, 2026

Nix currently doesn't do any resource control, and Delegate=yes turns on all the controllers.

In particular, this enables using cpusets with cgroups V1 alongside the Nix daemon.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

Nix currently doesn't do any resource control, and Delegate=yes turns on all the controllers.

In particular, this enables using cpusets with cgroups V1 alongside the Nix daemon.
@shlevy shlevy requested a review from edolstra as a code owner January 10, 2026 04:23
@Ericson2314
Copy link
Member

#11414 c.f. this is when it was originally added.

@shlevy
Copy link
Member Author

shlevy commented Jan 11, 2026

Before:

$ sudo cset set -c 0 cpu0
mount: /cpusets: none already mounted on /run/keys.
       dmesg(1) may have more information after failed mount system call.
cset: **> mount of cpuset filesystem failed, do you have permission?

After:

$ sudo cset set -c 0 cpu0
cset: --> created cpuset "cpu0"

@Mic92
Copy link
Member

Mic92 commented Jan 13, 2026

cc @parkerhoyes

@edolstra
Copy link
Member

Looks good to me. I guess we'll have to change this back if Nix does start doing resource control (see e.g. #7388)?

@shlevy
Copy link
Member Author

shlevy commented Jan 14, 2026

You can change it back to the list of specific controllers at least.

@edolstra edolstra added this pull request to the merge queue Jan 16, 2026
Merged via the queue into NixOS:master with commit f1d468e Jan 16, 2026
14 checks passed
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.

4 participants