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...

[Ruslan Bukin]

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 at 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