From c6c6d342a00f7f17c230d48114f22df08ae6a201 Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Fri, 21 Jun 2024 13:42:39 -0400 Subject: [PATCH] install/baseline: Drop aarch64 reserved partition This was originally done to match what CoreOS is doing, which is motivated by having Ignition configs that work cross-architecture. We don't have the same constraints or focus; we encourage operation at filesystem level in general. This just changes the baseline `to-disk` installer, anyone who wants to do things like cross-architecture matching can of course do so externally. Signed-off-by: Colin Walters --- lib/src/install/baseline.rs | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/lib/src/install/baseline.rs b/lib/src/install/baseline.rs index 17a29b2e9..ffc46a59c 100644 --- a/lib/src/install/baseline.rs +++ b/lib/src/install/baseline.rs @@ -246,7 +246,6 @@ pub(crate) fn install_create_rootfs( sgdisk.cmd.arg("-Z"); sgdisk.cmd.arg(device.path()); sgdisk.cmd.args(["-U", "R"]); - #[allow(unused_assignments)] if cfg!(target_arch = "x86_64") { // BIOS-BOOT partno += 1; @@ -258,15 +257,7 @@ pub(crate) fn install_create_rootfs( Some("21686148-6449-6E6F-744E-656564454649"), ); } else if cfg!(target_arch = "aarch64") { - // reserved - partno += 1; - sgdisk_partition( - &mut sgdisk.cmd, - partno, - "0:+1M", - "reserved", - Some("8DA63339-0007-60C0-C436-083AC8230908"), - ); + // Only UEFI here for now } else { anyhow::bail!("Unsupported architecture: {}", std::env::consts::ARCH); }