From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Schantl <stefan.schantl@ipfire.org>
To: development@lists.ipfire.org
Subject:
 [PATCHv2 11/12] installer: Add code to proper unmount the BTRFS layout
Date: Sat, 23 Mar 2024 11:56:28 +0100
Message-ID: <20240323105629.371511-11-stefan.schantl@ipfire.org>
In-Reply-To: <20240323105629.371511-1-stefan.schantl@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============3991171556102328563=="
List-Id: <development.lists.ipfire.org>

--===============3991171556102328563==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
---
 src/installer/hw.c | 72 +++++++++++++++++++++++++++++++++++++++++++++-
 1 file changed, 71 insertions(+), 1 deletion(-)

diff --git a/src/installer/hw.c b/src/installer/hw.c
index 5665cb955..3b26920d1 100644
--- a/src/installer/hw.c
+++ b/src/installer/hw.c
@@ -1123,13 +1123,83 @@ int hw_umount_filesystems(struct hw_destination* dest=
, const char* prefix) {
 		return -1;
=20
 	// root
-	r =3D hw_umount(prefix, NULL);
+	if(dest->filesystem =3D=3D HW_FS_BTRFS) {
+		r =3D hw_umount_btrfs_layout();
+	} else {
+		r =3D hw_umount(prefix, NULL);
+	}
+
 	if (r)
 		return -1;
=20
 	return 0;
 }
=20
+int hw_umount_btrfs_layout() {
+	const struct btrfs_subvolumes* subvolume =3D NULL;
+	char path[STRING_SIZE];
+	int counter =3D 0;
+	int retry =3D 1;
+	int r;
+
+	do {
+		// Reset the retry marker
+		retry =3D 0;
+
+		// Loop through the list of subvolumes
+		for (subvolume =3D btrfs_subvolumes; subvolume->name; subvolume++) {
+			// Abort if the subvolume path could not be assigned.
+			r =3D snprintf(path, sizeof(path), "%s%s", DESTINATION_MOUNT_PATH, subvol=
ume->mount_path);
+
+			if (r < 0) {
+				return r;
+			}
+
+			// Try to umount the subvolume.
+			r =3D umount2(path, 0);
+
+			// Handle return codes.
+			if (r) {
+				switch(errno) {
+					case EBUSY:
+						// Set marker to retry the umount.
+						retry =3D 1;
+
+						// Ignore if the subvolume could not be unmounted yet,
+						// because it is still used.
+						continue;
+
+					case EINVAL:
+						// Ignore if the subvolume already has been unmounted
+						continue;
+					case ENOENT:
+						// Ignore if the directory does not longer exist.
+						continue;
+					default:
+						fprintf(flog, "Could not umount %s from %s - Error: %d\n", subvolume->=
name, path, r);
+						return r;
+				}
+			}
+
+			// Print log message
+			fprintf(flog, "Umounted %s from %s\n", subvolume->name, path);
+		}
+
+		// Abort loop if all mountpoins got umounted
+		if (retry =3D=3D 0) {
+			return 0;
+		}
+
+		// Abort after five failed umount attempts
+		if (counter =3D=3D 5) {
+			return -1;
+		}
+
+		// Increment counter.
+		counter++;
+	} while (1);
+}
+
 int hw_destroy_raid_superblocks(const struct hw_destination* dest, const cha=
r* output) {
 	char cmd[STRING_SIZE];
=20
--=20
2.39.2


--===============3991171556102328563==--