KVM: X86: Propagate fetch faults
authorJoerg Roedel <joerg.roedel@amd.com>
Fri, 10 Sep 2010 15:30:56 +0000 (17:30 +0200)
committerAvi Kivity <avi@redhat.com>
Sun, 24 Oct 2010 08:52:41 +0000 (10:52 +0200)
KVM currently ignores fetch faults in the instruction
emulator. With nested-npt we could have such faults. This
patch adds the code to handle these.

Signed-off-by: Joerg Roedel <joerg.roedel@amd.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/kvm/emulate.c
arch/x86/kvm/x86.c

index 2b08b78b6cab4c02774d2bba12ccba4f33c9fc99..aead72e141b48bbc1c3cdde3a6f90d84410bd376 100644 (file)
@@ -1198,6 +1198,9 @@ static int emulate_popf(struct x86_emulate_ctxt *ctxt,
        *(unsigned long *)dest =
                (ctxt->eflags & ~change_mask) | (val & change_mask);
 
+       if (rc == X86EMUL_PROPAGATE_FAULT)
+               emulate_pf(ctxt);
+
        return rc;
 }
 
index 0281d920e9edb7fcf0bc43fa7415b1f00c6ff63d..3101060033ae33c0e506f1cdfbdadbbb17800246 100644 (file)
@@ -4247,6 +4247,9 @@ int emulate_instruction(struct kvm_vcpu *vcpu,
                vcpu->arch.emulate_ctxt.perm_ok = false;
 
                r = x86_decode_insn(&vcpu->arch.emulate_ctxt);
+               if (r == X86EMUL_PROPAGATE_FAULT)
+                       goto done;
+
                trace_kvm_emulate_insn_start(vcpu);
 
                /* Only allow emulation of specific instructions on #UD
@@ -4305,6 +4308,7 @@ restart:
                return handle_emulation_failure(vcpu);
        }
 
+done:
        if (vcpu->arch.emulate_ctxt.exception >= 0) {
                inject_emulated_exception(vcpu);
                r = EMULATE_DONE;