From: Avi Kivity <[email protected]>

This is an ordinary modrm source or destination; use the standard structure
representing it.

Signed-off-by: Avi Kivity <[email protected]>

diff --git a/arch/x86/kvm/emulate.c b/arch/x86/kvm/emulate.c
index c28323d..9883192 100644
--- a/arch/x86/kvm/emulate.c
+++ b/arch/x86/kvm/emulate.c
@@ -2211,8 +2211,8 @@ static struct opcode twobyte_table[256] = {
        /* 0x10 - 0x1F */
        N, N, N, N, N, N, N, N, D(ImplicitOps | ModRM), N, N, N, N, N, N, N,
        /* 0x20 - 0x2F */
-       D(ModRM | DstMem | Priv | Op3264), D(ModRM | Priv | Op3264),
-       D(ModRM | SrcMem | Priv | Op3264), D(ModRM | Priv | Op3264),
+       D(ModRM | DstMem | Priv | Op3264), D(ModRM | DstMem | Priv | Op3264),
+       D(ModRM | SrcMem | Priv | Op3264), D(ModRM | SrcMem | Priv | Op3264),
        N, N, N, N,
        N, N, N, N, N, N, N, N,
        /* 0x30 - 0x3F */
@@ -3249,8 +3249,7 @@ twobyte_insn:
                        emulate_ud(ctxt);
                        goto done;
                }
-               ops->get_dr(c->modrm_reg, &c->regs[c->modrm_rm], ctxt->vcpu);
-               c->dst.type = OP_NONE;  /* no writeback */
+               ops->get_dr(c->modrm_reg, &c->dst.val, ctxt->vcpu);
                break;
        case 0x22: /* mov reg, cr */
                if (ops->set_cr(c->modrm_reg, c->src.val, ctxt->vcpu)) {
@@ -3266,7 +3265,7 @@ twobyte_insn:
                        goto done;
                }
 
-               if (ops->set_dr(c->modrm_reg, c->regs[c->modrm_rm] &
+               if (ops->set_dr(c->modrm_reg, c->src.val &
                                ((ctxt->mode == X86EMUL_MODE_PROT64) ?
                                 ~0ULL : ~0U), ctxt->vcpu) < 0) {
                        /* #UD condition is already handled by the code above */
--
To unsubscribe from this list: send the line "unsubscribe kvm-commits" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to