From: David Tardon Date: Sat, 16 Jul 2022 07:49:12 +0000 (+0200) Subject: core: allow to set default timeout for devices X-Git-Tag: v252-rc1~498^2~2 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a0fe19f9f791c05af236265954b1d73e8fcf5468;p=thirdparty%2Fsystemd.git core: allow to set default timeout for devices Fixes: #19879 --- diff --git a/src/core/dbus-manager.c b/src/core/dbus-manager.c index 52521c41464..95defd36a37 100644 --- a/src/core/dbus-manager.c +++ b/src/core/dbus-manager.c @@ -2791,6 +2791,7 @@ const sd_bus_vtable bus_manager_vtable[] = { SD_BUS_PROPERTY("DefaultTimeoutStartUSec", "t", bus_property_get_usec, offsetof(Manager, default_timeout_start_usec), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("DefaultTimeoutStopUSec", "t", bus_property_get_usec, offsetof(Manager, default_timeout_stop_usec), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("DefaultTimeoutAbortUSec", "t", property_get_default_timeout_abort_usec, 0, 0), + SD_BUS_PROPERTY("DefaultDeviceTimeoutUSec", "t", bus_property_get_usec, offsetof(Manager, default_device_timeout_usec), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("DefaultRestartUSec", "t", bus_property_get_usec, offsetof(Manager, default_restart_usec), SD_BUS_VTABLE_PROPERTY_CONST), SD_BUS_PROPERTY("DefaultStartLimitIntervalUSec", "t", bus_property_get_usec, offsetof(Manager, default_start_limit_interval), SD_BUS_VTABLE_PROPERTY_CONST), /* The following two items are obsolete alias */ diff --git a/src/core/device.c b/src/core/device.c index fcde8a420e6..3120a6bc08c 100644 --- a/src/core/device.c +++ b/src/core/device.c @@ -102,7 +102,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. */ - u->job_running_timeout = u->manager->default_timeout_start_usec; + u->job_running_timeout = u->manager->default_device_timeout_usec; u->ignore_on_isolate = true; diff --git a/src/core/main.c b/src/core/main.c index 72d86d3efd0..00d2ee29305 100644 --- a/src/core/main.c +++ b/src/core/main.c @@ -134,6 +134,7 @@ static usec_t arg_default_restart_usec; static usec_t arg_default_timeout_start_usec; static usec_t arg_default_timeout_stop_usec; static usec_t arg_default_timeout_abort_usec; +static usec_t arg_default_device_timeout_usec; static bool arg_default_timeout_abort_set; static usec_t arg_default_start_limit_interval; static unsigned arg_default_start_limit_burst; @@ -627,6 +628,7 @@ static int parse_config_file(void) { { "Manager", "DefaultTimeoutStartSec", config_parse_sec, 0, &arg_default_timeout_start_usec }, { "Manager", "DefaultTimeoutStopSec", config_parse_sec, 0, &arg_default_timeout_stop_usec }, { "Manager", "DefaultTimeoutAbortSec", config_parse_default_timeout_abort, 0, NULL }, + { "Manager", "DefaultDeviceTimeoutSec", config_parse_sec, 0, &arg_default_device_timeout_usec }, { "Manager", "DefaultRestartSec", config_parse_sec, 0, &arg_default_restart_usec }, { "Manager", "DefaultStartLimitInterval", config_parse_sec, 0, &arg_default_start_limit_interval }, /* obsolete alias */ { "Manager", "DefaultStartLimitIntervalSec", config_parse_sec, 0, &arg_default_start_limit_interval }, @@ -716,6 +718,7 @@ static void set_manager_defaults(Manager *m) { m->default_timeout_stop_usec = arg_default_timeout_stop_usec; m->default_timeout_abort_usec = arg_default_timeout_abort_usec; m->default_timeout_abort_set = arg_default_timeout_abort_set; + m->default_device_timeout_usec = arg_default_device_timeout_usec; m->default_restart_usec = arg_default_restart_usec; m->default_start_limit_interval = arg_default_start_limit_interval; m->default_start_limit_burst = arg_default_start_limit_burst; @@ -2387,6 +2390,7 @@ static void reset_arguments(void) { arg_default_timeout_stop_usec = DEFAULT_TIMEOUT_USEC; arg_default_timeout_abort_usec = DEFAULT_TIMEOUT_USEC; arg_default_timeout_abort_set = false; + arg_default_device_timeout_usec = DEFAULT_TIMEOUT_USEC; arg_default_start_limit_interval = DEFAULT_START_LIMIT_INTERVAL; arg_default_start_limit_burst = DEFAULT_START_LIMIT_BURST; arg_runtime_watchdog = 0; diff --git a/src/core/manager.c b/src/core/manager.c index e7e077dcf0e..df06d2a57f4 100644 --- a/src/core/manager.c +++ b/src/core/manager.c @@ -831,6 +831,7 @@ int manager_new(LookupScope scope, ManagerTestRunFlags test_run_flags, Manager * .default_timeout_start_usec = DEFAULT_TIMEOUT_USEC, .default_timeout_stop_usec = DEFAULT_TIMEOUT_USEC, .default_restart_usec = DEFAULT_RESTART_USEC, + .default_device_timeout_usec = DEFAULT_TIMEOUT_USEC, .original_log_level = -1, .original_log_target = _LOG_TARGET_INVALID, diff --git a/src/core/manager.h b/src/core/manager.h index 63cff7989d4..5af69a0b1d7 100644 --- a/src/core/manager.h +++ b/src/core/manager.h @@ -356,6 +356,7 @@ struct Manager { ExecOutput default_std_output, default_std_error; usec_t default_restart_usec, default_timeout_start_usec, default_timeout_stop_usec; + usec_t default_device_timeout_usec; usec_t default_timeout_abort_usec; bool default_timeout_abort_set; diff --git a/src/core/system.conf.in b/src/core/system.conf.in index ae1b47b2ba3..318c0348264 100644 --- a/src/core/system.conf.in +++ b/src/core/system.conf.in @@ -46,6 +46,7 @@ #DefaultTimeoutStartSec=90s #DefaultTimeoutStopSec=90s #DefaultTimeoutAbortSec= +#DefaultDeviceTimeoutSec=90s #DefaultRestartSec=100ms #DefaultStartLimitIntervalSec=10s #DefaultStartLimitBurst=5