File 0002-units-enable-waiting-for-unit-termination-... of Package systemd.4328

Overview Repositories Revisions Requests Users Attributes Meta

File 0002-units-enable-waiting-for-unit-termination-in-certain.patch of Package systemd.4328

From e9db43d5910717a1084924c512bf85e2b8265375 Mon Sep 17 00:00:00 2001
From: Lennart Poettering <lennart@poettering.net>
Date: Tue, 1 Sep 2015 17:25:59 +0200
Subject: [PATCH] units: enable waiting for unit termination in certain
 cases

The legacy cgroup hierarchy does not support reliable empty
notifications in containers and if there are left-over subgroups in a
cgroup. This makes it hard to correctly wait for them running empty, and
thus we previously disabled this logic entirely.

With this change we explicitly check for the container case, and whether
the unit is a "delegation" unit (i.e. one where programs may create
their own subgroups). If we are neither in a container, nor operating on
a delegation unit cgroup empty notifications become reliable and thus we
start waiting for the empty notifications again.

This doesn't really fix the general problem around cgroup notifications
but reduces the effect around it.

(This also reorders #include lines by their focus, as suggsted in
CODING_STYLE. We have to add "virt.h", so let's do that at the right
place.)

Also see #317.

(cherry picked from commit e9db43d5910717a1084924c512bf85e2b8265375)

[ fb: removed the bits taking care of the "delegation" unit. Indeed
      this feature doesn't exist in v210. ]
---
 src/core/unit.c | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/core/unit.c b/src/core/unit.c
index 9d99964..c96320b 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -3175,14 +3175,18 @@ int unit_kill_context(
                 } else if (r > 0) {
 
                         /* FIXME: For now, we will not wait for the
-                         * cgroup members to die, simply because
-                         * cgroup notification is unreliable. It
-                         * doesn't work at all in containers, and
-                         * outside of containers it can be confused
-                         * easily by leaving directories in the
-                         * cgroup. */
-
-                        /* wait_for_exit = true; */
+                         * 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. */
+
+                        if  (detect_container(NULL) == 0)
+                                wait_for_exit = true;
 
                         if (c->send_sighup && !sigkill) {
                                 set_free(pid_set);
-- 
2.6.2

Places

File 0002-units-enable-waiting-for-unit-termination-in-certain.patch of Package systemd.4328

Places