Commit 51187683 authored by Takuya Yoshikawa's avatar Takuya Yoshikawa Committed by Avi Kivity

KVM: x86 emulator: Rename emulate_grpX() to em_grpX()

The prototypes are changed appropriately.

We also replaces "goto grp45;" with simple em_grp45() call.
Signed-off-by: default avatarTakuya Yoshikawa <yoshikawa.takuya@oss.ntt.co.jp>
Signed-off-by: default avatarAvi Kivity <avi@redhat.com>
parent 3b9be3bf
...@@ -1687,14 +1687,14 @@ static inline int emulate_iret(struct x86_emulate_ctxt *ctxt, ...@@ -1687,14 +1687,14 @@ static inline int emulate_iret(struct x86_emulate_ctxt *ctxt,
} }
} }
static inline int emulate_grp1a(struct x86_emulate_ctxt *ctxt) static int em_grp1a(struct x86_emulate_ctxt *ctxt)
{ {
struct decode_cache *c = &ctxt->decode; struct decode_cache *c = &ctxt->decode;
return emulate_pop(ctxt, &c->dst.val, c->dst.bytes); return emulate_pop(ctxt, &c->dst.val, c->dst.bytes);
} }
static inline void emulate_grp2(struct x86_emulate_ctxt *ctxt) static int em_grp2(struct x86_emulate_ctxt *ctxt)
{ {
struct decode_cache *c = &ctxt->decode; struct decode_cache *c = &ctxt->decode;
switch (c->modrm_reg) { switch (c->modrm_reg) {
...@@ -1721,10 +1721,10 @@ static inline void emulate_grp2(struct x86_emulate_ctxt *ctxt) ...@@ -1721,10 +1721,10 @@ static inline void emulate_grp2(struct x86_emulate_ctxt *ctxt)
emulate_2op_SrcB("sar", c->src, c->dst, ctxt->eflags); emulate_2op_SrcB("sar", c->src, c->dst, ctxt->eflags);
break; break;
} }
return X86EMUL_CONTINUE;
} }
static inline int emulate_grp3(struct x86_emulate_ctxt *ctxt, static int em_grp3(struct x86_emulate_ctxt *ctxt)
struct x86_emulate_ops *ops)
{ {
struct decode_cache *c = &ctxt->decode; struct decode_cache *c = &ctxt->decode;
unsigned long *rax = &c->regs[VCPU_REGS_RAX]; unsigned long *rax = &c->regs[VCPU_REGS_RAX];
...@@ -1763,7 +1763,7 @@ static inline int emulate_grp3(struct x86_emulate_ctxt *ctxt, ...@@ -1763,7 +1763,7 @@ static inline int emulate_grp3(struct x86_emulate_ctxt *ctxt,
return X86EMUL_CONTINUE; return X86EMUL_CONTINUE;
} }
static int emulate_grp45(struct x86_emulate_ctxt *ctxt) static int em_grp45(struct x86_emulate_ctxt *ctxt)
{ {
struct decode_cache *c = &ctxt->decode; struct decode_cache *c = &ctxt->decode;
int rc = X86EMUL_CONTINUE; int rc = X86EMUL_CONTINUE;
...@@ -1793,8 +1793,7 @@ static int emulate_grp45(struct x86_emulate_ctxt *ctxt) ...@@ -1793,8 +1793,7 @@ static int emulate_grp45(struct x86_emulate_ctxt *ctxt)
return rc; return rc;
} }
static inline int emulate_grp9(struct x86_emulate_ctxt *ctxt, static int em_grp9(struct x86_emulate_ctxt *ctxt)
struct x86_emulate_ops *ops)
{ {
struct decode_cache *c = &ctxt->decode; struct decode_cache *c = &ctxt->decode;
u64 old = c->dst.orig_val64; u64 old = c->dst.orig_val64;
...@@ -3916,7 +3915,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt) ...@@ -3916,7 +3915,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt)
break; break;
} }
case 0x8f: /* pop (sole member of Grp1a) */ case 0x8f: /* pop (sole member of Grp1a) */
rc = emulate_grp1a(ctxt); rc = em_grp1a(ctxt);
break; break;
case 0x90 ... 0x97: /* nop / xchg reg, rax */ case 0x90 ... 0x97: /* nop / xchg reg, rax */
if (c->dst.addr.reg == &c->regs[VCPU_REGS_RAX]) if (c->dst.addr.reg == &c->regs[VCPU_REGS_RAX])
...@@ -3932,7 +3931,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt) ...@@ -3932,7 +3931,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt)
case 0xa8 ... 0xa9: /* test ax, imm */ case 0xa8 ... 0xa9: /* test ax, imm */
goto test; goto test;
case 0xc0 ... 0xc1: case 0xc0 ... 0xc1:
emulate_grp2(ctxt); rc = em_grp2(ctxt);
break; break;
case 0xc3: /* ret */ case 0xc3: /* ret */
c->dst.type = OP_REG; c->dst.type = OP_REG;
...@@ -3967,11 +3966,11 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt) ...@@ -3967,11 +3966,11 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt)
rc = emulate_iret(ctxt, ops); rc = emulate_iret(ctxt, ops);
break; break;
case 0xd0 ... 0xd1: /* Grp2 */ case 0xd0 ... 0xd1: /* Grp2 */
emulate_grp2(ctxt); rc = em_grp2(ctxt);
break; break;
case 0xd2 ... 0xd3: /* Grp2 */ case 0xd2 ... 0xd3: /* Grp2 */
c->src.val = c->regs[VCPU_REGS_RCX]; c->src.val = c->regs[VCPU_REGS_RCX];
emulate_grp2(ctxt); rc = em_grp2(ctxt);
break; break;
case 0xe0 ... 0xe2: /* loop/loopz/loopnz */ case 0xe0 ... 0xe2: /* loop/loopz/loopnz */
register_address_increment(c, &c->regs[VCPU_REGS_RCX], -1); register_address_increment(c, &c->regs[VCPU_REGS_RCX], -1);
...@@ -4040,7 +4039,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt) ...@@ -4040,7 +4039,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt)
ctxt->eflags ^= EFLG_CF; ctxt->eflags ^= EFLG_CF;
break; break;
case 0xf6 ... 0xf7: /* Grp3 */ case 0xf6 ... 0xf7: /* Grp3 */
rc = emulate_grp3(ctxt, ops); rc = em_grp3(ctxt);
break; break;
case 0xf8: /* clc */ case 0xf8: /* clc */
ctxt->eflags &= ~EFLG_CF; ctxt->eflags &= ~EFLG_CF;
...@@ -4071,13 +4070,13 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt) ...@@ -4071,13 +4070,13 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt)
ctxt->eflags |= EFLG_DF; ctxt->eflags |= EFLG_DF;
break; break;
case 0xfe: /* Grp4 */ case 0xfe: /* Grp4 */
grp45: rc = em_grp45(ctxt);
rc = emulate_grp45(ctxt);
break; break;
case 0xff: /* Grp5 */ case 0xff: /* Grp5 */
if (c->modrm_reg == 5) if (c->modrm_reg == 5)
goto jump_far; goto jump_far;
goto grp45; rc = em_grp45(ctxt);
break;
default: default:
goto cannot_emulate; goto cannot_emulate;
} }
...@@ -4344,7 +4343,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt) ...@@ -4344,7 +4343,7 @@ x86_emulate_insn(struct x86_emulate_ctxt *ctxt)
(u64) c->src.val; (u64) c->src.val;
break; break;
case 0xc7: /* Grp9 (cmpxchg8b) */ case 0xc7: /* Grp9 (cmpxchg8b) */
rc = emulate_grp9(ctxt, ops); rc = em_grp9(ctxt);
break; break;
default: default:
goto cannot_emulate; goto cannot_emulate;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment