core: output unit status output strings to console, only if we actually are changing...

[elogind.git] / src / core / scope.c

diff --git a/src/core/scope.c b/src/core/scope.c
index 0c1d17e68c569a2fd5bcfaa1261db29a756875c3..b41db7872c4ef5a2811aacc7ff2ffe2249f79911 100644 (file)
--- a/src/core/scope.c
+++ b/src/core/scope.c
@@ -35,6 +35,7 @@
 static const UnitActiveState state_translation_table[_SCOPE_STATE_MAX] = {
         [SCOPE_DEAD] = UNIT_INACTIVE,
         [SCOPE_RUNNING] = UNIT_ACTIVE,
 static const UnitActiveState state_translation_table[_SCOPE_STATE_MAX] = {
         [SCOPE_DEAD] = UNIT_INACTIVE,
         [SCOPE_RUNNING] = UNIT_ACTIVE,

+        [SCOPE_ABANDONED] = UNIT_ACTIVE,

         [SCOPE_STOP_SIGTERM] = UNIT_DEACTIVATING,
         [SCOPE_STOP_SIGKILL] = UNIT_DEACTIVATING,
         [SCOPE_FAILED] = UNIT_FAILED
         [SCOPE_STOP_SIGTERM] = UNIT_DEACTIVATING,
         [SCOPE_STOP_SIGKILL] = UNIT_DEACTIVATING,
         [SCOPE_FAILED] = UNIT_FAILED


@@ -50,9 +51,6 @@ static void scope_init(Unit *u) {
 
         s->timeout_stop_usec = u->manager->default_timeout_stop_usec;
 
 
         s->timeout_stop_usec = u->manager->default_timeout_stop_usec;
 

-        cgroup_context_init(&s->cgroup_context);
-        kill_context_init(&s->kill_context);
-

         UNIT(s)->ignore_on_isolate = true;
         UNIT(s)->ignore_on_snapshot = true;
 }
         UNIT(s)->ignore_on_isolate = true;
         UNIT(s)->ignore_on_snapshot = true;
 }


@@ -62,13 +60,8 @@ static void scope_done(Unit *u) {
 
         assert(u);
 
 
         assert(u);
 

-        cgroup_context_done(&s->cgroup_context);
-

         free(s->controller);
 
         free(s->controller);
 

-        set_free(s->pids);
-        s->pids = NULL;
-

         s->timer_event_source = sd_event_source_unref(s->timer_event_source);
 }
 
         s->timer_event_source = sd_event_source_unref(s->timer_event_source);
 }
 


@@ -90,7 +83,12 @@ static int scope_arm_timer(Scope *s) {
                 return sd_event_source_set_enabled(s->timer_event_source, SD_EVENT_ONESHOT);
         }
 
                 return sd_event_source_set_enabled(s->timer_event_source, SD_EVENT_ONESHOT);
         }
 

-        return sd_event_add_monotonic(UNIT(s)->manager->event, now(CLOCK_MONOTONIC) + s->timeout_stop_usec, 0, scope_dispatch_timer, s, &s->timer_event_source);
+        return sd_event_add_time(
+                        UNIT(s)->manager->event,
+                        &s->timer_event_source,
+                        CLOCK_MONOTONIC,
+                        now(CLOCK_MONOTONIC) + s->timeout_stop_usec, 0,
+                        scope_dispatch_timer, s);

 }
 
 static void scope_set_state(Scope *s, ScopeState state) {
 }
 
 static void scope_set_state(Scope *s, ScopeState state) {


@@ -100,15 +98,14 @@ static void scope_set_state(Scope *s, ScopeState state) {
         old_state = s->state;
         s->state = state;
 
         old_state = s->state;
         s->state = state;
 

-        if (state != SCOPE_STOP_SIGTERM &&
-            state != SCOPE_STOP_SIGKILL)
+        if (!IN_SET(state, SCOPE_STOP_SIGTERM, SCOPE_STOP_SIGKILL))

                 s->timer_event_source = sd_event_source_unref(s->timer_event_source);
 
                 s->timer_event_source = sd_event_source_unref(s->timer_event_source);
 

+        if (IN_SET(state, SCOPE_DEAD, SCOPE_FAILED))
+                unit_unwatch_all_pids(UNIT(s));
+

         if (state != old_state)
         if (state != old_state)

-                log_debug("%s changed %s -> %s",
-                          UNIT(s)->id,
-                          scope_state_to_string(old_state),
-                          scope_state_to_string(state));
+                log_debug("%s changed %s -> %s", UNIT(s)->id, scope_state_to_string(old_state), scope_state_to_string(state));

 
         unit_notify(UNIT(s), state_translation_table[old_state], state_translation_table[state], true);
 }
 
         unit_notify(UNIT(s), state_translation_table[old_state], state_translation_table[state], true);
 }


