Skip site navigation (1)Skip section navigation (2) 
Date:      Fri, 01 Feb 2019 13:44:06 +0000
From:      bugzilla-noreply@freebsd.org
To:        bugs@FreeBSD.org
Subject:   [Bug 235380] gptzfsboot does not boot ZFS pool made from whole disks (regression)
Message-ID:  <bug-235380-227@https.bugs.freebsd.org/bugzilla/>
next in thread | raw e-mail | index | archive | help 
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D235380

            Bug ID: 235380
           Summary: gptzfsboot does not boot ZFS pool made from whole
                    disks (regression)
           Product: Base System
           Version: 12.0-RELEASE
          Hardware: amd64
                OS: Any
            Status: New
          Severity: Affects Some People
          Priority: ---
         Component: kern
          Assignee: bugs@FreeBSD.org
          Reporter: reto.haeuptli@infinox.ch

I used to boot zfs on whole disk(s) from a USB stick with a GPT partition
scheme and gptzfsboot on it since FreeBSD version 9 on many testing machines
and even production servers without any problems. With the release 12 this
feature is broken.=20

I found the reason in the source code: In the file
/head/stand/i386/libi386/biosdisk.c a regression was introduced in the
revisions base r37271, base r337317. Its about detecting the correct number=
 of
sectors/blocks for a block device. The change was introduced because a cert=
ain
device (Mediasonic HD3-U2B PATA to USB enclosure) was not working. I tested
various SASTA disks, even a 10TB disk, and USB Sticks, all detecting the wr=
ong
number of sectors instead. Reverting the change leaded to the correct detec=
tion
of the number of sectors and therefore I was able to boot again. I tested a=
lso
revision base r342785 of the file /head/stand/i386/libi386/biosdisk.c: The
detection of the number of sectors is implemented the right way again and h=
as
improved of course.

I propose the following patch to fix the regression for FreeBSD 12

Index: stand/i386/libi386/biosdisk.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- stand/i386/libi386/biosdisk.c       (revision 343578)
+++ stand/i386/libi386/biosdisk.c       (working copy)
@@ -270,8 +270,8 @@

                total =3D (uint64_t)params.cylinders *
                    params.heads * params.sectors_per_track;
-               if (total > 0 && bd->bd_sectors > total)
-                       bd->bd_sectors =3D total;
+                if (bd->bd_sectors < total)
+                        bd->bd_sectors =3D total;

                ret =3D 1;
        }

--=20
You are receiving this mail because:
You are the assignee for the bug.=

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-235380-227>