cvs commit: src/sys/boot/alpha Makefile.inc src/sys/boot/ficl Makefile

[Doug Rabson]

On Tuesday 24 August 2004 15:02, John Baldwin wrote:
> On Sunday 22 August 2004 04:40 am, Doug Rabson wrote:
> > On Sunday 22 August 2004 07:24, Marcel Moolenaar wrote:
> > > marcel      2004-08-22 06:24:59 UTC
> > >
> > >   FreeBSD src repository
> > >
> > >   Modified files:
> > >     sys/boot/alpha       Makefile.inc
> > >     sys/boot/ficl        Makefile
> > >   Log:
> > >   Part 2 of fixing the boot code: gcc 3.4 fixes.
> > >
> > >   The whole problem seems to be size. Which is odd, because it is
> > > said that size doesn't matter. Anyway... Add -Os to strategic
> > > places in the makefile to have the final loader be as mall as
> > > possible. This seems to be enough to make it work. For now... I
> > > think something is more fundamentally wrong; or something more
> > > fundamental is wrong. Potato, potaato.
> >
> > Size does matter for the alpha loader. The firmware gives it 256k
> > of address space which we overflowed many years ago. I extended it
> > in sys/boot/alpha/common/main.c:extend_heap() by adding 512k to the
> > loader's mapped address space. It might be necessary to extend it a
> > bit further.
>
> That would just require bumping the XTRA_PAGES constant up, yes? 
> Maybe we could bump it to 96 to add 768k for a total of 1m?  The i386
> loader has to fit into ~600kb including stack and heap, so hopefully
> 1m would be enough room for Alpha. :)

I think it should be harmless to increase XTRA_PAGES although I'm not 
exactly certain that this was the real cause of the recent loader 
changes (it just seemed plausible). I also can't quite remember whether 
we manage to reclaim the loader's memory and give it back to the VM 
system after boot.