On Wed, Mar 27, 2019 at 05:56:49PM -0400, Krish Sadhukhan wrote:
It'd be nice to have a short blurb here stating that the intent is to
match the organization in the SDM.
> Signed-off-by: Krish Sadhukhan <krish.sadhukhan@xxxxxxxxxx>
> Suggested-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>
> ---
> arch/x86/kvm/vmx/nested.c | 25 +++++++++++++++++++------
> 1 file changed, 19 insertions(+), 6 deletions(-)
>
> diff --git a/arch/x86/kvm/vmx/nested.c b/arch/x86/kvm/vmx/nested.c
> index b8bd449350b4..87449b67eaf2 100644
> --- a/arch/x86/kvm/vmx/nested.c
> +++ b/arch/x86/kvm/vmx/nested.c
> @@ -2674,6 +2674,24 @@ static int nested_vmx_check_vmcs_link_ptr(struct kvm_vcpu *vcpu,
> return r;
> }
>
> +/*
> + * Checks related to Control Registers, Debug Registers and MSRs in
> + * Guest State Area.
> + */
> +static int nested_check_guest_ctrl_dbg_regs_msrs(struct kvm_vcpu *vcpu,
> + struct vmcs12 *vmcs12)
I like the idea of shortening the function name, but "ctrl_dbg_regs" is a
bit confusing, e.g. one might think "ctrl" in this context refers to the
VMCS's control bit/field.
What about nested_check_guest_crs_drs_msrs()?
> +{
> + if (!nested_guest_cr0_valid(vcpu, vmcs12->guest_cr0) ||
> + !nested_guest_cr4_valid(vcpu, vmcs12->guest_cr4))
> + return 1;
> +
> + if ((vmcs12->vm_entry_controls & VM_ENTRY_LOAD_IA32_PAT) &&
> + !kvm_mtrr_valid(vcpu, MSR_IA32_CR_PAT, vmcs12->guest_ia32_pat))
> + return 1;
> +
> + return 0;
> +}
> +
> static int nested_vmx_check_vmentry_postreqs(struct kvm_vcpu *vcpu,
> struct vmcs12 *vmcs12,
> u32 *exit_qual)
> @@ -2682,12 +2700,7 @@ static int nested_vmx_check_vmentry_postreqs(struct kvm_vcpu *vcpu,
>
> *exit_qual = ENTRY_FAIL_DEFAULT;
>
> - if (!nested_guest_cr0_valid(vcpu, vmcs12->guest_cr0) ||
> - !nested_guest_cr4_valid(vcpu, vmcs12->guest_cr4))
> - return 1;
> -
> - if ((vmcs12->vm_entry_controls & VM_ENTRY_LOAD_IA32_PAT) &&
> - !kvm_mtrr_valid(vcpu, MSR_IA32_CR_PAT, vmcs12->guest_ia32_pat))
> + if (nested_check_guest_ctrl_dbg_regs_msrs(vcpu, vmcs12))
> return 1;
>
> if (nested_vmx_check_vmcs_link_ptr(vcpu, vmcs12)) {
> --
> 2.17.2
>
