Zbigniew Jędrzejewski-Szmek a48f80
From 754ba6bd338a15a698a3152e8fbecbc20d09afec Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek a48f80
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
Zbigniew Jędrzejewski-Szmek a48f80
Date: Wed, 6 May 2015 01:09:53 -0400
Zbigniew Jędrzejewski-Szmek a48f80
Subject: [PATCH] Generate systemd-fsck-root.service in the initramfs
Zbigniew Jędrzejewski-Szmek a48f80
Zbigniew Jędrzejewski-Szmek a48f80
In the initrafms, generate a systemd-fsck-root.service to replace
Zbigniew Jędrzejewski-Szmek a48f80
systemd-fsck@<sysroot-device>.service. This way, after we transition
Zbigniew Jędrzejewski-Szmek a48f80
to the real root, systemd-fsck-root.service is marked as already done.
Zbigniew Jędrzejewski-Szmek a48f80
Zbigniew Jędrzejewski-Szmek a48f80
This introduces an unnecessary synchronization point, because
Zbigniew Jędrzejewski-Szmek a48f80
systemd-fsck@* is ordered after systemd-fsck-root also in the
Zbigniew Jędrzejewski-Szmek a48f80
initramfs. In practice this shouldn't be a problem.
Zbigniew Jędrzejewski-Szmek a48f80
Zbigniew Jędrzejewski-Szmek a48f80
https://bugzilla.redhat.com/show_bug.cgi?id=1201979
Zbigniew Jędrzejewski-Szmek a48f80
Zbigniew Jędrzejewski-Szmek a48f80
C.f. 956eaf2b8d6c9999024705ddadc7393bc707de02.
Zbigniew Jędrzejewski-Szmek a48f80
Zbigniew Jędrzejewski-Szmek a48f80
(cherry picked from commit 4dda4e637e4c17a14db6cd265f36f5e8a5050367)
Zbigniew Jędrzejewski-Szmek a48f80
---
Zbigniew Jędrzejewski-Szmek a48f80
 src/shared/generator.c | 63 ++++++++++++++++++++++++++++++++++++++++++++++----
Zbigniew Jędrzejewski-Szmek a48f80
 1 file changed, 59 insertions(+), 4 deletions(-)
Zbigniew Jędrzejewski-Szmek a48f80
Zbigniew Jędrzejewski-Szmek a48f80
diff --git a/src/shared/generator.c b/src/shared/generator.c
Zbigniew Jędrzejewski-Szmek a48f80
index cd37812f86..148a0b077b 100644
Zbigniew Jędrzejewski-Szmek a48f80
--- a/src/shared/generator.c
Zbigniew Jędrzejewski-Szmek a48f80
+++ b/src/shared/generator.c
Zbigniew Jędrzejewski-Szmek a48f80
@@ -29,8 +29,51 @@
Zbigniew Jędrzejewski-Szmek a48f80
 #include "generator.h"
Zbigniew Jędrzejewski-Szmek a48f80
 #include "path-util.h"
Zbigniew Jędrzejewski-Szmek a48f80
 #include "fstab-util.h"
Zbigniew Jędrzejewski-Szmek a48f80
+#include "fileio.h"
Zbigniew Jędrzejewski-Szmek a48f80
 #include "dropin.h"
Zbigniew Jędrzejewski-Szmek a48f80
 
