KVM: inject #UD in 64bit mode from instruction that are not valid there
authorGleb Natapov <gleb@redhat.com>
Thu, 11 Feb 2010 12:43:14 +0000 (14:43 +0200)
committerAvi Kivity <avi@redhat.com>
Sun, 25 Apr 2010 09:27:40 +0000 (12:27 +0300)
Some instruction are obsolete in a long mode. Inject #UD.

Signed-off-by: Gleb Natapov <gleb@redhat.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/kvm/emulate.c

index 4dade6ac08276537b07824ab7c23224d93946905..96d4bef06e1405a7ab1d8adacc350f75d849d5a5 100644 (file)
@@ -1015,11 +1015,6 @@ done_prefixes:
                }
        }
 
-       if (mode == X86EMUL_MODE_PROT64 && (c->d & No64)) {
-               kvm_report_emulation_failure(ctxt->vcpu, "invalid x86/64 instruction");
-               return -1;
-       }
-
        if (c->d & Group) {
                group = c->d & GroupMask;
                c->modrm = insn_fetch(u8, 1, c->eip);
@@ -1828,6 +1823,11 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt, struct x86_emulate_ops *ops)
        memcpy(c->regs, ctxt->vcpu->arch.regs, sizeof c->regs);
        saved_eip = c->eip;
 
+       if (ctxt->mode == X86EMUL_MODE_PROT64 && (c->d & No64)) {
+               kvm_queue_exception(ctxt->vcpu, UD_VECTOR);
+               goto done;
+       }
+
        /* LOCK prefix is allowed only with some instructions */
        if (c->lock_prefix && !(c->d & Lock)) {
                kvm_queue_exception(ctxt->vcpu, UD_VECTOR);