Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 23 Mar 2018 15:16:54 +0000
From:      Ruslan Bukin <ruslan.bukin@cl.cam.ac.uk>
To:        Alan Somers <asomers@freebsd.org>
Cc:        Conrad Meyer <cem@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r330894 - in head/sys/contrib/zstd: . contrib/meson doc doc/images lib lib/common lib/compress lib/decompress lib/deprecated lib/dictBuilder lib/legacy programs tests zlibWrapper zlibWr...
Message-ID:  <20180323151654.GA24598@bsdpad.com>
In-Reply-To: <20180323115227.GA21270@bsdpad.com>
References:  <201803140300.w2E30HmE024745@repo.freebsd.org> <CAOtMX2hfcKkSPy5QzOoQG0FcmSDZOHOgiAgHQ_2qWDcgwQDqGg@mail.gmail.com> <20180323084636.GA18215@bsdpad.com> <20180323115227.GA21270@bsdpad.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Mar 23, 2018 at 11:52:27AM +0000, Ruslan Bukin wrote:
> On Fri, Mar 23, 2018 at 08:46:36AM +0000, Ruslan Bukin wrote:
> > On Sat, Mar 17, 2018 at 03:49:15PM -0600, Alan Somers wrote:
> > >    On Tue, Mar 13, 2018 at 9:00 PM, Conrad Meyer <[1]cem@freebsd.org> wrote:
> > > 
> > >      Author: cem
> > >      Date: Wed Mar 14 03:00:17 2018
> > >      New Revision: 330894
> > >      URL: [2]https://svnweb.freebsd.org/changeset/base/330894
> > > 
> > >      Log:
> > >        Update to Zstandard 1.3.3
> > > 
> > >    I think this broke the build on RISC-V.  Could you please take a look?
> > 
> > I can confirm it breaks RISC-V
> > 
> 
> I found the problem: after this commit kernel become slightly bigger: it was 0x5fd080 bytes, become 0x61b080 bytes.
> 
> Spike gives us 0x600000 free space only between start of physram to place where it puts DTB.
> So DTB appears under our BSS section and we clear it during boot, so DTB is no longer valid and kernel get stuck in fdt header checking routine.
> 
> I'm currently looking for a solution and rearrangements, it will take some time.
> 

I found that Berkeley Boot Loader (BBL) miscalculates the size of payload (freebsd kernel) -- they does not include BSS section to calculatons.

I made a fix to our local riscv-pk (https://github.com/freebsd-riscv/riscv-pk) and created a pull request:
https://github.com/riscv/riscv-pk/pull/91

Thanks

Ruslan



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