Zbigniew Jędrzejewski-Szmek a48f80
+static int write_fsck_sysroot_service(const char *dir, const char *what) {
Zbigniew Jędrzejewski-Szmek a48f80
+        const char *unit;
Zbigniew Jędrzejewski-Szmek a48f80
+        _cleanup_free_ char *device = NULL;
Zbigniew Jędrzejewski-Szmek a48f80
+        _cleanup_fclose_ FILE *f = NULL;
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+        unit = strjoina(dir, "/systemd-fsck-root.service");
Zbigniew Jędrzejewski-Szmek a48f80
+        log_debug("Creating %s", unit);
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+        device = unit_name_from_path(what, ".device");
Zbigniew Jędrzejewski-Szmek a48f80
+        if (!device)
Zbigniew Jędrzejewski-Szmek a48f80
+                return log_oom();
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+        f = fopen(unit, "wxe");
Zbigniew Jędrzejewski-Szmek a48f80
+        if (!f)
Zbigniew Jędrzejewski-Szmek a48f80
+                return log_error_errno(errno, "Failed to create unit file %s: %m", unit);
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+        fprintf(f,
Zbigniew Jędrzejewski-Szmek a48f80
+                "# Automatically generated by %1$s\n\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "[Unit]\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "Documentation=man:systemd-fsck-root.service(8)\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "Description=File System Check on %2$s\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "DefaultDependencies=no\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "BindsTo=%3$s\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "After=%3$s\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "Before=shutdown.target\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "[Service]\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "Type=oneshot\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "RemainAfterExit=yes\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "ExecStart=/usr/lib/systemd/systemd-fsck %2$s\n"
Zbigniew Jędrzejewski-Szmek a48f80
+                "TimeoutSec=0\n",
Zbigniew Jędrzejewski-Szmek a48f80
+                program_invocation_short_name,
Zbigniew Jędrzejewski-Szmek a48f80
+                what,
Zbigniew Jędrzejewski-Szmek a48f80
+                device);
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+        fflush(f);
Zbigniew Jędrzejewski-Szmek a48f80
+        if (ferror(f))
Zbigniew Jędrzejewski-Szmek a48f80
+                return log_error_errno(errno, "Failed to write unit file %s: %m", unit);
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+        return 0;
Zbigniew Jędrzejewski-Szmek a48f80
+}
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
 int generator_write_fsck_deps(
Zbigniew Jędrzejewski-Szmek a48f80
                 FILE *f,
Zbigniew Jędrzejewski-Szmek a48f80
                 const char *dir,
Zbigniew Jędrzejewski-Szmek a48f80
@@ -69,11 +112,23 @@ int generator_write_fsck_deps(
Zbigniew Jędrzejewski-Szmek a48f80
                         return log_error_errno(errno, "Failed to create symlink %s: %m", lnk);
Zbigniew Jędrzejewski-Szmek a48f80
 
Zbigniew Jędrzejewski-Szmek a48f80
         } else {
Zbigniew Jędrzejewski-Szmek a48f80
-                _cleanup_free_ char *fsck = NULL;
Zbigniew Jędrzejewski-Szmek a48f80
+                _cleanup_free_ char *_fsck = NULL;
Zbigniew Jędrzejewski-Szmek a48f80
+                const char *fsck;
Zbigniew Jędrzejewski-Szmek a48f80
+                int r;
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+                if (in_initrd() && path_equal(where, "/sysroot")) {
Zbigniew Jędrzejewski-Szmek a48f80
+                        r = write_fsck_sysroot_service(dir, what);
Zbigniew Jędrzejewski-Szmek a48f80
+                        if (r < 0)
Zbigniew Jędrzejewski-Szmek a48f80
+                                return r;
Zbigniew Jędrzejewski-Szmek a48f80
+
Zbigniew Jędrzejewski-Szmek a48f80
+                        fsck = "systemd-fsck-root.service";
Zbigniew Jędrzejewski-Szmek a48f80
+                } else {
Zbigniew Jędrzejewski-Szmek a48f80
+                        _fsck = unit_name_from_path_instance("systemd-fsck", what, ".service");
Zbigniew Jędrzejewski-Szmek a48f80
+                        if (!_fsck)
Zbigniew Jędrzejewski-Szmek a48f80
+                                return log_oom();
Zbigniew Jędrzejewski-Szmek a48f80
 
Zbigniew Jędrzejewski-Szmek a48f80
-                fsck = unit_name_from_path_instance("systemd-fsck", what, ".service");
Zbigniew Jędrzejewski-Szmek a48f80
-                if (!fsck)
Zbigniew Jędrzejewski-Szmek a48f80
-                        return log_oom();
Zbigniew Jędrzejewski-Szmek a48f80
+                        fsck = _fsck;
Zbigniew Jędrzejewski-Szmek a48f80
+                }
Zbigniew Jędrzejewski-Szmek a48f80
 
Zbigniew Jędrzejewski-Szmek a48f80
                 fprintf(f,
Zbigniew Jędrzejewski-Szmek a48f80
                         "RequiresOverridable=%1$s\n"