[coreboot] r3259 - trunk/coreboot-v2/targets/amd/serengeti_cheetah_fam10

Marc Jones marc.jones at amd.com
Thu Apr 24 18:37:27 CEST 2008


Carl-Daniel Hailfinger wrote:
> On 24.04.2008 01:25, Marc Jones wrote:
>   
>> Thanks, I was thinking of increasing this to match the normal target.
>>   
>>     
>
> You're welcome.
>
>   
>> Thoughts?
>>   
>>     
>
> I think most of the size problems can be fixed by removing all those
> "inline" keywords (unless this is required by the processor).
>
> So far, these size problems have only been appearing on Stefan's build
> host (probably due to gcc version), so I'm a bit reluctant to see these
> patches as more than a band-aid until we remove unnecessary "inline"s.
>
> Regards,
> Carl-Daniel
>
>   
I think some of those inlines should go away. They are certainly left 
over from previous versions of the code. They are no longer important 
when the ROM is instruction cached and you have CAR for a stack. 
Historically, without instruction caching you can loose performance 
doing random reads to the ROM. You can also lose any prefetch advantage 
your CPU and/or chipset might be doing.

Marc


-- 
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors 






More information about the coreboot mailing list