Page MenuHomeFreeBSD

bsdinstall: Remove support for ZFS + MBR disk layouts
ClosedPublic

Authored by emaste on Jun 25 2025, 1:24 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Aug 3, 1:27 PM
Unknown Object (File)
Sun, Aug 3, 11:38 AM
Unknown Object (File)
Sun, Aug 3, 7:53 AM
Unknown Object (File)
Fri, Aug 1, 7:02 PM
Unknown Object (File)
Fri, Aug 1, 3:21 PM
Unknown Object (File)
Fri, Aug 1, 5:19 AM
Unknown Object (File)
Thu, Jul 31, 9:03 PM
Unknown Object (File)
Thu, Jul 31, 7:58 PM
Subscribers

Details

Summary
It hasn't worked for some time -- as reported in review D40816
"Installing FreeBSD with Auto ZFS + MBR has been broken ever since the
move to OpenZFS with FreeBSD 13."  It relied on the partition table and
ZFS data overlapping in a very fragile way and is not a good idea.

Diff Detail

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

Event Timeline

usr.sbin/bsdinstall/scripts/zfsboot
1674
emaste retitled this revision from bsdinstall: Disable BIOS + MBR to bsdinstall: Disable BIOS + MBR ZFS config.Jun 25 2025, 1:54 AM

If you want to remove it outright, that's probably a bigger change? I would say we should probably remove it entirely from bsdinstall, then probably stop building the non-GPT zfsboot as well.

We will probably want a general RELNOTES entry at some point about removing ZFS MBR boot support?

This revision is now accepted and ready to land.Jun 30 2025, 6:08 PM

A few more stragglers. I would maybe reword the commit log as "bsdinstall: Remove support for ZFS + MBR disk layouts".

Note that this means you can't use ZFS + MBR on a non-booting disk, but that seems ok to me. There's no good reason to do that anyway. You can always use the raw disk or GPT on non-boot disks.

usr.sbin/bsdinstall/scripts/zfsboot
828

Hmm, my suggestion to remove the invocation of sysctl kern.geom.part.mbr.enforce_chs=0 wasn't posted, but that should be removed as well.

emaste retitled this revision from bsdinstall: Disable BIOS + MBR ZFS config to bsdinstall: Remove support for ZFS + MBR disk layouts.
emaste edited the summary of this revision. (Show Details)

rebase, incorporate feedback from @jhb

This revision now requires review to proceed.Fri, Jul 25, 5:11 PM

Let's land this. I can work on removing /boot/zfsboot afterwards.

This revision is now accepted and ready to land.Fri, Jul 25, 6:54 PM