[coreboot] Data in memory changes unexpectedly ininitialize_cpus

[Marc Jones]

On Thu, Feb 11, 2010 at 6:54 PM, Bao, Zheng <Zheng.Bao at amd.com> wrote:
> I kept on finding the problem. Now it is narrowed down to the line 108,
> src/arch/i386/coreboot_ram.ld. It is very, very, very, very confused.
> My CONFIG_RAMBASE should be 0x200000, which is more than 0x100000, but
> the location counter "." is CONFIG_STACK_SIZE. Even if I manually set
> ". = CONFIG_STACK_SIZE * 2 ;", I can not get the correct size. But if I
> set
> ". = 0x2000 * 2 ;", or ". = CONFIG_STACK_SIZE << 1", it is right. I
> kinda doubt the compiler and linker.
>
> Can you try it on your building machine? Just try the
> Serengeti_cheetah_fam10 and don't need the hardware. Just check the
> build/coreboot_ram.map to see if the _estack - _stack is multiple of
> CONFIG_STACK_SIZE. That problem should happen on all the fam10 board if
> it is built on my building machine.

Is this due to the hard coded values in lapic_cpu_init. I don't really
understand the comment or what it is doing. I would expect the AP
stacks to be setup based on estack as well.

// for all APs, let use stack after pgtbl, 20480 is the pgtbl size for every cpu
stack_end = 0x100000+(20480 + CONFIG_STACK_SIZE)*CONFIG_MAX_CPUS -
(CONFIG_STACK_SIZE*index);

Marc

-- 
http://se-eng.com