[LinuxBIOS] [PATCH v3] v2, v3: Bug in CAR setup with CacheSize!={4k, 8k, 16k}
Marc Jones
marc.jones at amd.com
Fri Nov 30 18:36:20 CET 2007
Marc Jones wrote:
>
> yhlu wrote:
>> my own tree have
>>
>> #if CacheSize < 0x8000
>> /* enable caching for 16K/8K/4K using fixed mtrr */
>> movl $0x269, %ecx /* fix4k_cc000*/
>> #if CacheSize == 0x4000
>> movl $0x06060606, %edx /* WB IO type */
>> #endif
>> #if CacheSize == 0x2000
>> movl $0x06060000, %edx /* WB IO type */
>> #endif
>> #if CacheSize == 0x1000
>> movl $0x06000000, %edx /* WB IO type */
>> #endif
>> xorl %eax, %eax
>> wrmsr
>> #endif
>>
>> maybe i didn't post sth...
>>
> This is in the v2 code but not in v3.
>
>
>> normally we can use 4k only at first.
>>
>> run_time decide is not good. for example REV E, you could use 64k. for
>> for REV F you can only use 48K.
>>
> So this is still a problem since 48K isn't an option and it isn't a
> power of 2 so I was incorrect on that point.
>
> It would be good to have an automatic runtime setting for each CPU that
> could be adjusted by a mainboard override.
>
>
>> Family 10h need more tricks.
> These changes are coming soon.
>
> Marc
>
>
Also, 48K is an option in v2. It seems that v3 should be brought up to
date with v2 first.
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