diff options
| author | Daan De Meyer <daan.j.demeyer@gmail.com> | 2022-10-07 21:21:46 +0200 |
|---|---|---|
| committer | Daan De Meyer <daan.j.demeyer@gmail.com> | 2022-11-15 20:07:54 +0100 |
| commit | 59e2be46ccaef58b4cc652c6ef8cd3507726d215 (patch) | |
| tree | 187b4870a4d02b538e63689e1f9b48849ddfcdd6 /src/shared/mkfs-util.c | |
| parent | ddf615a11c7cc38f0077bc67aa78c8b1b2f54ea2 (diff) | |
| download | systemd-59e2be46ccaef58b4cc652c6ef8cd3507726d215.tar.gz | |
mkfs-util: Add root support for ext and btrfs
For these filesysrems, it's useful to provide the filesystem upfront
so that we don't have to mount it later which requires root privileges.
Diffstat (limited to 'src/shared/mkfs-util.c')
| -rw-r--r-- | src/shared/mkfs-util.c | 34 |
1 files changed, 29 insertions, 5 deletions
diff --git a/src/shared/mkfs-util.c b/src/shared/mkfs-util.c index 0f23a96de2..f7f4a35212 100644 --- a/src/shared/mkfs-util.c +++ b/src/shared/mkfs-util.c @@ -33,6 +33,10 @@ int mkfs_exists(const char *fstype) { return true; } +int mkfs_supports_root_option(const char *fstype) { + return fstype_is_ro(fstype) || STR_IN_SET(fstype, "ext2", "ext3", "ext4", "btrfs"); +} + static int mangle_linux_fs_label(const char *s, size_t max_len, char **ret) { /* Not more than max_len bytes (12 or 16) */ @@ -129,9 +133,9 @@ int make_filesystem( "Don't know how to create read-only file system '%s', refusing.", fstype); } else { - if (root) + if (root && !mkfs_supports_root_option(fstype)) return log_error_errno(SYNTHETIC_ERRNO(EOPNOTSUPP), - "Populating with source tree is only supported for read-only filesystems"); + "Populating with source tree is not supported for %s", fstype); r = mkfs_exists(fstype); if (r < 0) return log_error_errno(r, "Failed to determine whether mkfs binary for %s exists: %m", fstype); @@ -171,7 +175,7 @@ int make_filesystem( assert_se(sd_id128_to_uuid_string(uuid, vol_id)); /* When changing this conditional, also adjust the log statement below. */ - if (streq(fstype, "ext2")) + if (streq(fstype, "ext2")) { argv = strv_new(mkfs, "-q", "-L", label, @@ -180,8 +184,16 @@ int make_filesystem( "-m", "0", "-E", discard ? "discard,lazy_itable_init=1" : "nodiscard,lazy_itable_init=1", node); + if (!argv) + return log_oom(); + + if (root) { + r = strv_extend_strv(&argv, STRV_MAKE("-d", root), false); + if (r < 0) + return log_oom(); + } - else if (STR_IN_SET(fstype, "ext3", "ext4")) + } else if (STR_IN_SET(fstype, "ext3", "ext4")) { argv = strv_new(mkfs, "-q", "-L", label, @@ -192,7 +204,13 @@ int make_filesystem( "-E", discard ? "discard,lazy_itable_init=1" : "nodiscard,lazy_itable_init=1", node); - else if (streq(fstype, "btrfs")) { + if (root) { + r = strv_extend_strv(&argv, STRV_MAKE("-d", root), false); + if (r < 0) + return log_oom(); + } + + } else if (streq(fstype, "btrfs")) { argv = strv_new(mkfs, "-q", "-L", label, @@ -207,6 +225,12 @@ int make_filesystem( return log_oom(); } + if (root) { + r = strv_extend_strv(&argv, STRV_MAKE("-r", root), false); + if (r < 0) + return log_oom(); + } + } else if (streq(fstype, "f2fs")) { argv = strv_new(mkfs, "-q", |