diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2014-03-18 05:47:17 -0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2014-03-21 05:20:10 -0400 |
commit | 0bc830b05c667218d703f2026ec866c49df974fc (patch) | |
tree | 7360f66e70717eb144a16444c30e49eade21cdea /virt | |
parent | 0b10a1c87a2b0fb459baaefba9cb163dbb8d3344 (diff) |
KVM: ioapic: clear IRR for edge-triggered interrupts at delivery
This ensures that IRR bits are set in the KVM_GET_IRQCHIP result only if
the interrupt is still sitting in the IOAPIC. After the next patches, it
avoids spurious reinjection of the interrupt when KVM_SET_IRQCHIP is
called.
Reviewed-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'virt')
-rw-r--r-- | virt/kvm/ioapic.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/virt/kvm/ioapic.c b/virt/kvm/ioapic.c index 0b4914147b9d..25e16a6898ed 100644 --- a/virt/kvm/ioapic.c +++ b/virt/kvm/ioapic.c | |||
@@ -288,6 +288,9 @@ static int ioapic_service(struct kvm_ioapic *ioapic, int irq, bool line_status) | |||
288 | irqe.level = 1; | 288 | irqe.level = 1; |
289 | irqe.shorthand = 0; | 289 | irqe.shorthand = 0; |
290 | 290 | ||
291 | if (irqe.trig_mode == IOAPIC_EDGE_TRIG) | ||
292 | ioapic->irr &= ~(1 << irq); | ||
293 | |||
291 | if (irq == RTC_GSI && line_status) { | 294 | if (irq == RTC_GSI && line_status) { |
292 | BUG_ON(ioapic->rtc_status.pending_eoi != 0); | 295 | BUG_ON(ioapic->rtc_status.pending_eoi != 0); |
293 | ret = kvm_irq_delivery_to_apic(ioapic->kvm, NULL, &irqe, | 296 | ret = kvm_irq_delivery_to_apic(ioapic->kvm, NULL, &irqe, |