KVM: x86 emulator: set RFLAGS outside x86 emulator code
authorGleb Natapov <gleb@redhat.com>
Wed, 28 Apr 2010 16:15:40 +0000 (19:15 +0300)
committerAvi Kivity <avi@redhat.com>
Sun, 1 Aug 2010 07:35:35 +0000 (10:35 +0300)
Removes the need for set_flags() callback.

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

index b7e00cb21c6eac57460e826b5ee99ea5fd8206a3..a87d95f09572a210c38502ac0ee5a05d99626f54 100644 (file)
@@ -142,7 +142,6 @@ struct x86_emulate_ops {
        ulong (*get_cr)(int cr, struct kvm_vcpu *vcpu);
        int (*set_cr)(int cr, ulong val, struct kvm_vcpu *vcpu);
        int (*cpl)(struct kvm_vcpu *vcpu);
-       void (*set_rflags)(struct kvm_vcpu *vcpu, unsigned long rflags);
        int (*get_dr)(int dr, unsigned long *dest, struct kvm_vcpu *vcpu);
        int (*set_dr)(int dr, unsigned long value, struct kvm_vcpu *vcpu);
        int (*set_msr)(struct kvm_vcpu *vcpu, u32 msr_index, u64 data);
index 437f31bcffeaf8849008e7b39ddcad107e5ff623..291e220c69a5504328d96689dac9a553af9ff76d 100644 (file)
@@ -3034,7 +3034,6 @@ writeback:
        /* Commit shadow register state. */
        memcpy(ctxt->vcpu->arch.regs, c->regs, sizeof c->regs);
        ctxt->eip = c->eip;
-       ops->set_rflags(ctxt->vcpu, ctxt->eflags);
 
 done:
        return (rc == X86EMUL_UNHANDLEABLE) ? -1 : 0;
index 8f45cc712dda2775dfc9c71f712e819f3caf3504..04ca343ee512198cd90d1c53cbb3f54e13e2a339 100644 (file)
@@ -3806,11 +3806,6 @@ static void emulator_set_segment_selector(u16 sel, int seg,
        kvm_set_segment(vcpu, &kvm_seg, seg);
 }
 
-static void emulator_set_rflags(struct kvm_vcpu *vcpu, unsigned long rflags)
-{
-       kvm_x86_ops->set_rflags(vcpu, rflags);
-}
-
 static struct x86_emulate_ops emulate_ops = {
        .read_std            = kvm_read_guest_virt_system,
        .write_std           = kvm_write_guest_virt_system,
@@ -3829,7 +3824,6 @@ static struct x86_emulate_ops emulate_ops = {
        .get_cr              = emulator_get_cr,
        .set_cr              = emulator_set_cr,
        .cpl                 = emulator_get_cpl,
-       .set_rflags          = emulator_set_rflags,
        .get_dr              = emulator_get_dr,
        .set_dr              = emulator_set_dr,
        .set_msr             = kvm_set_msr,
@@ -3941,6 +3935,7 @@ restart:
 
        shadow_mask = vcpu->arch.emulate_ctxt.interruptibility;
        kvm_x86_ops->set_interrupt_shadow(vcpu, shadow_mask);
+       kvm_x86_ops->set_rflags(vcpu, vcpu->arch.emulate_ctxt.eflags);
        kvm_rip_write(vcpu, vcpu->arch.emulate_ctxt.eip);
 
        if (vcpu->arch.pio.count) {