Re: [Xen-devel] [PATCH 2/8] x86/hyperv: detect absolutely necessary MSRs

[Jan Beulich <jbeulich@xxxxxxxx>]

On 03.01.2020 12:01, Paul Durrant wrote:
> On Sun, 29 Dec 2019 at 18:34, Wei Liu <wl@xxxxxxx> wrote:
>> --- a/xen/arch/x86/guest/hyperv/hyperv.c
>> +++ b/xen/arch/x86/guest/hyperv/hyperv.c
>> @@ -32,6 +32,8 @@ static const struct hypervisor_ops ops = {
>>  const struct hypervisor_ops *__init hyperv_probe(void)
>>  {
>>      uint32_t eax, ebx, ecx, edx;
>> +    uint64_t required_msrs = HV_X64_MSR_HYPERCALL_AVAILABLE |
>> +        HV_X64_MSR_VP_INDEX_AVAILABLE;
>>
>>      cpuid(0x40000000, &eax, &ebx, &ecx, &edx);
>>      if ( !((ebx == 0x7263694d) &&  /* "Micr" */
>> @@ -57,6 +59,15 @@ const struct hypervisor_ops *__init hyperv_probe(void)
>>      ms_hyperv.max_vp_index = eax;
>>      ms_hyperv.max_lp_index = ebx;
>>
>> +    if ( (ms_hyperv.features & required_msrs) != required_msrs )
>> +    {
>> +        /* Oops, required MSRs are not available. Treat this as
>> +         * "Hyper-V is not available".
>> +         */
> 
> That block comment violates default coding style.

And with it corrected
Acked-by: Jan Beulich <jbeulich@xxxxxxxx>

Jan

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