chiark / gitweb /
main: fix auto restarting of units after a configuration reload
authorLennart Poettering <lennart@poettering.net>
Mon, 9 Aug 2010 21:33:48 +0000 (23:33 +0200)
committerLennart Poettering <lennart@poettering.net>
Mon, 9 Aug 2010 21:53:49 +0000 (23:53 +0200)
fixme
src/service.c
src/service.h

diff --git a/fixme b/fixme
index 373d100..186d3c7 100644 (file)
--- a/fixme
+++ b/fixme
@@ -82,8 +82,6 @@
 
 * don't show file not found msgs for irrelevant units
 
-* getty doesn't respawn
-
 External:
 
 * sysv functions should color when stdout is tty, not stdin
index d6086ca..318d8a7 100644 (file)
@@ -1500,7 +1500,7 @@ static void service_enter_dead(Service *s, bool success, bool allow_restart) {
                 s->failure = true;
 
         if (allow_restart &&
-            s->allow_restart &&
+            !s->forbid_restart &&
             (s->restart == SERVICE_RESTART_ALWAYS ||
              (s->restart == SERVICE_RESTART_ON_SUCCESS && !s->failure))) {
 
@@ -1511,6 +1511,8 @@ static void service_enter_dead(Service *s, bool success, bool allow_restart) {
         } else
                 service_set_state(s, s->failure ? SERVICE_MAINTENANCE : SERVICE_DEAD);
 
+        s->forbid_restart = false;
+
         return;
 
 fail:
@@ -1932,7 +1934,7 @@ static int service_start(Unit *u) {
 
         s->failure = false;
         s->main_pid_known = false;
-        s->allow_restart = true;
+        s->forbid_restart = false;
 
         service_enter_start_pre(s);
         return 0;
@@ -1945,7 +1947,7 @@ static int service_stop(Unit *u) {
 
         /* This is a user request, so don't do restarts on this
          * shutdown. */
-        s->allow_restart = false;
+        s->forbid_restart = true;
 
         /* Already on it */
         if (s->state == SERVICE_STOP ||
index 3e18d22..725e213 100644 (file)
@@ -115,7 +115,7 @@ struct Service {
         bool failure:1;
         bool main_pid_known:1;
         bool bus_name_good:1;
-        bool allow_restart:1;
+        bool forbid_restart:1;
         bool got_socket_fd:1;
         bool sysv_has_lsb:1;
         bool sysv_enabled:1;