diff options
author | Gleb Natapov <gleb@redhat.com> | 2009-08-24 04:54:26 -0400 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2009-12-03 02:32:08 -0500 |
commit | 680b3648ba89c44ac8d0316f78a0d6e147b88809 (patch) | |
tree | b2b40034c4c33affef8b89822673cca835874651 /arch/x86/kvm/lapic.c | |
parent | eba0226bdfffe262e72b8360e4d0d12070e9a0f0 (diff) |
KVM: Drop kvm->irq_lock lock from irq injection path
The only thing it protects now is interrupt injection into lapic and
this can work lockless. Even now with kvm->irq_lock in place access
to lapic is not entirely serialized since vcpu access doesn't take
kvm->irq_lock.
Signed-off-by: Gleb Natapov <gleb@redhat.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'arch/x86/kvm/lapic.c')
-rw-r--r-- | arch/x86/kvm/lapic.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/arch/x86/kvm/lapic.c b/arch/x86/kvm/lapic.c index df8bcb0f66d8..8787637b1a9c 100644 --- a/arch/x86/kvm/lapic.c +++ b/arch/x86/kvm/lapic.c | |||
@@ -501,9 +501,7 @@ static void apic_send_ipi(struct kvm_lapic *apic) | |||
501 | irq.trig_mode, irq.level, irq.dest_mode, irq.delivery_mode, | 501 | irq.trig_mode, irq.level, irq.dest_mode, irq.delivery_mode, |
502 | irq.vector); | 502 | irq.vector); |
503 | 503 | ||
504 | mutex_lock(&apic->vcpu->kvm->irq_lock); | ||
505 | kvm_irq_delivery_to_apic(apic->vcpu->kvm, apic, &irq); | 504 | kvm_irq_delivery_to_apic(apic->vcpu->kvm, apic, &irq); |
506 | mutex_unlock(&apic->vcpu->kvm->irq_lock); | ||
507 | } | 505 | } |
508 | 506 | ||
509 | static u32 apic_get_tmcct(struct kvm_lapic *apic) | 507 | static u32 apic_get_tmcct(struct kvm_lapic *apic) |