KVM: x86 emulator: push segment override out of decode_modrm()
authorAvi Kivity <avi@redhat.com>
Sun, 1 Aug 2010 09:07:29 +0000 (12:07 +0300)
committerAvi Kivity <avi@redhat.com>
Sun, 24 Oct 2010 08:50:33 +0000 (10:50 +0200)
Let it compute modrm_seg instead, and have the caller apply it.

Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/include/asm/kvm_emulate.h
arch/x86/kvm/emulate.c

index 8762411fe9bb4621fee65e7e1a9d906eca8a8514..cbdf76722d7dc01faf531392ebe33035fe83a4a9 100644 (file)
@@ -198,6 +198,7 @@ struct decode_cache {
        u8 modrm_mod;
        u8 modrm_reg;
        u8 modrm_rm;
+       u8 modrm_seg;
        u8 use_modrm_ea;
        bool rip_relative;
        unsigned long modrm_ea;
index 760e2b030e685a22149ca0e298d784bc89b7d742..471f12ae29cf7eab2585509d1750e152947f40f5 100644 (file)
@@ -593,6 +593,7 @@ static int decode_modrm(struct x86_emulate_ctxt *ctxt,
        c->modrm_rm |= (c->modrm & 0x07);
        c->modrm_ea = 0;
        c->use_modrm_ea = 1;
+       c->modrm_seg = VCPU_SREG_DS;
 
        if (c->modrm_mod == 3) {
                c->modrm_ptr = decode_register(c->modrm_rm,
@@ -649,8 +650,7 @@ static int decode_modrm(struct x86_emulate_ctxt *ctxt,
                }
                if (c->modrm_rm == 2 || c->modrm_rm == 3 ||
                    (c->modrm_rm == 6 && c->modrm_mod != 0))
-                       if (!c->has_seg_override)
-                               set_seg_override(c, VCPU_SREG_SS);
+                       c->modrm_seg = VCPU_SREG_SS;
                c->modrm_ea = (u16)c->modrm_ea;
        } else {
                /* 32/64-bit ModR/M decode. */
@@ -2405,9 +2405,11 @@ done_prefixes:
                c->op_bytes = 8;
 
        /* ModRM and SIB bytes. */
-       if (c->d & ModRM)
+       if (c->d & ModRM) {
                rc = decode_modrm(ctxt, ops);
-       else if (c->d & MemAbs)
+               if (!c->has_seg_override)
+                       set_seg_override(c, c->modrm_seg);
+       } else if (c->d & MemAbs)
                rc = decode_abs(ctxt, ops);
        if (rc != X86EMUL_CONTINUE)
                goto done;