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

Re: [Xen-devel] [PATCH 03/15] x86/cpu/vpmu: Add Hygon Dhyana support for vPMU



On 2019/1/15 0:47, Jan Beulich wrote:
>>>> On 20.12.18 at 14:12, <puwen@xxxxxxxx> wrote:
>> --- a/xen/arch/x86/cpu/vpmu.c
>> +++ b/xen/arch/x86/cpu/vpmu.c
>> @@ -473,6 +473,7 @@ static int vpmu_arch_initialise(struct vcpu *v)
>>   
>>       switch ( vendor )
>>       {
>> +    case X86_VENDOR_HYGON:
>>       case X86_VENDOR_AMD:
>>           ret = svm_vpmu_initialise(v);
>>           break;
>> @@ -890,6 +891,7 @@ static int __init vpmu_init(void)
>>   
>>       switch ( vendor )
>>       {
>> +    case X86_VENDOR_HYGON:
>>       case X86_VENDOR_AMD:
>>           if ( amd_vpmu_init() )
>>              vpmu_mode = XENPMU_MODE_OFF;
> 
> Here and everywhere else, may I ask that you do your insertions
> below the respective AMD ones instead of above?

In all the switch cases, if Hygon directly followed AMD's cases, the
insertions are placed above the respective AMD ones.

If Hygon used its own cases, such as in calc_ler_msr, the new cases
are placed below the AMD ones.

-- 
Regards,
Pu Wen

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.