On Fri, Feb 26, 2010 at 1:24 PM, Patrick Georgi <[email protected]>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 <[email protected]>
>
Rev 5166.

Thanks,
Myles
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to