From eca3d07dd1a0d3dc0e18a5fe203f2cccfbd4cc4d Mon Sep 17 00:00:00 2001 From: Daan De Meyer Date: Wed, 26 Jun 2024 20:33:06 +0200 Subject: [PATCH] repart: Allow Subvolumes= and DefaultSubvolume= when running offline mkfs.btrfs has recently learned new options --subvol and --default-subvol so let's stop failing when Subvolumes= and DefaultSubvolume= are used offline and use the new --subvol and --default-subvol options instead to create subvolumes in the generated root filesystem without root privileges or loop devices. --- man/repart.d.xml | 8 +++--- src/partition/repart.c | 56 +++++++++++++++++++++++++++++------------- 2 files changed, 43 insertions(+), 21 deletions(-) diff --git a/man/repart.d.xml b/man/repart.d.xml index ecf14e30998..4ad27649335 100644 --- a/man/repart.d.xml +++ b/man/repart.d.xml @@ -548,8 +548,8 @@ Note that this option only takes effect if the target filesystem supports subvolumes, such as btrfs. - Note that due to limitations of mkfs.btrfs, this option is only supported - when running with . + Note that this option is only supported in combination with + since btrfs-progs 6.11 or newer. @@ -564,8 +564,8 @@ Note that this option only takes effect if the target filesystem supports subvolumes, such as btrfs. - Note that due to limitations of mkfs.btrfs, this option is only supported - when running with . + Note that this option is only supported in combination with + since btrfs-progs 6.11 or newer. diff --git a/src/partition/repart.c b/src/partition/repart.c index 967059dbeab..222c0958ef7 100644 --- a/src/partition/repart.c +++ b/src/partition/repart.c @@ -2087,14 +2087,6 @@ static int partition_read_definition(Partition *p, const char *path, const char "SizeMinBytes=/SizeMaxBytes= cannot be used with Verity=%s.", verity_mode_to_string(p->verity)); - if (!strv_isempty(p->subvolumes) && arg_offline > 0) - return log_syntax(NULL, LOG_ERR, path, 1, SYNTHETIC_ERRNO(EOPNOTSUPP), - "Subvolumes= cannot be used with --offline=yes."); - - if (p->default_subvolume && arg_offline > 0) - return log_syntax(NULL, LOG_ERR, path, 1, SYNTHETIC_ERRNO(EOPNOTSUPP), - "DefaultSubvolume= cannot be used with --offline=yes."); - if (p->default_subvolume && !path_strv_contains(p->subvolumes, p->default_subvolume)) return log_syntax(NULL, LOG_ERR, path, 1, SYNTHETIC_ERRNO(EINVAL), "DefaultSubvolume= must be one of the paths in Subvolumes=."); @@ -3840,7 +3832,7 @@ static int prepare_temporary_file(Context *context, PartitionTarget *t, uint64_t static bool loop_device_error_is_fatal(const Partition *p, int r) { assert(p); - return arg_offline == 0 || (r != -ENOENT && !ERRNO_IS_PRIVILEGE(r)) || !strv_isempty(p->subvolumes) || p->default_subvolume; + return arg_offline == 0 || (r != -ENOENT && !ERRNO_IS_PRIVILEGE(r)); } static int partition_target_prepare( @@ -5156,6 +5148,40 @@ static int partition_populate_filesystem(Context *context, Partition *p, const c return 0; } +static int finalize_extra_mkfs_options(const Partition *p, const char *root, char ***ret) { + int r; + _cleanup_strv_free_ char **sv = NULL; + + assert(p); + assert(ret); + + r = mkfs_options_from_env("REPART", p->format, &sv); + if (r < 0) + return log_error_errno(r, + "Failed to determine mkfs command line options for '%s': %m", + p->format); + + if (partition_needs_populate(p) && root && streq(p->format, "btrfs")) { + STRV_FOREACH(subvol, p->subvolumes) { + if (p->default_subvolume && streq(*subvol, p->default_subvolume)) + continue; + + r = strv_extend_many(&sv, "--subvol", *subvol); + if (r < 0) + return log_oom(); + } + + if (p->default_subvolume) { + r = strv_extend_many(&sv, "--default-subvol", p->default_subvolume); + if (r < 0) + return log_oom(); + } + } + + *ret = TAKE_PTR(sv); + return 0; +} + static int context_mkfs(Context *context) { int r; @@ -5224,11 +5250,9 @@ static int context_mkfs(Context *context) { return r; } - r = mkfs_options_from_env("REPART", p->format, &extra_mkfs_options); + r = finalize_extra_mkfs_options(p, root, &extra_mkfs_options); if (r < 0) - return log_error_errno(r, - "Failed to determine mkfs command line options for '%s': %m", - p->format); + return r; r = make_filesystem(partition_target_path(t), p->format, strempty(p->new_label), root, p->fs_uuid, arg_discard, /* quiet = */ false, @@ -6785,11 +6809,9 @@ static int context_minimize(Context *context) { return r; } - r = mkfs_options_from_env("REPART", p->format, &extra_mkfs_options); + r = finalize_extra_mkfs_options(p, root, &extra_mkfs_options); if (r < 0) - return log_error_errno(r, - "Failed to determine mkfs command line options for '%s': %m", - p->format); + return r; r = make_filesystem(d ? d->node : temp, p->format, -- 2.47.3