@@ -135,8 +132,8 @@ static int scope_verify(Scope *s) {
         if (UNIT(s)->load_state != UNIT_LOADED)
                 return 0;
 
         if (UNIT(s)->load_state != UNIT_LOADED)
                 return 0;
 

-        if (set_size(s->pids) <= 0 && UNIT(s)->manager->n_reloading <= 0) {
-                log_error_unit(UNIT(s)->id, "Scope %s has no PIDs. Refusing.", UNIT(s)->id);
+        if (set_isempty(UNIT(s)->pids) && UNIT(s)->manager->n_reloading <= 0) {
+                log_unit_error(UNIT(s)->id, "Scope %s has no PIDs. Refusing.", UNIT(s)->id);

                 return -EINVAL;
         }
 
                 return -EINVAL;
         }
 


@@ -159,7 +156,11 @@ static int scope_load(Unit *u) {
         if (r < 0)
                 return r;
 
         if (r < 0)
                 return r;
 

-        r = unit_add_default_slice(u);
+        r = unit_patch_contexts(u);
+        if (r < 0)
+                return r;
+
+        r = unit_add_default_slice(u, &s->cgroup_context);

         if (r < 0)
                 return r;
 
         if (r < 0)
                 return r;
 


@@ -181,12 +182,15 @@ static int scope_coldplug(Unit *u) {
 
         if (s->deserialized_state != s->state) {
 
 
         if (s->deserialized_state != s->state) {
 

-                if (s->deserialized_state == SCOPE_STOP_SIGKILL || s->deserialized_state == SCOPE_STOP_SIGTERM) {
+                if (IN_SET(s->deserialized_state, SCOPE_STOP_SIGKILL, SCOPE_STOP_SIGTERM)) {

                         r = scope_arm_timer(s);
                         if (r < 0)
                                 return r;
                 }
 
                         r = scope_arm_timer(s);
                         if (r < 0)
                                 return r;
                 }
 

+                if (!IN_SET(s->deserialized_state, SCOPE_DEAD, SCOPE_FAILED))
+                        unit_watch_all_pids(UNIT(s));
+

                 scope_set_state(s, s->deserialized_state);
         }
 
                 scope_set_state(s, s->deserialized_state);
         }
 


@@ -227,6 +231,8 @@ static void scope_enter_signal(Scope *s, ScopeState state, ScopeResult f) {
         if (f != SCOPE_SUCCESS)
                 s->result = f;
 
         if (f != SCOPE_SUCCESS)
                 s->result = f;
 

+        unit_watch_all_pids(UNIT(s));
+

         /* If we have a controller set let's ask the controller nicely
          * to terminate the scope, instead of us going directly into
          * SIGTERM beserk mode */
         /* If we have a controller set let's ask the controller nicely
          * to terminate the scope, instead of us going directly into
          * SIGTERM beserk mode */


@@ -237,7 +243,7 @@ static void scope_enter_signal(Scope *s, ScopeState state, ScopeResult f) {
                 r = unit_kill_context(
                                 UNIT(s),
                                 &s->kill_context,
                 r = unit_kill_context(
                                 UNIT(s),
                                 &s->kill_context,

-                                state != SCOPE_STOP_SIGTERM,
+                                state != SCOPE_STOP_SIGTERM ? KILL_KILL : KILL_TERMINATE,

                                 -1, -1, false);
                 if (r < 0)
                         goto fail;
                                 -1, -1, false);
                 if (r < 0)
                         goto fail;


@@ -258,7 +264,7 @@ static void scope_enter_signal(Scope *s, ScopeState state, ScopeResult f) {
         return;
 
 fail:
         return;
 
 fail:

-        log_warning_unit(UNIT(s)->id,
+        log_unit_warning(UNIT(s)->id,

                          "%s failed to kill processes: %s", UNIT(s)->id, strerror(-r));
 
         scope_enter_dead(s, SCOPE_FAILURE_RESOURCES);
                          "%s failed to kill processes: %s", UNIT(s)->id, strerror(-r));
 
         scope_enter_dead(s, SCOPE_FAILURE_RESOURCES);


@@ -282,39 +288,30 @@ static int scope_start(Unit *u) {
         if (!u->transient && UNIT(s)->manager->n_reloading <= 0)
                 return -ENOENT;
 
         if (!u->transient && UNIT(s)->manager->n_reloading <= 0)
                 return -ENOENT;
 

-        r = unit_realize_cgroup(u);
-        if (r < 0) {
-                log_error("Failed to realize cgroup: %s", strerror(-r));
-                return r;
-        }
-
-        r = cg_attach_many_everywhere(u->manager->cgroup_supported, u->cgroup_path, s->pids);
+        r = unit_attach_pids_to_cgroup(u);

         if (r < 0)
                 return r;
 
         if (r < 0)
                 return r;
 

-        set_free(s->pids);
-        s->pids = NULL;
-

         s->result = SCOPE_SUCCESS;
 
         scope_set_state(s, SCOPE_RUNNING);
         s->result = SCOPE_SUCCESS;
 
         scope_set_state(s, SCOPE_RUNNING);

-        return 0;
+        return 1;

 }
 
 static int scope_stop(Unit *u) {
         Scope *s = SCOPE(u);
 
         assert(s);
 }
 
 static int scope_stop(Unit *u) {
         Scope *s = SCOPE(u);
 
         assert(s);

-        assert(s->state == SCOPE_RUNNING);

 
         if (s->state == SCOPE_STOP_SIGTERM ||
             s->state == SCOPE_STOP_SIGKILL)
                 return 0;
 
 
         if (s->state == SCOPE_STOP_SIGTERM ||
             s->state == SCOPE_STOP_SIGKILL)
                 return 0;
 

-        assert(s->state == SCOPE_RUNNING);
+        assert(s->state == SCOPE_RUNNING ||
+               s->state == SCOPE_ABANDONED);

 
         scope_enter_signal(s, SCOPE_STOP_SIGTERM, SCOPE_SUCCESS);
 
         scope_enter_signal(s, SCOPE_STOP_SIGTERM, SCOPE_SUCCESS);

-        return 0;
+        return 1;

 }
 
 static void scope_reset_failed(Unit *u) {
 }
 
 static void scope_reset_failed(Unit *u) {


@@ -381,16 +378,15 @@ static int scope_deserialize_item(Unit *u, const char *key, const char *value, F
 }
 
 static bool scope_check_gc(Unit *u) {
 }
 
 static bool scope_check_gc(Unit *u) {

-        Scope *s = SCOPE(u);
-        int r;
-
-        assert(s);
+        assert(u);

 
         /* Never clean up scopes that still have a process around,
          * even if the scope is formally dead. */
 
 
         /* Never clean up scopes that still have a process around,
          * even if the scope is formally dead. */
 

-        if (UNIT(s)->cgroup_path) {
-                r = cg_is_empty_recursive(SYSTEMD_CGROUP_CONTROLLER, UNIT(s)->cgroup_path, true);
+        if (u->cgroup_path) {
+                int r;
+
+                r = cg_is_empty_recursive(SYSTEMD_CGROUP_CONTROLLER, u->cgroup_path, true);

                 if (r <= 0)
                         return true;
         }
                 if (r <= 0)
                         return true;
         }


@@ -398,6 +394,32 @@ static bool scope_check_gc(Unit *u) {
         return false;
 }
 
         return false;
 }
 

+static void scope_notify_cgroup_empty_event(Unit *u) {
+        Scope *s = SCOPE(u);
+        assert(u);
+
+        log_unit_debug(u->id, "%s: cgroup is empty", u->id);
+
+        if (IN_SET(s->state, SCOPE_RUNNING, SCOPE_ABANDONED, SCOPE_STOP_SIGTERM, SCOPE_STOP_SIGKILL))
+                scope_enter_dead(s, SCOPE_SUCCESS);
+}
+
+static void scope_sigchld_event(Unit *u, pid_t pid, int code, int status) {
+
+        /* If we get a SIGCHLD event for one of the processes we were
+           interested in, then we look for others to watch, under the
+           assumption that we'll sooner or later get a SIGCHLD for
+           them, as the original process we watched was probably the
+           parent of them, and they are hence now our children. */
+
+        unit_tidy_watch_pids(u, 0, 0);
+        unit_watch_all_pids(u);
+
+        /* If the PID set is empty now, then let's finish this off */
+        if (set_isempty(u->pids))
+                scope_notify_cgroup_empty_event(u);
+}
+

 static int scope_dispatch_timer(sd_event_source *source, usec_t usec, void *userdata) {
         Scope *s = SCOPE(userdata);
 
 static int scope_dispatch_timer(sd_event_source *source, usec_t usec, void *userdata) {
         Scope *s = SCOPE(userdata);
 


@@ -408,17 +430,17 @@ static int scope_dispatch_timer(sd_event_source *source, usec_t usec, void *user
 
         case SCOPE_STOP_SIGTERM:
                 if (s->kill_context.send_sigkill) {
 
         case SCOPE_STOP_SIGTERM:
                 if (s->kill_context.send_sigkill) {

-                        log_warning_unit(UNIT(s)->id, "%s stopping timed out. Killing.", UNIT(s)->id);
+                        log_unit_warning(UNIT(s)->id, "%s stopping timed out. Killing.", UNIT(s)->id);

                         scope_enter_signal(s, SCOPE_STOP_SIGKILL, SCOPE_FAILURE_TIMEOUT);
                 } else {
                         scope_enter_signal(s, SCOPE_STOP_SIGKILL, SCOPE_FAILURE_TIMEOUT);
                 } else {

-                        log_warning_unit(UNIT(s)->id, "%s stopping timed out. Skipping SIGKILL.", UNIT(s)->id);
+                        log_unit_warning(UNIT(s)->id, "%s stopping timed out. Skipping SIGKILL.", UNIT(s)->id);

                         scope_enter_dead(s, SCOPE_FAILURE_TIMEOUT);
                 }
 
                 break;
 
         case SCOPE_STOP_SIGKILL:
                         scope_enter_dead(s, SCOPE_FAILURE_TIMEOUT);
                 }
 
                 break;
 
         case SCOPE_STOP_SIGKILL:

-                log_warning_unit(UNIT(s)->id, "%s still around after SIGKILL. Ignoring.", UNIT(s)->id);
+                log_unit_warning(UNIT(s)->id, "%s still around after SIGKILL. Ignoring.", UNIT(s)->id);

                 scope_enter_dead(s, SCOPE_FAILURE_TIMEOUT);
                 break;
 
                 scope_enter_dead(s, SCOPE_FAILURE_TIMEOUT);
                 break;
 


@@ -429,24 +451,30 @@ static int scope_dispatch_timer(sd_event_source *source, usec_t usec, void *user
         return 0;
 }
 
         return 0;
 }
 

-static void scope_notify_cgroup_empty_event(Unit *u) {
-        Scope *s = SCOPE(u);
-        assert(u);
+int scope_abandon(Scope *s) {
+        assert(s);

 
 

-        log_debug_unit(u->id, "%s: cgroup is empty", u->id);
+        if (!IN_SET(s->state, SCOPE_RUNNING, SCOPE_ABANDONED))
+                return -ESTALE;

 
 

-        switch (s->state) {
+        free(s->controller);
+        s->controller = NULL;

 
 

-        case SCOPE_RUNNING:
-        case SCOPE_STOP_SIGTERM:
-        case SCOPE_STOP_SIGKILL:
-                scope_enter_dead(s, SCOPE_SUCCESS);
+        /* The client is no longer watching the remaining processes,
+         * so let's step in here, under the assumption that the
+         * remaining processes will be sooner or later reassigned to
+         * us as parent. */

 
 

-                break;
+        unit_tidy_watch_pids(UNIT(s), 0, 0);
+        unit_watch_all_pids(UNIT(s));

 
 

-        default:
-                ;
-        }
+        /* If the PID set is empty now, then let's finish this off */
+        if (set_isempty(UNIT(s)->pids))
+                scope_notify_cgroup_empty_event(UNIT(s));
+        else
+                scope_set_state(s, SCOPE_ABANDONED);
+
+        return 0;

 }
 
 _pure_ static UnitActiveState scope_active_state(Unit *u) {
 }
 
 _pure_ static UnitActiveState scope_active_state(Unit *u) {


@@ -464,6 +492,7 @@ _pure_ static const char *scope_sub_state_to_string(Unit *u) {
 static const char* const scope_state_table[_SCOPE_STATE_MAX] = {
         [SCOPE_DEAD] = "dead",
         [SCOPE_RUNNING] = "running",
 static const char* const scope_state_table[_SCOPE_STATE_MAX] = {
         [SCOPE_DEAD] = "dead",
         [SCOPE_RUNNING] = "running",

+        [SCOPE_ABANDONED] = "abandoned",

         [SCOPE_STOP_SIGTERM] = "stop-sigterm",
         [SCOPE_STOP_SIGKILL] = "stop-sigkill",
         [SCOPE_FAILED] = "failed",
         [SCOPE_STOP_SIGTERM] = "stop-sigterm",
         [SCOPE_STOP_SIGKILL] = "stop-sigkill",
         [SCOPE_FAILED] = "failed",


@@ -516,6 +545,8 @@ const UnitVTable scope_vtable = {
 
         .check_gc = scope_check_gc,
 
 
         .check_gc = scope_check_gc,
 

+        .sigchld_event = scope_sigchld_event,
+

         .reset_failed = scope_reset_failed,
 
         .notify_cgroup_empty = scope_notify_cgroup_empty_event,
         .reset_failed = scope_reset_failed,
 
         .notify_cgroup_empty = scope_notify_cgroup_empty_event,