KVM: x86: Use a typedef for fastop functions
authorSean Christopherson <sean.j.christopherson@intel.com>
Wed, 22 Jan 2020 04:43:39 +0000 (20:43 -0800)
committerPaolo Bonzini <pbonzini@redhat.com>
Mon, 27 Jan 2020 19:00:11 +0000 (20:00 +0100)
Add a typedef to for the fastop function prototype to make the code more
readable.

No functional change intended.

Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com>
Reviewed-by: Miaohe Lin <linmiaohe@huawei.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/emulate.c

index 0accce94f660816e50f5e87f0cc277d1d7dd1df9..ddbc61984227c3dbee763ed3759e9a5351f32606 100644 (file)
@@ -311,7 +311,9 @@ static void invalidate_registers(struct x86_emulate_ctxt *ctxt)
 #define ON64(x)
 #endif
 
-static int fastop(struct x86_emulate_ctxt *ctxt, void (*fop)(struct fastop *));
+typedef void (*fastop_t)(struct fastop *);
+
+static int fastop(struct x86_emulate_ctxt *ctxt, fastop_t fop);
 
 #define __FOP_FUNC(name) \
        ".align " __stringify(FASTOP_SIZE) " \n\t" \
@@ -5502,7 +5504,7 @@ static void fetch_possible_mmx_operand(struct operand *op)
                read_mmx_reg(&op->mm_val, op->addr.mm);
 }
 
-static int fastop(struct x86_emulate_ctxt *ctxt, void (*fop)(struct fastop *))
+static int fastop(struct x86_emulate_ctxt *ctxt, fastop_t fop)
 {
        ulong flags = (ctxt->eflags & EFLAGS_MASK) | X86_EFLAGS_IF;
 
@@ -5680,12 +5682,10 @@ special_insn:
                ctxt->eflags &= ~X86_EFLAGS_RF;
 
        if (ctxt->execute) {
-               if (ctxt->d & Fastop) {
-                       void (*fop)(struct fastop *) = (void *)ctxt->execute;
-                       rc = fastop(ctxt, fop);
-               } else {
+               if (ctxt->d & Fastop)
+                       rc = fastop(ctxt, (fastop_t)ctxt->execute);
+               else
                        rc = ctxt->execute(ctxt);
-               }
                if (rc != X86EMUL_CONTINUE)
                        goto done;
                goto writeback;