On Sun, 17 Jul 2022 at 13:08, Rodney W. Grimes
<freebsd-...@gndrsh.dnsmgr.net> wrote:

> Perhaps an assert for kern.geom.part.mbr.enforce_chs not
> being zero in /usr/src/release scripts is in order so that
> the builds blow up rather than produce BAD images.

This is a good interim step (before switching to mkimg).

Perhaps:

diff --git a/release/tools/arm.subr b/release/tools/arm.subr
index 77b708bca4a2..94e0ee89deaf 100644
--- a/release/tools/arm.subr
+++ b/release/tools/arm.subr
@@ -62,6 +62,10 @@ umount_loop() {
 }

 arm_create_disk() {
+       if [ $(sysctl -n kern.geom.part.mbr.enforce_chs) != 0 ]; then
+               return 1
+       fi
+
        # Create the target raw file and temporary work directory.
        chroot ${CHROOTDIR} gpart create -s ${PART_SCHEME} ${mddev}
        if [ "${PART_SCHEME}" = "GPT" ]; then

Reply via email to