From de3416f55bc9cf1dd3944b20b135a18ff3543943 Mon Sep 17 00:00:00 2001
From: mintsuki <36459316+mintsuki@users.noreply.github.com>
Date: Sun, 6 Apr 2025 22:34:20 +0200
Subject: [PATCH] Start the boot partition at 1MiB by default on MBR too
 (#3344)

---
 archinstall/lib/interactions/disk_conf.py | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/archinstall/lib/interactions/disk_conf.py b/archinstall/lib/interactions/disk_conf.py
index e0dd5bec..9d0e8c35 100644
--- a/archinstall/lib/interactions/disk_conf.py
+++ b/archinstall/lib/interactions/disk_conf.py
@@ -232,11 +232,9 @@ def select_lvm_config(
 def _boot_partition(sector_size: SectorSize, using_gpt: bool) -> PartitionModification:
 	flags = [PartitionFlag.BOOT]
 	size = Size(1, Unit.GiB, sector_size)
+	start = Size(1, Unit.MiB, sector_size)
 	if using_gpt:
-		start = Size(1, Unit.MiB, sector_size)
 		flags.append(PartitionFlag.ESP)
-	else:
-		start = Size(3, Unit.MiB, sector_size)
 
 	# boot partition
 	return PartitionModification(
@@ -362,15 +360,6 @@ def suggest_single_disk_layout(
 	available_space = available_space.align()
 
 	# Used for reference: https://wiki.archlinux.org/title/partitioning
-	# 2 MiB is unallocated for GRUB on BIOS. Potentially unneeded for other bootloaders?
-
-	# TODO: On BIOS, /boot partition is only needed if the drive will
-	# be encrypted, otherwise it is not recommended. We should probably
-	# add a check for whether the drive will be encrypted or not.
-
-	# Increase the UEFI partition if UEFI is detected.
-	# Also re-align the start to 1MiB since we don't need the first sectors
-	# like we do in MBR layouts where the boot loader is installed traditionally.
 
 	boot_partition = _boot_partition(sector_size, using_gpt)
 	device_modification.add_partition(boot_partition)