KVM: ioapic: reinject pending interrupts on KVM_SET_IRQCHIP
authorPaolo Bonzini <pbonzini@redhat.com>
Tue, 18 Mar 2014 10:39:23 +0000 (11:39 +0100)
committerPaolo Bonzini <pbonzini@redhat.com>
Fri, 21 Mar 2014 14:10:43 +0000 (15:10 +0100)
After the previous patches, an interrupt whose bit is set in the IRR
register will never be in the LAPIC's IRR and has never been injected
on the migration source.  So inject it on the destination.

This fixes migration of Windows guests without HPET (they use the RTC
to trigger the scheduler tick, and lose it after migration).

Reviewed-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
virt/kvm/ioapic.c

index 270f7fe73f39bcc9ec7d7a48687b3a4c5a38c3e0..d4b601547f1f13c8f336b2352b46d087bb94e7d6 100644 (file)
@@ -212,6 +212,18 @@ out:
        return ret;
 }
 
+static void kvm_ioapic_inject_all(struct kvm_ioapic *ioapic, unsigned long irr)
+{
+       u32 idx;
+
+       rtc_irq_eoi_tracking_reset(ioapic);
+       for_each_set_bit(idx, &irr, IOAPIC_NUM_PINS)
+               ioapic_set_irq(ioapic, idx, 1, true);
+
+       kvm_rtc_eoi_tracking_restore_all(ioapic);
+}
+
+
 static void update_handled_vectors(struct kvm_ioapic *ioapic)
 {
        DECLARE_BITMAP(handled_vectors, 256);
@@ -612,9 +624,10 @@ int kvm_set_ioapic(struct kvm *kvm, struct kvm_ioapic_state *state)
 
        spin_lock(&ioapic->lock);
        memcpy(ioapic, state, sizeof(struct kvm_ioapic_state));
+       ioapic->irr = 0;
        update_handled_vectors(ioapic);
        kvm_vcpu_request_scan_ioapic(kvm);
-       kvm_rtc_eoi_tracking_restore_all(ioapic);
+       kvm_ioapic_inject_all(ioapic, state->irr);
        spin_unlock(&ioapic->lock);
        return 0;
 }