cgroup: if we do a cgroup operation then do something on all supported controllers
Previously we did operations like attach, trim or migrate only on the controllers that were enabled for a specific unit. With this changes we will now do them for all supproted controllers, and fall back to all possible prefix paths if the specified paths do not exist. This fixes issues if a controller is being disabled for a unit where it was previously enabled, and makes sure that all processes stay as "far down" the tree as groups exist.
This commit is contained in:
+2
-2
@@ -1017,7 +1017,7 @@ int exec_spawn(ExecCommand *command,
|
||||
bool apply_chroot,
|
||||
bool apply_tty_stdin,
|
||||
bool confirm_spawn,
|
||||
CGroupControllerMask cgroup_mask,
|
||||
CGroupControllerMask cgroup_supported,
|
||||
const char *cgroup_path,
|
||||
const char *unit_id,
|
||||
int idle_pipe[4],
|
||||
@@ -1198,7 +1198,7 @@ int exec_spawn(ExecCommand *command,
|
||||
}
|
||||
|
||||
if (cgroup_path) {
|
||||
err = cg_attach_with_mask(cgroup_mask, cgroup_path, 0);
|
||||
err = cg_attach_everywhere(cgroup_supported, cgroup_path, 0);
|
||||
if (err < 0) {
|
||||
r = EXIT_CGROUP;
|
||||
goto fail_child;
|
||||
|
||||
Reference in New Issue
Block a user