shaba/systemd
1
0
Fork 0
mirror of https://github.com/systemd/systemd.git synced 2025-01-06 17:18:12 +03:00
Code

generators: improve logging in generator_write_timeouts() and ignore most errors

Browse Source 
Some generators would ignore the value and warn, possibly logging twice. Other
generators would treat the failure as fatal, even though it probably doesn't
matter too much if drop-in with the timeout is not written.

fstab_filter_options() can only fail with -ENOMEM, which we want to treat as
a fatal error, so use log_error and drop ", ignoring".

Simplify the function to always log uniformly and simplify the callers.
This commit is contained in:
Zbigniew Jędrzejewski-Szmek 2024-06-20 09:35:04 +02:00
parent 526298a8dd
commit 3a0ec74049
5 changed files with 33 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/cryptsetup/cryptsetup-generator.c
View File

@ -518,9 +518,9 @@ static int create_disk(
"After=modprobe@loop.service\n",
u_escaped);
r = generator_write_timeouts(arg_dest, device, name, options, &filtered);
r = generator_write_timeouts(arg_dest, device, options, &filtered);
if (r < 0)
log_warning_errno(r, "Failed to write device timeout drop-in: %m");
return r;
r = generator_write_cryptsetup_service_section(f, name, u, key_file, filtered);
if (r < 0)

6
src/fstab-generator/fstab-generator.c
View File

@ -288,8 +288,7 @@ static int add_swap(
if (r < 0)
return log_error_errno(r, "Failed to write unit file %s: %m", name);
/* use what as where, to have a nicer error message */
r = generator_write_timeouts(arg_dest, what, what, options, NULL);
r = generator_write_timeouts(arg_dest, what, options, NULL);
if (r < 0)
return r;
@ -590,8 +589,7 @@ static int add_mount(
return log_error_errno(r, "Failed to generate unit name: %m");
/* Write timeout dropin and get filtered options */
r = generator_write_timeouts(dest, what, where, opts, &filtered);
r = generator_write_timeouts(dest, what, opts, &filtered);
if (r < 0)
return r;

3
src/hibernate-resume/hibernate-resume-generator.c
View File

@ -76,12 +76,11 @@ static int process_resume(const HibernateInfo *info) {
log_warning_errno(r, "Failed to write device timeout drop-in, ignoring: %m");
r = generator_write_timeouts(arg_dest,
info->device,
info->device,
arg_resume_options ?: arg_root_options,
NULL);
if (r < 0)
log_warning_errno(r, "Failed to write device timeout drop-in, ignoring: %m");
return r;
r = write_drop_in_format(arg_dest, SPECIAL_HIBERNATE_RESUME_SERVICE, 90, "device-dependency",
"# Automatically generated by systemd-hibernate-resume-generator\n\n"

49
src/shared/generator.c
View File

@ -370,31 +370,31 @@ int generator_write_fsck_deps(
int generator_write_timeouts(
const char *dir,
const char *what,
const char *where,
const char *opts,
char **filtered) {
char **ret_filtered) {
/* Configure how long we wait for a device that backs a mount point or a
* swap partition to show up. This is useful to support endless device timeouts
* for devices that show up only after user input, like crypto devices. */
/* Configure how long we wait for a device that backs a mount point or a swap partition to
* show up. This is useful to support endless device timeouts for devices that show up only
* after user input, like crypto devices.
*/
_cleanup_free_ char *node = NULL, *unit = NULL, *timeout = NULL;
usec_t u;
int r;
assert(what);
r = fstab_filter_options(opts, "comment=systemd.device-timeout\0"
"x-systemd.device-timeout\0",
NULL, &timeout, NULL, filtered);
if (r < 0) {
log_warning_errno(r, "Failed to parse fstab options, ignoring: %m");
return 0;
}
NULL, &timeout, NULL, ret_filtered);
if (r < 0)
return log_error_errno(r, "%s: failed to parse fstab options: %m", what);
if (r == 0)
return 0;
r = parse_sec_fix_0(timeout, &u);
if (r < 0) {
log_warning("Failed to parse timeout for %s, ignoring: %s", where, timeout);
log_warning_errno(r, "%s: failed to parse timeout, ignoring: %s", what, timeout);
return 0;
}
@ -402,22 +402,27 @@ int generator_write_timeouts(
if (!node)
return log_oom();
if (!is_device_path(node)) {
log_warning("x-systemd.device-timeout ignored for %s", what);
log_warning("x-systemd.device-timeout ignored for %s.", what);
return 0;
}
r = unit_name_from_path(node, ".device", &unit);
if (r < 0)
return log_error_errno(r, "Failed to make unit name from path: %m");
if (r < 0) {
log_error_errno(r, "%s: failed to make unit name from path '%s', ignoring: %m", what, node);
return 0;
}
return write_drop_in_format(dir, unit, 50, "device-timeout",
"# Automatically generated by %s\n"
"# from supplied options \"%s\"\n\n"
"[Unit]\n"
"JobRunningTimeoutSec=%s",
program_invocation_short_name,
opts,
timeout);
r = write_drop_in_format(dir, unit, 50, "device-timeout",
"# Automatically generated by %s\n"
"# from supplied options \"%s\"\n\n"
"[Unit]\n"
"JobRunningTimeoutSec=%s",
program_invocation_short_name,
opts,
timeout);
if (r < 0)
log_warning_errno(r, "%s: failed to write drop-in with device timeout, ignoring: %m", what);
return 0;
}
int generator_write_device_deps(

3
src/shared/generator.h
View File

@ -28,9 +28,8 @@ int generator_write_fsck_deps(
int generator_write_timeouts(
const char *dir,
const char *what,
const char *where,
const char *opts,
char **filtered);
char **ret_filtered);
int generator_write_blockdev_dependency(
FILE *f,