From a9b837aa34a2d0bff1687427c66bed3b74cf0fed Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Tue, 6 Jun 2023 09:29:51 +0200 Subject: [PATCH] generators: change TimeoutSec=0 to TimeoutSec=infinity With these settings we intend to turn off timeouts for possibly interactive/slow commands. The officially documented way to turn off the time-outs is to setting them to infinity. So far we set them to zero here though. This lead to some confusiong, for example #18224. Let's fix this by uniformly spelling out TimeoutSec=infinity. This doesn't change behaviour. It just makes our generated files match what we document, without relying on historic compat support. Fixes: #18224 --- src/cryptsetup/cryptsetup-generator.c | 3 ++- src/gpt-auto-generator/gpt-auto-generator.c | 2 +- src/hibernate-resume/hibernate-resume-generator.c | 3 ++- src/integritysetup/integritysetup-generator.c | 2 +- src/shared/generator.c | 8 ++++---- .../test-12-dev-sdx.expected/systemd-fsck-root.service | 2 +- .../test-13-label.expected/systemd-fsck-root.service | 2 +- .../test-14-uuid.expected/systemd-fsck-root.service | 2 +- .../test-15-partuuid.expected/systemd-fsck-root.service | 2 +- .../systemd-fsck-root.service | 2 +- .../systemd-fsck-root.service | 2 +- .../systemd-makefs@dev-sdx12.service | 2 +- .../systemd-mkswap@dev-sdy2.service | 2 +- .../systemd-mkswap@dev-sdy3.service | 2 +- 14 files changed, 19 insertions(+), 17 deletions(-) diff --git a/src/cryptsetup/cryptsetup-generator.c b/src/cryptsetup/cryptsetup-generator.c index a22d9323421..702b2615e29 100644 --- a/src/cryptsetup/cryptsetup-generator.c +++ b/src/cryptsetup/cryptsetup-generator.c @@ -562,7 +562,8 @@ static int create_disk( if (!noauto && !nofail) { r = write_drop_in(arg_dest, dmname, 40, "device-timeout", "# Automatically generated by systemd-cryptsetup-generator\n\n" - "[Unit]\nJobTimeoutSec=0"); + "[Unit]\n" + "JobTimeoutSec=infinity\n"); if (r < 0) log_warning_errno(r, "Failed to write device timeout drop-in: %m"); } diff --git a/src/gpt-auto-generator/gpt-auto-generator.c b/src/gpt-auto-generator/gpt-auto-generator.c index 8036724034f..77f51c42450 100644 --- a/src/gpt-auto-generator/gpt-auto-generator.c +++ b/src/gpt-auto-generator/gpt-auto-generator.c @@ -142,7 +142,7 @@ static int add_cryptsetup( r = write_drop_in_format(arg_dest, dmname, 50, "job-timeout", "# Automatically generated by systemd-gpt-auto-generator\n\n" "[Unit]\n" - "JobTimeoutSec=0"); /* the binary handles timeouts anyway */ + "JobTimeoutSec=infinity"); /* the binary handles timeouts anyway */ if (r < 0) log_warning_errno(r, "Failed to write device timeout drop-in, ignoring: %m"); diff --git a/src/hibernate-resume/hibernate-resume-generator.c b/src/hibernate-resume/hibernate-resume-generator.c index be52c21d00d..1bcf9d69df0 100644 --- a/src/hibernate-resume/hibernate-resume-generator.c +++ b/src/hibernate-resume/hibernate-resume-generator.c @@ -95,7 +95,8 @@ static int process_resume(void) { r = write_drop_in(arg_dest, device_unit, 40, "device-timeout", "# Automatically generated by systemd-hibernate-resume-generator\n\n" - "[Unit]\nJobTimeoutSec=0"); + "[Unit]\n" + "JobTimeoutSec=infinity\n"); if (r < 0) log_warning_errno(r, "Failed to write device timeout drop-in: %m"); diff --git a/src/integritysetup/integritysetup-generator.c b/src/integritysetup/integritysetup-generator.c index 15f508902d3..5df6d81a844 100644 --- a/src/integritysetup/integritysetup-generator.c +++ b/src/integritysetup/integritysetup-generator.c @@ -100,7 +100,7 @@ static int create_disk( "[Service]\n" "Type=oneshot\n" "RemainAfterExit=yes\n" - "TimeoutSec=0\n" + "TimeoutSec=infinity\n" "ExecStart=" ROOTLIBEXECDIR "/systemd-integritysetup attach '%s' '%s' '%s' '%s'\n" "ExecStop=" ROOTLIBEXECDIR "/systemd-integritysetup detach '%s'\n", name_escaped, device, empty_to_dash(key_file_escaped), empty_to_dash(options), diff --git a/src/shared/generator.c b/src/shared/generator.c index 49c5fe5c05b..44ed3199222 100644 --- a/src/shared/generator.c +++ b/src/shared/generator.c @@ -235,7 +235,7 @@ static int write_fsck_sysroot_service( "Type=oneshot\n" "RemainAfterExit=yes\n" "ExecStart=" SYSTEMD_FSCK_PATH " %7$s\n" - "TimeoutSec=0\n", + "TimeoutSec=infinity\n", program_invocation_short_name, escaped, unit, @@ -530,7 +530,7 @@ int generator_hook_up_mkswap( "Type=oneshot\n" "RemainAfterExit=yes\n" "ExecStart="SYSTEMD_MAKEFS_PATH " swap %s\n" - "TimeoutSec=0\n", + "TimeoutSec=infinity\n", program_invocation_short_name, where_unit, escaped); @@ -619,7 +619,7 @@ int generator_hook_up_mkfs( "Type=oneshot\n" "RemainAfterExit=yes\n" "ExecStart="SYSTEMD_MAKEFS_PATH " %s %s\n" - "TimeoutSec=0\n", + "TimeoutSec=infinity\n", program_invocation_short_name, fsck_unit, where_unit, @@ -801,7 +801,7 @@ int generator_write_cryptsetup_service_section( "[Service]\n" "Type=oneshot\n" "RemainAfterExit=yes\n" - "TimeoutSec=0\n" /* The binary handles timeouts on its own */ + "TimeoutSec=infinity\n" /* The binary handles timeouts on its own */ "KeyringMode=shared\n" /* Make sure we can share cached keys among instances */ "OOMScoreAdjust=500\n" /* Unlocking can allocate a lot of memory if Argon2 is used */ "ExecStart=" SYSTEMD_CRYPTSETUP_PATH " attach '%s' '%s' '%s' '%s'\n" diff --git a/test/test-fstab-generator/test-12-dev-sdx.expected/systemd-fsck-root.service b/test/test-fstab-generator/test-12-dev-sdx.expected/systemd-fsck-root.service index 95d943b87a0..147348899d1 100644 --- a/test/test-fstab-generator/test-12-dev-sdx.expected/systemd-fsck-root.service +++ b/test/test-fstab-generator/test-12-dev-sdx.expected/systemd-fsck-root.service @@ -14,4 +14,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-fsck /dev/sdx1 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-13-label.expected/systemd-fsck-root.service b/test/test-fstab-generator/test-13-label.expected/systemd-fsck-root.service index d6c59ff6089..85c1936bce1 100644 --- a/test/test-fstab-generator/test-13-label.expected/systemd-fsck-root.service +++ b/test/test-fstab-generator/test-13-label.expected/systemd-fsck-root.service @@ -14,4 +14,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-fsck /dev/disk/by-label/Root -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-14-uuid.expected/systemd-fsck-root.service b/test/test-fstab-generator/test-14-uuid.expected/systemd-fsck-root.service index cd9583c4dd0..1c7eaea1035 100644 --- a/test/test-fstab-generator/test-14-uuid.expected/systemd-fsck-root.service +++ b/test/test-fstab-generator/test-14-uuid.expected/systemd-fsck-root.service @@ -14,4 +14,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-fsck /dev/disk/by-uuid/3f5ad593-4546-4a94-a374-bcfb68aa11f7 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-15-partuuid.expected/systemd-fsck-root.service b/test/test-fstab-generator/test-15-partuuid.expected/systemd-fsck-root.service index 650ed8070a6..ab27bfd79cc 100644 --- a/test/test-fstab-generator/test-15-partuuid.expected/systemd-fsck-root.service +++ b/test/test-fstab-generator/test-15-partuuid.expected/systemd-fsck-root.service @@ -14,4 +14,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-fsck /dev/disk/by-partuuid/3f5ad593-4546-4a94-a374-bcfb68aa11f7 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-17-initrd-sysroot.fstab.expected/systemd-fsck-root.service b/test/test-fstab-generator/test-17-initrd-sysroot.fstab.expected/systemd-fsck-root.service index 95d943b87a0..147348899d1 100644 --- a/test/test-fstab-generator/test-17-initrd-sysroot.fstab.expected/systemd-fsck-root.service +++ b/test/test-fstab-generator/test-17-initrd-sysroot.fstab.expected/systemd-fsck-root.service @@ -14,4 +14,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-fsck /dev/sdx1 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-18-options.fstab.expected/systemd-fsck-root.service b/test/test-fstab-generator/test-18-options.fstab.expected/systemd-fsck-root.service index 95d943b87a0..147348899d1 100644 --- a/test/test-fstab-generator/test-18-options.fstab.expected/systemd-fsck-root.service +++ b/test/test-fstab-generator/test-18-options.fstab.expected/systemd-fsck-root.service @@ -14,4 +14,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-fsck /dev/sdx1 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-18-options.fstab.expected/systemd-makefs@dev-sdx12.service b/test/test-fstab-generator/test-18-options.fstab.expected/systemd-makefs@dev-sdx12.service index 303c1ee680e..4670c23e27d 100644 --- a/test/test-fstab-generator/test-18-options.fstab.expected/systemd-makefs@dev-sdx12.service +++ b/test/test-fstab-generator/test-18-options.fstab.expected/systemd-makefs@dev-sdx12.service @@ -15,4 +15,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-makefs ext4 /dev/sdx12 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy2.service b/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy2.service index 0911f03f623..0b0e7270bab 100644 --- a/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy2.service +++ b/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy2.service @@ -15,4 +15,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-makefs swap /dev/sdy2 -TimeoutSec=0 +TimeoutSec=infinity diff --git a/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy3.service b/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy3.service index 6201fec86be..1164c99476c 100644 --- a/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy3.service +++ b/test/test-fstab-generator/test-20-swap-from-cmdline.expected/systemd-mkswap@dev-sdy3.service @@ -15,4 +15,4 @@ Before=shutdown.target Type=oneshot RemainAfterExit=yes ExecStart=/usr/lib/systemd/systemd-makefs swap /dev/sdy3 -TimeoutSec=0 +TimeoutSec=infinity -- 2.39.2