Re: [PATCH] x86: trim ram need to check if mtrr is there v3

[Jeremy Fitzhardinge <jeremy@goop.org>]

Yinghai Lu wrote:
> On Friday 25 January 2008 10:55:31 am Jeremy Fitzhardinge wrote:
>   
>> Ingo Molnar wrote:
>>     
>>> * Yinghai Lu <Yinghai.Lu@Sun.COM> wrote:
>>>
>>>   
>>>       
>>>> [PATCH] x86: trim ram need to check if mtrr is there v3
>>>>     
>>>>         
>>>   
>>>       
>>>>> H. Peter Anvin wrote:
>>>>> Looks like the code doesn't check that the CPU *has* MTRRs...
>>>>>       
>>>>>           
>>>> so more strict check if mtrr is there really.
>>>> bail out if mtrr all blank when qemu cpu model is used
>>>>
>>>> and check if is AMD as early
>>>> also remove 4G less check, according to hpa.
>>>>     
>>>>         
>>> thanks, applied. Shouldnt we put in an exception for when there is MTRR 
>>> support, but they dont cover anything. Still emit a warning - but 
>>> booting up real slow is still better than losing all of RAM and crashing 
>>>   
>>>       
>> The problem is re-occuring for me with current x86.git.  Looks like v2 
>> did the trick, and v3 is broken...
>>     
>
> so the ram size less 4g is way out for your case.
>
> again, can you post /proc/mtrrs with v2 patch?
>   

It was empty.  But ignore this report; it wasn't against the right 
version of x86.git.

    J