Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:Leap:42.1:Staging:B
systemd
0002-units-enable-waiting-for-unit-termination-...
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 0002-units-enable-waiting-for-unit-termination-in-certain.patch of Package systemd
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. --- src/core/cgroup.c | 12 ++++++++++++ src/core/cgroup.h | 2 ++ src/core/unit.c | 18 +++++++++++------- 3 files changed, 25 insertions(+), 7 deletions(-) --- src/core/cgroup.c +++ src/core/cgroup.c 2015-09-03 00:00:00.000000000 +0000 @@ -964,6 +964,18 @@ int manager_notify_cgroup_empty(Manager return 0; } +bool unit_cgroup_delegate(Unit *u) { + CGroupContext *c; + + assert(u); + + c = unit_get_cgroup_context(u); + if (!c) + return false; + + return c->delegate; +} + static const char* const cgroup_device_policy_table[_CGROUP_DEVICE_POLICY_MAX] = { [CGROUP_AUTO] = "auto", [CGROUP_CLOSED] = "closed", --- src/core/cgroup.h +++ src/core/cgroup.h 2015-09-03 00:00:00.000000000 +0000 @@ -118,5 +118,7 @@ pid_t unit_search_main_pid(Unit *u); int manager_notify_cgroup_empty(Manager *m, const char *group); +bool unit_cgroup_delegate(Unit *u); + const char* cgroup_device_policy_to_string(CGroupDevicePolicy i) _const_; CGroupDevicePolicy cgroup_device_policy_from_string(const char *s) _pure_; --- src/core/unit.c +++ src/core/unit.c 2015-09-03 00:00:00.000000000 +0000 @@ -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. */ + * 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. */ - /* wait_for_exit = true; */ + if (detect_container(NULL) == 0 && !unit_cgroup_delegate(u)) + wait_for_exit = true; if (c->send_sighup && !sigkill) { set_free(pid_set);
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor