KVM: VMX: Fix locking order in handle_invalid_guest_state
authorJan Kiszka <jan.kiszka@web.de>
Wed, 22 Jul 2009 21:53:26 +0000 (23:53 +0200)
committerAvi Kivity <avi@redhat.com>
Wed, 5 Aug 2009 10:59:44 +0000 (13:59 +0300)
Release and re-acquire preemption and IRQ lock in the same order as
vcpu_enter_guest does.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
arch/x86/kvm/vmx.c

index 356a0ce85c68d92427260f8478b88a9036f41ef2..6bf58c083f0a2108609404ffaa193685916cdf6a 100644 (file)
@@ -3157,8 +3157,8 @@ static void handle_invalid_guest_state(struct kvm_vcpu *vcpu,
        struct vcpu_vmx *vmx = to_vmx(vcpu);
        enum emulation_result err = EMULATE_DONE;
 
-       preempt_enable();
        local_irq_enable();
+       preempt_enable();
 
        while (!guest_state_valid(vcpu)) {
                err = emulate_instruction(vcpu, kvm_run, 0, 0, 0);
@@ -3177,8 +3177,8 @@ static void handle_invalid_guest_state(struct kvm_vcpu *vcpu,
                        schedule();
        }
 
-       local_irq_disable();
        preempt_disable();
+       local_irq_disable();
 
        vmx->invalid_state_emulation_result = err;
 }