diff options
author | Lukas Nykryn <lnykryn@redhat.com> | 2015-06-25 09:20:59 +0200 |
---|---|---|
committer | Lukáš Nykrýn <lnykryn@redhat.com> | 2019-04-30 10:50:20 +0200 |
commit | 9d0046ceca10911361137d6496987cb15ffff132 (patch) | |
tree | 32a2dc34cfbddac19ece86e9d4aac51d0ee675a2 | |
parent | 0ed1150ccf8837ca85cbb8d5a42fe81b5efeac32 (diff) |
Revert "core: one step back again, for nspawn we actually can't wait for cgroups running empty since systemd will get exactly zero notifications about it"v239-14
This reverts commit 743970d2ea6d08aa7c7bff8220f6b7702f2b1db7.
RHEL-only
https://bugzilla.redhat.com/show_bug.cgi?id=1141137
https://github.com/systemd/systemd/pull/350
Resolves: #1703485
-rw-r--r-- | src/core/unit.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/src/core/unit.c b/src/core/unit.c index cc43ddc4f1..d298afb0d4 100644 --- a/src/core/unit.c +++ b/src/core/unit.c @@ -4579,16 +4579,7 @@ int unit_kill_context( } else if (r > 0) { - /* FIXME: For now, on the legacy hierarchy, we will not wait for the cgroup members to die if - * we are running in a container or if this is a delegation unit, simply because cgroup - * notification is unreliable in these cases. It doesn't work at all in containers, and outside - * of containers it can be confused easily by left-over directories in the cgroup — which - * however should not exist in non-delegated units. On the unified hierarchy that's different, - * there we get proper events. Hence rely on them. */ - - if (cg_unified_controller(SYSTEMD_CGROUP_CONTROLLER) > 0 || - (detect_container() == 0 && !unit_cgroup_delegate(u))) - wait_for_exit = true; + wait_for_exit = true; if (send_sighup) { set_free(pid_set); |