Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 1 Dec 2017 14:53:53 -0700
From:      Warner Losh <imp@bsdimp.com>
To:        Shawn Webb <shawn.webb@hardenedbsd.org>
Cc:        "freebsd-arm@freebsd.org" <freebsd-arm@freebsd.org>, FreeBSD Current <freebsd-current@freebsd.org>
Subject:   Re: Booting UEFI ZFS is broken on arm64
Message-ID:  <CANCZdfp6CdTCTQ-wDbHQO2v4w1HC8=iY12F%2BNVofu=kq1%2BVvkg@mail.gmail.com>
In-Reply-To: <20171201214955.zopa6v2f2mx3rkt2@mutt-hbsd>
References:  <20171130002135.q7p27hh6qkog4slr@mutt-hbsd> <CANCZdfrjKkR-Lr4FKVyCBBnYy8k-n6rgz6ES6=_K6SeXtFOtMg@mail.gmail.com> <20171130003458.bnltotlgfdbke5ue@mutt-hbsd> <CANCZdfqgco6gp-ccph3hSgJg81=%2BEfW2DJp0DOx2xgCdgH7UkQ@mail.gmail.com> <20171130004358.usia2jrvman4cvvz@mutt-hbsd> <CANCZdfrH%2ByhkP3RK9aRvgXXnOJ2cw%2B9xU4G9Ge82vZFPERCCCg@mail.gmail.com> <CANCZdfqzHYRt-99MuzWJzYW6bn2YYkQwHE=0APfQM_iOY6RkoQ@mail.gmail.com> <20171201214955.zopa6v2f2mx3rkt2@mutt-hbsd>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Dec 1, 2017 at 2:49 PM, Shawn Webb <shawn.webb@hardenedbsd.org>
wrote:

