Re: [PATCH] powerpc/mm: Add translation mode information in /proc/cpuinfo

["Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>]

On Wednesday 22 February 2017 11:15 AM, Michael Ellerman wrote:
> "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> writes:
>
>> With this we have on powernv and pseries /proc/cpuinfo reporting
>>
>> timebase        : 512000000
>> platform        : PowerNV
>> model           : 8247-22L
>> machine         : PowerNV 8247-22L
>> firmware        : OPAL
>> translation     : Hash
>>
>> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>
>> ---
>>   arch/powerpc/platforms/powernv/setup.c | 4 ++++
>>   arch/powerpc/platforms/pseries/setup.c | 4 ++++
>>   2 files changed, 8 insertions(+)
>>
>> diff --git a/arch/powerpc/platforms/powernv/setup.c b/arch/powerpc/platforms/powernv/setup.c
>> index d50c7d99baaf..d38571e289bb 100644
>> --- a/arch/powerpc/platforms/powernv/setup.c
>> +++ b/arch/powerpc/platforms/powernv/setup.c
>> @@ -95,6 +95,10 @@ static void pnv_show_cpuinfo(struct seq_file *m)
>>   	else
>>   		seq_printf(m, "firmware\t: BML\n");
>>   	of_node_put(root);
>> +	if (radix_enabled())
>> +		seq_printf(m, "translation\t: Radix\n");
>> +	else
>> +		seq_printf(m, "translation\t: Hash\n");
>>   }
> Can we just call it "MMU" ?
>
> I don't think it's entirely clear what "translation" means here if you
> don't already know.
>
> cheers
>
I avoided using MMU, because it will confuse hardware guys. Radix is not 
clearly the full definition of
Memory management unit, but rather the translation mode used by memory 
management unit. But
i don't have strong opinion on this.

Do you want me to send an updated patch ? or you can update it when you 
apply it to your tree ?

-aneesh