>>> On 09.06.16 at 12:14, <euan.har...@citrix.com> wrote:
> --- a/xen/arch/x86/hvm/vmx/vvmx.c
> +++ b/xen/arch/x86/hvm/vmx/vvmx.c
> @@ -1820,11 +1820,20 @@ int nvmx_msr_read_intercept(unsigned int msr, u64 
> *msr_content)
>          return 0;
>  
>      /*
> -     * Those MSRs are available only when bit 55 of
> -     * MSR_IA32_VMX_BASIC is set.
> +     * These MSRs are only available when flags in other MSRs are set.
> +     * These prerequisites are listed in the Intel 64 and IA-32
> +     * Architectures Software Developer’s Manual, Vol 3, Appendix A.
>       */
> -    switch ( msr )
> -    {
> +    switch ( msr ) { case MSR_IA32_VMX_PROCBASED_CTLS2:

I hope you didn't really mean to produce a garbled line like this?
With proper formatting restored
Reviewed-by: Jan Beulich <jbeul...@suse.com>

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

Reply via email to