> On Wed, Nov 29, 2017 at 07:31:17PM -0700, Warner Losh wrote:
> > On Wed, Nov 29, 2017 at 5:54 PM, Warner Losh <imp@bsdimp.com> wrote:
> >
> > >
> > >
> > > On Wed, Nov 29, 2017 at 5:43 PM, Shawn Webb <
> shawn.webb@hardenedbsd.org>
> > > wrote:
> > >
> > >> On Wed, Nov 29, 2017 at 05:42:52PM -0700, Warner Losh wrote:
> > >> > On Wed, Nov 29, 2017 at 5:34 PM, Shawn Webb <
> shawn.webb@hardenedbsd.org
> > >> >
> > >> > wrote:
> > >> >
> > >> > > On Wed, Nov 29, 2017 at 05:33:46PM -0700, Warner Losh wrote:
> > >> > > > On Wed, Nov 29, 2017 at 5:21 PM, Shawn Webb <
> > >> shawn.webb@hardenedbsd.org>
> > >> > > > wrote:
> > >> > > >
> > >> > > > > It appears that in the latest FreeBSD 12-CURRENT/arm64
> snapshot,
> > >> > > > > booting UEFI GPT ZFS on my OverDrive 1000 is broken. It boots
> up
> > >> to
> > >> > > > > this line:
> > >> > > > >
> > >> > > > > Using DTB provided by EFI at 0x801fe00000.
> > >> > > >
> > >> > > >
> > >> > > > Which snapshot is that? Boot1 was broken until recently.
> > >> > >
> > >> > > FreeBSD-12.0-CURRENT-arm64-aarch64-20171121-r326056-memstick.img
> > >> > >
> > >> > > It also happens on latest HEAD, so it would appear to still be
> broken.
> > >> >
> > >> >
> > >> > Is this boot1.efi producing the output, or loader.efi? I'm guessing
> the
> > >> > latter, but wanted to make sure. If so, then we're past the point
> where
> > >> > boot1.efi would have failed (besides, it was fixed before that
> > >> snapshot).
> > >>
> > >> With DEBUG turned on for stand/fdt:
> > >>
> > >> Booting [/boot/kernel/kernel]...
> > >> fdt_copy(): fdt_copy va 0x01208000
> > >> fdt_setup_fdtp(): fdt_setup_fdtp()
> > >> fdt_load_dtb_addr(): fdt_load_dtb_addr(0x801fe00000)
> > >> Using DTB provided by EFI at 0x801fe00000.
> > >> Loaded the platform dtb: 0x81f56f1630.
> > >> fdt_fixup(): fdt_fixup()
> > >>
> > >> ^ hangs after that message
> > >
> > >
> > > That doesn't sound like anything I've changed, but it could well be...
> I
> > > think to find this breakage, you may need to bisect backwards along
> stand /
> > > sys/boot until we find the spot where it broke.
> > >
> >
> > There's been several conversations on IRC about how others are hitting a
> > scheduler bug, at least on x86. hps' fix seems to do the trick for their
> > issues.
> >
> > Author: hselasky <hselasky@ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f>
> > Date:   Wed Nov 29 23:28:40 2017 +0000
> >
> >     The sched_add() function is not only used when the thread is
> initially
> >     started, but also by the turnstiles to mark a thread as runnable for
> >     all locks, for instance sleepqueues do:
> >     setrunnable()->sched_wakeup()->sched_add()
> >
> >     In r326218 code was added to allow booting from non-zero CPU numbers
> >     by setting the ts_cpu field inside the ULE scheduler's sched_add()
> >     function. This had an undesired side-effect that prior sched_pin()
> and
> >     sched_bind() calls got disregarded. This patch fixes the
> >     initialization of the ts_cpu field for the ULE scheduler to only
> >     happen once when the initial thread is constructed during system
> >     init. Forking will then later on ensure that a valid ts_cpu value
> gets
> >     copied to all children.
> >
> >     Reviewed by:    jhb, kib
> >     Discussed with: nwhitehorn
> >     MFC after:      1 month
> >     Differential revision:  https://reviews.freebsd.org/D13298
> >     Sponsored by:   Mellanox Technologies
> >
> >
> >     git-svn-id: svn+ssh://svn.freebsd.org/base/head@326376
> > ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f
> >
> > is the fix.... But the bug it fixes post-dates the snapshot, so maybe
> this
> > isn't the same thing...
>
> Definitely is not the same thing. I've so far got it printf'd to where
> the uefi loader jumps into the kernel's entry point. So the loader
> itself might be fine. Something in the kernel, then, is going funky.
>
> Booting in verbose mode does not provide any additional input.
>
> Here's the output I get (some of the output is from printf's I've
> done):
>
> FreeBSD/arm64 EFI loader, Revision 1.1
> (Wed Nov 29 21:51:14 EST 2017 shawn@hbsd-dev-laptop)
> EFI boot environment
> Loading /boot/defaults/loader.conf
> /boot/kernel/kernel text=0x7e0a78 data=0xaad80+0x443f62
> syms=[0x8+0x10ec78+0x8+0x1021d4]
> /boot/entropy size=0x1000
> /boot/kernel/zfs.ko text=0x99070 text=0x130390 data=0x21ff8+0x9ef98
> syms=[0x8+0x22c68+0x8+0x1b99b]
> /boot/kernel/opensolaris.ko text=0x1330 text=0xd00 data=0x10160+0x125d0
> syms=[0x8+0xff0+0x8+0x8d8]
>
> Hit [Enter] to boot immediately, or any other key for command prompt.
> Booting [/boot/kernel/kernel]...
> Using DTB provided by EFI at 0x801fe00000.
> fdt_copy returned. dtb_size is 9060.
> bi_load finished. err: 0
> dev_cleanup finished
> About to call into the entry point at 0x81ee601000
>

You might try booting the same kernel off a small UFS partition. There's a
tiny chance that the loader didn't load it right, but more likely the
kernel is borked. Maybe DTB issues? Maybe something else... A quick test
like that would remove ZFS from the equation, even if it's just a USB
stick...

Warner



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfp6CdTCTQ-wDbHQO2v4w1HC8=iY12F%2BNVofu=kq1%2BVvkg>