2013-04-13 13:09:53 +02:00
|
|
|
From: Frederic Crozat <fcrozat@suse.com>
|
|
|
|
Date: Wed, 7 Dec 2011 15:15:07 +0000
|
|
|
|
Subject: remain_after_exit initscript heuristic and add new LSB headers
|
|
|
|
|
|
|
|
Add remain_after_exit heuristic for initscripts and add LSB headers
|
|
|
|
PIDFile: and X-Systemd-RemainAfterExit to control it.
|
|
|
|
|
|
|
|
(bnc#721426) (bnc#727771)
|
|
|
|
---
|
|
|
|
src/core/service.c | 34 ++++++++++++++++++++++++++++++++--
|
|
|
|
src/core/service.h | 1 +
|
|
|
|
2 files changed, 33 insertions(+), 2 deletions(-)
|
|
|
|
|
|
|
|
diff --git a/src/core/service.c b/src/core/service.c
|
2013-04-19 22:31:17 +02:00
|
|
|
index 5118b06..c13ff35 100644
|
2013-04-13 13:09:53 +02:00
|
|
|
--- a/src/core/service.c
|
|
|
|
+++ b/src/core/service.c
|
|
|
|
@@ -135,6 +135,7 @@ static void service_init(Unit *u) {
|
2011-12-09 15:44:59 +01:00
|
|
|
#ifdef HAVE_SYSV_COMPAT
|
|
|
|
s->sysv_start_priority = -1;
|
|
|
|
s->sysv_start_priority_from_rcnd = -1;
|
|
|
|
+ s->sysv_remain_after_exit_heuristic = true;
|
|
|
|
#endif
|
|
|
|
s->socket_fd = -1;
|
|
|
|
s->guess_main_pid = true;
|
2013-04-13 13:09:53 +02:00
|
|
|
@@ -875,6 +876,34 @@ static int service_load_sysv_path(Service *s, const char *path) {
|
2012-05-24 14:06:55 +02:00
|
|
|
free(short_description);
|
|
|
|
short_description = d;
|
2011-12-09 15:44:59 +01:00
|
|
|
|
|
|
|
+ } else if (startswith_no_case(t, "PIDFile:")) {
|
|
|
|
+ char *fn;
|
|
|
|
+
|
|
|
|
+ state = LSB;
|
|
|
|
+
|
|
|
|
+ fn = strstrip(t+8);
|
|
|
|
+ if (!path_is_absolute(fn)) {
|
|
|
|
+ log_warning("[%s:%u] PID file not absolute. Ignoring.", path, line);
|
|
|
|
+ continue;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ if (!(fn = strdup(fn))) {
|
|
|
|
+ r = -ENOMEM;
|
|
|
|
+ goto finish;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ free(s->pid_file);
|
|
|
|
+ s->pid_file = fn;
|
|
|
|
+ s->sysv_remain_after_exit_heuristic = false;
|
2012-02-09 18:04:56 +01:00
|
|
|
+ s->remain_after_exit = false;
|
2011-12-09 15:44:59 +01:00
|
|
|
+ } else if (startswith_no_case(t, "X-Systemd-RemainAfterExit:")) {
|
|
|
|
+ char *j;
|
|
|
|
+
|
|
|
|
+ state = LSB;
|
2012-02-09 18:04:56 +01:00
|
|
|
+ if ((j = strstrip(t+26)) && *j) {
|
|
|
|
+ s->remain_after_exit = parse_boolean(j);
|
|
|
|
+ s->sysv_remain_after_exit_heuristic = false;
|
|
|
|
+ }
|
2012-05-24 14:06:55 +02:00
|
|
|
} else if (state == LSB_DESCRIPTION) {
|
|
|
|
|
|
|
|
if (startswith(l, "#\t") || startswith(l, "# ")) {
|
2013-04-13 13:09:53 +02:00
|
|
|
@@ -925,7 +954,8 @@ static int service_load_sysv_path(Service *s, const char *path) {
|
2012-02-09 18:04:56 +01:00
|
|
|
|
|
|
|
/* Special setting for all SysV services */
|
|
|
|
s->type = SERVICE_FORKING;
|
|
|
|
- s->remain_after_exit = !s->pid_file;
|
|
|
|
+ if (s->sysv_remain_after_exit_heuristic)
|
|
|
|
+ s->remain_after_exit = !s->pid_file;
|
|
|
|
s->guess_main_pid = false;
|
|
|
|
s->restart = SERVICE_RESTART_NO;
|
|
|
|
s->exec_context.ignore_sigpipe = false;
|
2013-04-13 13:09:53 +02:00
|
|
|
@@ -2074,7 +2104,7 @@ static void service_enter_running(Service *s, ServiceResult f) {
|
2011-12-09 15:44:59 +01:00
|
|
|
if ((main_pid_ok > 0 || (main_pid_ok < 0 && cgroup_ok != 0)) &&
|
|
|
|
(s->bus_name_good || s->type != SERVICE_DBUS)) {
|
|
|
|
#ifdef HAVE_SYSV_COMPAT
|
|
|
|
- if (s->sysv_enabled && !s->pid_file)
|
|
|
|
+ if (s->sysv_enabled && !s->pid_file && s->sysv_remain_after_exit_heuristic)
|
|
|
|
s->remain_after_exit = false;
|
|
|
|
#endif
|
|
|
|
service_set_state(s, SERVICE_RUNNING);
|
2013-04-13 13:09:53 +02:00
|
|
|
diff --git a/src/core/service.h b/src/core/service.h
|
|
|
|
index d1e53bf..4cd1a79 100644
|
|
|
|
--- a/src/core/service.h
|
|
|
|
+++ b/src/core/service.h
|
2012-10-26 14:34:44 +02:00
|
|
|
@@ -176,6 +176,7 @@ struct Service {
|
|
|
|
bool is_sysv:1;
|
2011-12-09 15:44:59 +01:00
|
|
|
bool sysv_has_lsb:1;
|
|
|
|
bool sysv_enabled:1;
|
|
|
|
+ bool sysv_remain_after_exit_heuristic:1;
|
|
|
|
int sysv_start_priority_from_rcnd;
|
|
|
|
int sysv_start_priority;
|
|
|
|
|