valeriyvdovin / rpms / systemd

Forked from rpms/systemd 3 years ago
Clone

Blame SOURCES/0188-Revert-core-one-step-back-again-for-nspawn-we-actual.patch

923a60
From 647a7761e2fa423c6e1bd6785b043dbe7b525e3c Mon Sep 17 00:00:00 2001
923a60
From: Lukas Nykryn <lnykryn@redhat.com>
923a60
Date: Thu, 25 Jun 2015 09:20:59 +0200
923a60
Subject: [PATCH] Revert "core: one step back again, for nspawn we actually
923a60
 can't wait for cgroups running empty since systemd will get exactly zero
923a60
 notifications about it"
923a60
923a60
This reverts commit 743970d2ea6d08aa7c7bff8220f6b7702f2b1db7.
923a60
923a60
RHEL-only
923a60
https://bugzilla.redhat.com/show_bug.cgi?id=1141137
923a60
https://github.com/systemd/systemd/pull/350
923a60
923a60
Resolves: #1199644
923a60
---
923a60
 src/core/unit.c | 10 +---------
923a60
 1 file changed, 1 insertion(+), 9 deletions(-)
923a60
923a60
diff --git a/src/core/unit.c b/src/core/unit.c
923a60
index b9e1f13eaf..fa17567dd3 100644
923a60
--- a/src/core/unit.c
923a60
+++ b/src/core/unit.c
923a60
@@ -3546,15 +3546,7 @@ int unit_kill_context(
923a60
                                 log_unit_warning_errno(u->id, r, "Failed to kill control group: %m");
923a60
                 } else if (r > 0) {
923a60
 
923a60
-                        /* FIXME: For now, we will not wait for the
923a60
-                         * cgroup members to die, simply because
923a60
-                         * cgroup notification is unreliable. It
923a60
-                         * doesn't work at all in containers, and
923a60
-                         * outside of containers it can be confused
923a60
-                         * easily by leaving directories in the
923a60
-                         * cgroup. */
923a60
-
923a60
-                        /* wait_for_exit = true; */
923a60
+                        wait_for_exit = true;
923a60
 
923a60
                         if (c->send_sighup && k != KILL_KILL) {
923a60
                                 set_free(pid_set);