Re: [Xen-devel] [PATCH v2] x86: generic MSRs save/restore

["Liu, Jinsong" <jinsong.liu@xxxxxxxxx>]

Jan Beulich wrote:
>>>> On 16.12.13 at 10:53, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:
>> @@ -467,6 +487,7 @@ int main(int argc, char **argv)
>>          case HVM_SAVE_CODE(VIRIDIAN_VCPU): dump_viridian_vcpu();
>>          break; case HVM_SAVE_CODE(VMCE_VCPU): dump_vmce_vcpu();
>>          break; case HVM_SAVE_CODE(TSC_ADJUST): dump_tsc_adjust();
>> break; +        case HVM_SAVE_CODE(CPU_MSR): dump_cpu_msr(); break;
>>          case HVM_SAVE_CODE(END): break;
>>          default:
>>              printf(" ** Don't understand type %u: skipping\n",
>> ======================= 
>> 
>> w/o the declaration, case HVM_SAVE_CODE(CPU_MSR) compiling fail.
> 
> Right, because you ought to use CPU_MSR_CODE here, which the
> public header does define.
> 
> Jan

Isn't it ugly? not unified style w/ other typecodes.
Why not simply add declaration at Xen side?

Thanks,
Jinsong
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel