[coreboot] [PATCH] RAMTOP fixes
Myles Watson
mylesgw at gmail.com
Tue Apr 27 18:50:30 CEST 2010
> >> I think we should generally #define RAMBASE for x86 architectures to
> >> (above) 1MB and remove it from Kconfig.
> >>
> >> It will allow us to simplify CAR and make the different CPUs' CAR code
> >> look more alike
> >
> > I think that's a great idea. Extra complexity has never helped us.
> >
> > For RAMTOP would we have to play some tricks to figure out how big
> > coreboot_ram was if RAMTOP isn't a config variable? The fam10 boards
> use a
> > lot of RAM right now.
>
> Excellent idea!
>
> Myles, is there ever a case where RAMTOP would be so low that the
> fam10 boards would run out of RAM? What is the need for RAMTOP?
For suspend/resume I think we like to keep RAMTOP-RAMBASE = 1M. Last time I
checked, some of the fam10 boards are using RAMBASE=2M and RAMTOP=16M.
Once you start allowing 48 cores and you want page tables on all of their
stacks, it gets big quickly.
Thanks,
Myles
More information about the coreboot
mailing list