From cd1c485f2fc5b23c7cdccb0dd852c7228fc79868 Mon Sep 17 00:00:00 2001 From: Oleksii Shevchuk Date: Mon, 4 Nov 2013 18:47:43 +0200 Subject: [PATCH] Configurable Timeouts/Restarts default values https://bugs.freedesktop.org/show_bug.cgi?id=71132 Patch adds DefaultTimeoutStartSec, DefaultTimeoutStopSec, DefaultRestartSec configuration options to manager configuration file. --- man/systemd-system.conf.xml | 24 +++++++++++++++++++++++- man/systemd.mount.xml | 4 ++-- man/systemd.service.xml | 6 ++++-- man/systemd.socket.xml | 4 ++-- man/systemd.swap.xml | 4 ++-- src/core/device.c | 2 +- src/core/main.c | 9 +++++++++ src/core/manager.h | 3 +++ src/core/mount.c | 2 +- src/core/scope.c | 2 +- src/core/service.c | 6 +++--- src/core/socket.c | 2 +- src/core/swap.c | 2 +- src/core/system.conf | 3 +++ src/core/user.conf | 3 +++ 15 files changed, 59 insertions(+), 17 deletions(-) diff --git a/man/systemd-system.conf.xml b/man/systemd-system.conf.xml index e8cf8a9..c1f2648 100644 --- a/man/systemd-system.conf.xml +++ b/man/systemd-system.conf.xml @@ -61,7 +61,6 @@ otherwise user.conf. These configuration files contain a few settings controlling basic manager operations. - @@ -95,6 +94,29 @@ + DefaultTimeoutStartSec= + DefaultTimeoutStopSec= + DefaultRestartSec= + + Configures the default + time-outs for starting and stopping of + units, as well as the default time to + sleep between automatic restarts of a + units, as configured per-unit in + TimeoutStartSec=, + TimeoutStopSec= and + RestartSec= (for + service units see + systemd.service5 + for details on the per-unit + settings). For non-service units + DefaultTimeoutStartSec= + sets the default + TimeoutSec= value. + + + + CPUAffinity= Configures the initial diff --git a/man/systemd.mount.xml b/man/systemd.mount.xml index 48af1ca..71a5736 100644 --- a/man/systemd.mount.xml +++ b/man/systemd.mount.xml @@ -260,8 +260,8 @@ Takes a unit-less value in seconds, or a time span value such as "5min 20s". Pass 0 to disable the timeout - logic. Defaults to - 90s. + logic. Default value is setted up in manager configuration + file via DefaultTimeoutStart=. diff --git a/man/systemd.service.xml b/man/systemd.service.xml index 5e1ddf7..df04048 100644 --- a/man/systemd.service.xml +++ b/man/systemd.service.xml @@ -524,7 +524,8 @@ Takes a unit-less value in seconds, or a time span value such as "5min 20s". Pass 0 to disable the timeout - logic. Defaults to 90s, except when + logic. Defaults to TimeoutStartSec= in + manager configuration file, except when Type=oneshot is used in which case the timeout is disabled by default. @@ -545,7 +546,8 @@ Takes a unit-less value in seconds, or a time span value such as "5min 20s". Pass 0 to disable the timeout - logic. Defaults to 90s. + logic. Defaults to TimeoutStartSec= in + manager configuration file. diff --git a/man/systemd.socket.xml b/man/systemd.socket.xml index 8c88d9f..1c78562 100644 --- a/man/systemd.socket.xml +++ b/man/systemd.socket.xml @@ -679,8 +679,8 @@ Takes a unit-less value in seconds, or a time span value such as "5min 20s". Pass 0 to disable the timeout - logic. Defaults to - 90s. + logic. Defaults to TimeoutStartSec= in + manager configuration file. diff --git a/man/systemd.swap.xml b/man/systemd.swap.xml index 813ae6c..13f6c84 100644 --- a/man/systemd.swap.xml +++ b/man/systemd.swap.xml @@ -186,8 +186,8 @@ Takes a unit-less value in seconds, or a time span value such as "5min 20s". Pass 0 to disable the timeout - logic. Defaults to - 90s. + logic. Defaults to TimeoutStartSec= in + manager configuration file. diff --git a/src/core/device.c b/src/core/device.c index 9fca82a..5397bd6 100644 --- a/src/core/device.c +++ b/src/core/device.c @@ -70,7 +70,7 @@ static void device_init(Unit *u) { * indefinitely for plugged in devices, something which cannot * happen for the other units since their operations time out * anyway. */ - UNIT(d)->job_timeout = DEFAULT_TIMEOUT_USEC; + UNIT(d)->job_timeout = u->manager->default_timeout_start_usec; UNIT(d)->ignore_on_isolate = true; UNIT(d)->ignore_on_snapshot = true; diff --git a/src/core/main.c b/src/core/main.c index fe291f8..937994c 100644 --- a/src/core/main.c +++ b/src/core/main.c @@ -90,6 +90,9 @@ static bool arg_switched_root = false; static char ***arg_join_controllers = NULL; static ExecOutput arg_default_std_output = EXEC_OUTPUT_JOURNAL; static ExecOutput arg_default_std_error = EXEC_OUTPUT_INHERIT; +static usec_t arg_default_restart_usec = DEFAULT_RESTART_USEC; +static usec_t arg_default_timeout_start_usec = DEFAULT_TIMEOUT_USEC; +static usec_t arg_default_timeout_stop_usec = DEFAULT_TIMEOUT_USEC; static usec_t arg_runtime_watchdog = 0; static usec_t arg_shutdown_watchdog = 10 * USEC_PER_MINUTE; static char **arg_default_environment = NULL; @@ -636,6 +639,9 @@ static int parse_config_file(void) { { "Manager", "CPUAffinity", config_parse_cpu_affinity2, 0, NULL }, { "Manager", "DefaultStandardOutput", config_parse_output, 0, &arg_default_std_output }, { "Manager", "DefaultStandardError", config_parse_output, 0, &arg_default_std_error }, + { "Manager", "DefaultTimeoutStartSec", config_parse_sec, 0, &arg_default_timeout_start_usec }, + { "Manager", "DefaultTimeoutStopSec", config_parse_sec, 0, &arg_default_timeout_stop_usec }, + { "Manager", "DefaultRestartSec", config_parse_sec, 0, &arg_default_restart_usec }, { "Manager", "JoinControllers", config_parse_join_controllers, 0, &arg_join_controllers }, { "Manager", "RuntimeWatchdogSec", config_parse_sec, 0, &arg_runtime_watchdog }, { "Manager", "ShutdownWatchdogSec", config_parse_sec, 0, &arg_shutdown_watchdog }, @@ -1542,6 +1548,9 @@ int main(int argc, char *argv[]) { m->confirm_spawn = arg_confirm_spawn; m->default_std_output = arg_default_std_output; m->default_std_error = arg_default_std_error; + m->default_restart_usec = arg_default_restart_usec; + m->default_timeout_start_usec = arg_default_timeout_start_usec; + m->default_timeout_stop_usec = arg_default_timeout_stop_usec; m->runtime_watchdog = arg_runtime_watchdog; m->shutdown_watchdog = arg_shutdown_watchdog; m->userspace_timestamp = userspace_timestamp; diff --git a/src/core/manager.h b/src/core/manager.h index a3049b5..e74c609 100644 --- a/src/core/manager.h +++ b/src/core/manager.h @@ -230,6 +230,9 @@ struct Manager { ExecOutput default_std_output, default_std_error; + usec_t default_restart_usec, default_timeout_start_usec, + default_timeout_stop_usec; + struct rlimit *rlimit[RLIMIT_NLIMITS]; /* non-zero if we are reloading or reexecuting, */ diff --git a/src/core/mount.c b/src/core/mount.c index 70cd372..c0445a6 100644 --- a/src/core/mount.c +++ b/src/core/mount.c @@ -131,7 +131,7 @@ static void mount_init(Unit *u) { assert(u); assert(u->load_state == UNIT_STUB); - m->timeout_usec = DEFAULT_TIMEOUT_USEC; + m->timeout_usec = u->manager->default_timeout_start_usec; m->directory_mode = 0755; exec_context_init(&m->exec_context); diff --git a/src/core/scope.c b/src/core/scope.c index 50e5dba..41da3b9 100644 --- a/src/core/scope.c +++ b/src/core/scope.c @@ -46,7 +46,7 @@ static void scope_init(Unit *u) { assert(u); assert(u->load_state == UNIT_STUB); - s->timeout_stop_usec = DEFAULT_TIMEOUT_USEC; + s->timeout_stop_usec = u->manager->default_timeout_stop_usec; watch_init(&s->timer_watch); diff --git a/src/core/service.c b/src/core/service.c index 96ed2d3..e81aa1f 100644 --- a/src/core/service.c +++ b/src/core/service.c @@ -124,9 +124,9 @@ static void service_init(Unit *u) { assert(u); assert(u->load_state == UNIT_STUB); - s->timeout_start_usec = DEFAULT_TIMEOUT_USEC; - s->timeout_stop_usec = DEFAULT_TIMEOUT_USEC; - s->restart_usec = DEFAULT_RESTART_USEC; + s->timeout_start_usec = u->manager->default_timeout_start_usec; + s->timeout_stop_usec = u->manager->default_timeout_stop_usec; + s->restart_usec = u->manager->default_restart_usec; s->type = _SERVICE_TYPE_INVALID; watch_init(&s->watchdog_watch); diff --git a/src/core/socket.c b/src/core/socket.c index 6c0ac1a..d368f7e 100644 --- a/src/core/socket.c +++ b/src/core/socket.c @@ -73,7 +73,7 @@ static void socket_init(Unit *u) { assert(u->load_state == UNIT_STUB); s->backlog = SOMAXCONN; - s->timeout_usec = DEFAULT_TIMEOUT_USEC; + s->timeout_usec = u->manager->default_timeout_start_usec; s->directory_mode = 0755; s->socket_mode = 0666; diff --git a/src/core/swap.c b/src/core/swap.c index a68ab7c..147f710 100644 --- a/src/core/swap.c +++ b/src/core/swap.c @@ -86,7 +86,7 @@ static void swap_init(Unit *u) { assert(s); assert(UNIT(s)->load_state == UNIT_STUB); - s->timeout_usec = DEFAULT_TIMEOUT_USEC; + s->timeout_usec = u->manager->default_timeout_start_usec; exec_context_init(&s->exec_context); s->exec_context.std_output = u->manager->default_std_output; diff --git a/src/core/system.conf b/src/core/system.conf index 7b03c87..3c6cc03 100644 --- a/src/core/system.conf +++ b/src/core/system.conf @@ -24,6 +24,9 @@ #ShutdownWatchdogSec=10min #CapabilityBoundingSet= #TimerSlackNSec= +#DefaultTimeoutStartSec=90s +#DefaultTimeoutStopSec=90s +#DefaultRestartSec=100ms #DefaultEnvironment= #DefaultLimitCPU= #DefaultLimitFSIZE= diff --git a/src/core/user.conf b/src/core/user.conf index 4a0129a..b030701 100644 --- a/src/core/user.conf +++ b/src/core/user.conf @@ -14,3 +14,6 @@ #LogLocation=no #DefaultStandardOutput=inherit #DefaultStandardError=inherit +#DefaultTimeoutStartSec=90s +#DefaultTimeoutStopSec=90s +#DefaultRestartSec=100ms