diff options
| author | Joerg Roedel | 2010-08-02 16:46:44 +0200 |
|---|---|---|
| committer | Avi Kivity | 2010-10-24 10:50:32 +0200 |
| commit | 52c65a30a5c6f31cd66dba57c22d18cafa5e327f (patch) | |
| tree | a2bb3effc2bfac345335bfecbdc9be52dafc7c00 /arch/x86/kvm/svm.c | |
| parent | KVM: MMU: mark page dirty only when page is really written (diff) | |
| download | kernel-qcow2-linux-52c65a30a5c6f31cd66dba57c22d18cafa5e327f.tar.gz kernel-qcow2-linux-52c65a30a5c6f31cd66dba57c22d18cafa5e327f.tar.xz kernel-qcow2-linux-52c65a30a5c6f31cd66dba57c22d18cafa5e327f.zip | |
KVM: SVM: Check for nested vmrun intercept before emulating vmrun
This patch lets the nested vmrun fail if the L1 hypervisor
has not intercepted vmrun. This fixes the "vmrun intercept
check" unit test.
Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'arch/x86/kvm/svm.c')
| -rw-r--r-- | arch/x86/kvm/svm.c | 19 |
1 files changed, 19 insertions, 0 deletions
diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c index 116e0341bf4c..a0e5c7e26104 100644 --- a/arch/x86/kvm/svm.c +++ b/arch/x86/kvm/svm.c @@ -2014,6 +2014,14 @@ static bool nested_svm_vmrun_msrpm(struct vcpu_svm *svm) return true; } +static bool nested_vmcb_checks(struct vmcb *vmcb) +{ + if ((vmcb->control.intercept & (1ULL << INTERCEPT_VMRUN)) == 0) + return false; + + return true; +} + static bool nested_svm_vmrun(struct vcpu_svm *svm) { struct vmcb *nested_vmcb; @@ -2028,6 +2036,17 @@ static bool nested_svm_vmrun(struct vcpu_svm *svm) if (!nested_vmcb) return false; + if (!nested_vmcb_checks(nested_vmcb)) { + nested_vmcb->control.exit_code = SVM_EXIT_ERR; + nested_vmcb->control.exit_code_hi = 0; + nested_vmcb->control.exit_info_1 = 0; + nested_vmcb->control.exit_info_2 = 0; + + nested_svm_unmap(page); + + return false; + } + trace_kvm_nested_vmrun(svm->vmcb->save.rip - 3, vmcb_gpa, nested_vmcb->save.rip, nested_vmcb->control.int_ctl, |