Re: [coreboot] [patch] RE: Fam10 breakage

[Myles Watson]

On Fri, Feb 26, 2010 at 1:24 PM, Patrick Georgi 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

Rev 5166.

Thanks,
Myles