[coreboot] RAMINIT_SYSINFO?

[Marc Jones]

On Mon, Oct 17, 2011 at 4:40 AM, Oskar Enoksson <enok at lysator.liu.se> wrote:
> On 10/17/2011 04:24 AM, Marc Jones wrote:
>>
>> On Sun, Oct 16, 2011 at 4:57 AM, Oskar Enoksson<enok at lysator.liu.se>
>>  wrote:
>>>
>>> Can someone explain what RAMINIT_SYSINFO configuration option does? I
>>> thought I understood, but I don't.
>>>
>>> /Oskar
>>
>> Hi Oskar,
>>
>> RAMINIT_SYSINFO is the config option to create the AMD sysinfo
>> structure in CAR above the stack. This is a way to get global data in
>> the romstage (heap like without any management). The space for the
>> data structure is allocated by the DCACHE_RAM_GLOBAL_VAR_SIZE config
>> and the pointer is setup in cache_as_ram_main.
>
> So setting RAMINIT_SYSINFO enables some functionality that needs that data
> in the romstage, e.g. optimize_link_incoherent_ht (?) But what would be the
> reason to NOT use RAMINIT_SYSINFO? To save space in the cache?

Yes, and if you had a platform that didn't require it for memory
training or other features that use the sysinfo structure. For K8, I
recommend that you leave it enabled.

Marc


-- 
http://se-eng.com