Re: [Xen-devel] [PATCH] VMX: PLATFORM_INFO MSR is r/o

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 22/09/17 10:07, Jan Beulich wrote:
> Therefore all write attempts should produce #GP, just like on real
> hardware.
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Sergey has already posted a patch series to fix this.

What is the benefit of this version?  As far as I can tell, it isn't as 
complete as his work, because it doesn't fix the problems with this MSR 
on AMD systems.
~Andrew

> --- a/xen/arch/x86/hvm/vmx/vmx.c
> +++ b/xen/arch/x86/hvm/vmx/vmx.c
> @@ -3138,10 +3138,7 @@ static int vmx_msr_write_intercept(unsig
>           break;
>   
>       case MSR_INTEL_PLATFORM_INFO:
> -        if ( msr_content ||
> -             rdmsr_safe(MSR_INTEL_PLATFORM_INFO, msr_content) )
> -            goto gp_fault;
> -        break;
> +        goto gp_fault;
>   
>       case MSR_INTEL_MISC_FEATURES_ENABLES:
>       {
>
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> https://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel