Page MenuHomeFreeBSD

Put /boot/efi back in mtree
ClosedPublic

Authored by nwhitehorn on Mar 4 2021, 7:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 10, 8:38 PM
Unknown Object (File)
Wed, Apr 10, 2:32 AM
Unknown Object (File)
Tue, Apr 2, 5:08 AM
Unknown Object (File)
Feb 12 2024, 5:51 PM
Unknown Object (File)
Feb 7 2024, 8:07 PM
Unknown Object (File)
Feb 7 2024, 7:52 PM
Unknown Object (File)
Jan 14 2024, 2:32 AM
Unknown Object (File)
Jan 12 2024, 7:21 AM
Subscribers

Details

Summary

The attached patch puts /boot/efi back in mtree, so it is created on systems without an ESP as well as systems with an ESP (I still don't really understand why we want this...). To make installations work so that only real ESPs get set up with the loader and efibootmgr, the logic in the installer is switched from whether /boot/efi exists (which it now always does) to whether it is present as a mountpoint in the in-progress fstab.

NOTE: I have not tested this fully yet, but it takes long enough to do end-to-end build and test for installer changes that I'm putting this up for review now to save time. It's a 2-line diff anyway, so I'm guessing most of the discussion will be on concept rather than any bugs that might exist.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

nwhitehorn created this revision.
This revision is now accepted and ready to land.Mar 4 2021, 11:20 PM
rgrimes added a subscriber: rgrimes.

This is a move in the right directions, at least this is state created by the installer.

Confirmed that I can still install on powerpc64le in qemu with this change, and that the empty /boot/efi directory exists again.

This is a move in the right directions, at least this is state created by the installer.

It was always state created by the installer, for the record. It's just different state now (whether it make a directory earlier vs. whether it added that directory to /etc/fstab after making it).

This revision was automatically updated to reflect the committed changes.