Re: ***UNCHECKED*** Re: [PATCH V2 1/2] cpu: Handle only high order 16 bits of PVR for IBM Power processors

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/15/2014 02:56 PM, Daniel P. Berrange wrote:
> On Sat, Sep 13, 2014 at 12:00:53PM +0530, Pradipta Kr. Banerjee wrote:
>> cpu: Handle only high order 16 bits of PVR for IBM Power processors
>>
>> IBM Power processors encode PVR as CPU family in higher
>> 16 bits and a CPU version in lower 16 bits. Since there is no significant
>> change in behavior between versions, there is no point to add every single
>> CPU version in cpu_map.xml
>>
>> Signed-off-by: Pradipta Kr. Banerjee <bpradip@xxxxxxxxxx>
>> ---
>>  src/cpu/cpu_map.xml   | 22 ++++++++--------------
>>  src/cpu/cpu_powerpc.c |  7 ++++++-
>>  2 files changed, 14 insertions(+), 15 deletions(-)
>>
>> diff --git a/src/cpu/cpu_map.xml b/src/cpu/cpu_map.xml
>> index 18c7b0d..f2f13fb 100644
>> --- a/src/cpu/cpu_map.xml
>> +++ b/src/cpu/cpu_map.xml
>> @@ -602,30 +602,24 @@
>>      <vendor name='IBM'/>
>>  
>>      <!-- IBM-based CPU models -->
>> -    <model name='POWER7'>
>> +    <model name='power6'>
>>        <vendor name='IBM'/>
>> -      <pvr value='0x003f0200'/>
>> +      <pvr value='0x003e0000'/>
>>      </model>
>>  
>> -    <model name='POWER7_v2.1'>
>> +    <model name='power7'>
>>        <vendor name='IBM'/>
>> -      <pvr value='0x003f0201'/>
>> +      <pvr value='0x003f0000'/>
>>      </model>
>>  
>> -    <model name='POWER7_v2.3'>
>> +    <model name='power7+'>
>>        <vendor name='IBM'/>
>> -      <pvr value='0x003f0203'/>
>> +      <pvr value='0x004a0000'/>
>>      </model>
>>  
>> -    <model name='POWER7+_v2.1'>
>> +    <model name='power8'>
>>        <vendor name='IBM'/>
>> -      <pvr value='0x004a0201'/>
>> +      <pvr value='0x004b0000'/>
>>      </model>
>> -
>> -    <model name='POWER8_v1.0'>
>> -      <vendor name='IBM'/>
>> -      <pvr value='0x004b0100'/>
>> -    </model>
> 
> Changing names of all the CPU models in this way is going to break every
> existing XML config on PPC. So NACK to that.

Hi Dan,
One thing to note is that official KVM support on Power processor is from Power8
only. So possibility of breakage is very limited imo.
However I see your point. What is your advise? Should I retain all the existing
processor models in the cpu_map.xml ?


> 
> Regards,
> Daniel
> 


-- 
Regards,
Pradipta Kumar B(bpradip@xxxxxxxxxx)
IBM Systems & Technology Labs,
India.

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]