[coreboot] [PATCH] K8: Improve RAM init debug messages

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Fri Jun 5 22:37:44 CEST 2009


On 05.06.2009 22:06, Myles Watson wrote:
>> Old messages for my machine with 5 GB:
>> RAM: 0x00400000 kB
>> Ram3
>> [...]
>> Initializing memory:  done
>> RAM: 0x00500000 kB
>>
>> New messages:
>> RAM end at 0x00400000 kB, hole starts at 0x00000000 kB
>>     
> I liked this one better before.  Here it really means there is 4 GB of RAM.
> The "hole at 0x0" part is not helpful.
>   

Since we already print hole location in a separate function, I'll kill
it here.


>> Handling memory hole at 0x00300000 (default)
>> RAM end at 0x00500000 kB, hole starts at 0x00300000 kB
>>     
> Here the new message makes sense.  Do we need to print out the hole start
> again?
>   

Good question. The "Handling memory hole" message is in a separate
function. I'll kill the "hole starts at" message.


>> I hope this helps understand my patch better.
>>     
> Yes.  There's nothing like the real output for succinctness.
>   

I'll to do that in the future. Thanks for the hint.


> Acked-by: Myles Watson <mylesgw at gmail.com>
>   

Thanks, committed in r4341.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the coreboot mailing list