Rene Herman wrote:
quoted text
> On 21-05-08 00:21, H. Peter Anvin wrote:
> 
>> Rene Herman wrote:
>>>> +}
>>>> +__setup("enablepat", x86_forcepat_setup);
>>>
>>> This should probably be called plain "pat" to mirror 
>>> arch/x86/mm/pat.c "nopat" force off parameter. That by the way is an 
>>> early_param which I guess this should then also be?
>>>
>>
>> Either that or "pat={off,on,force}" to give space for other options... 
>> would mean keeping "nopat" around as an alias for now, though...
> 
> Yes, that would be nicer. As to the alias; "nopat" hasn't been in a 
> released kernel yet so should be okay to do away with? It's not like 
> it's in Documentation/kernel-parameters.txt or anything... ;-/
> 

OK, just double-checked... since it's not in Linus we can still change 
it, and as so I'd suggest the pat= option.

	-hpa
--
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/