service: kill processes with SIGKILL on watchdog failure

Just calling service_enter_dead() does not kill any processes.
As a result, the old process may still be running when the new one is
started.

After a watchdog failure the service is in an undefined state.
Using the normal shutdown mechanism makes no sense. Instead all processes
are just killed and the service can try to restart.

diff --git a/src/core/service.c b/src/core/service.c
index 3617c247117d7c88f23b266cf0604fd9cda204d7..e110a41daebbdcbcd01e8550ec98a4dca750dd3c 100644 (file)
--- a/src/core/service.c
+++ b/src/core/service.c
@@ -235,7 +235,7 @@ static void service_stop_watchdog(Service *s) {
         s->watchdog_timestamp.monotonic = 0;
 }
 
         s->watchdog_timestamp.monotonic = 0;
 }
 

-static void service_enter_dead(Service *s, ServiceResult f, bool allow_restart);
+static void service_enter_signal(Service *s, ServiceState state, ServiceResult f);

 
 static void service_handle_watchdog(Service *s) {
         usec_t offset;
 
 static void service_handle_watchdog(Service *s) {
         usec_t offset;


@@ -249,7 +249,7 @@ static void service_handle_watchdog(Service *s) {
         offset = now(CLOCK_MONOTONIC) - s->watchdog_timestamp.monotonic;
         if (offset >= s->watchdog_usec) {
                 log_error_unit(UNIT(s)->id, "%s watchdog timeout!", UNIT(s)->id);
         offset = now(CLOCK_MONOTONIC) - s->watchdog_timestamp.monotonic;
         if (offset >= s->watchdog_usec) {
                 log_error_unit(UNIT(s)->id, "%s watchdog timeout!", UNIT(s)->id);

-                service_enter_dead(s, SERVICE_FAILURE_WATCHDOG, true);
+                service_enter_signal(s, SERVICE_FINAL_SIGKILL, SERVICE_FAILURE_WATCHDOG);

                 return;
         }
 
                 return;
         }
 


@@ -1939,8 +1939,6 @@ fail:
         service_enter_dead(s, SERVICE_FAILURE_RESOURCES, false);
 }
 
         service_enter_dead(s, SERVICE_FAILURE_RESOURCES, false);
 }
 

-static void service_enter_signal(Service *s, ServiceState state, ServiceResult f);
-

 static void service_enter_stop_post(Service *s, ServiceResult f) {
         int r;
         assert(s);
 static void service_enter_stop_post(Service *s, ServiceResult f) {
         int r;
         assert(s);