[coreboot] [patch] RE: Fam10 breakage

Myles Watson mylesgw at gmail.com
Fri Feb 26 21:32:49 CET 2010


On Fri, Feb 26, 2010 at 1:24 PM, Patrick Georgi <patrick at georgi-clan.de>wrote:

> Am 26.02.2010 16:35, schrieb Myles Watson:
> > For me, the only change that needs to be made is:
> >
> > -           . = ((CONFIG_CONSOLE_VGA ||
> > CONFIG_PCI_ROM_RUN)&&(CONFIG_RAMBASE<0x100000)&&(CONFIG_RAMTOP>0x100000)
> > ) ? CONFIG_STACK_SIZE : (CONFIG_MAX_CPUS*CONFIG_STACK_SIZE);
> >
> > +           . += ((CONFIG_CONSOLE_VGA ||
> > CONFIG_PCI_ROM_RUN)&&(CONFIG_RAMBASE<0x100000)&&(CONFIG_RAMTOP>0x100000)
> > ) ? CONFIG_STACK_SIZE : (CONFIG_MAX_CPUS*CONFIG_STACK_SIZE);
> >
> > Removing the .stack construct makes no difference.
> >
> > I like the idea of minimizing the change.
> Sounds good, and should be stable (unless that's part of the bug Zheng
> Bao is experiencing).
>
> I'd say, commit this (as it fixes things for you). If it's not enough,
> we can do the full change.
>
Great.


> Acked-by: Patrick Georgi <patrick.georgi at coresystems.de>
>
Rev 5166.

Thanks,
Myles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20100226/e9a87ff9/attachment.html>


More information about the coreboot mailing list