kvm: irqchip: skip update msi when disabled
It's possible that one device kept its irqfd/virq there even when
MSI/MSIX was disabled globally for that device. One example is
virtio-net-pci (see commit f1d0f15a6
and virtio_pci_vq_vector_mask()).
It is used as a fast path to avoid allocate/release irqfd/virq
frequently when guest enables/disables MSIX.
However, this fast path brought a problem to msi_route_list, that the
device MSIRouteEntry is still dangling there even if MSIX disabled -
then we cannot know which message to fetch, even if we can, the messages
are meaningless. In this case, we can just simply ignore this entry.
It's safe, since when MSIX is enabled again, we'll rebuild them no
matter what.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1448813
Signed-off-by: Peter Xu <peterx@redhat.com>
Message-Id: <1494309644-18743-4-git-send-email-peterx@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
993b1f4b2c
commit
fd56356422
|
@ -43,6 +43,7 @@
|
||||||
#include "standard-headers/asm-x86/hyperv.h"
|
#include "standard-headers/asm-x86/hyperv.h"
|
||||||
#include "hw/pci/pci.h"
|
#include "hw/pci/pci.h"
|
||||||
#include "hw/pci/msi.h"
|
#include "hw/pci/msi.h"
|
||||||
|
#include "hw/pci/msix.h"
|
||||||
#include "migration/blocker.h"
|
#include "migration/blocker.h"
|
||||||
#include "exec/memattrs.h"
|
#include "exec/memattrs.h"
|
||||||
#include "trace.h"
|
#include "trace.h"
|
||||||
|
@ -3510,12 +3511,17 @@ static void kvm_update_msi_routes_all(void *private, bool global,
|
||||||
int cnt = 0;
|
int cnt = 0;
|
||||||
MSIRouteEntry *entry;
|
MSIRouteEntry *entry;
|
||||||
MSIMessage msg;
|
MSIMessage msg;
|
||||||
|
PCIDevice *dev;
|
||||||
|
|
||||||
/* TODO: explicit route update */
|
/* TODO: explicit route update */
|
||||||
QLIST_FOREACH(entry, &msi_route_list, list) {
|
QLIST_FOREACH(entry, &msi_route_list, list) {
|
||||||
cnt++;
|
cnt++;
|
||||||
msg = pci_get_msi_message(entry->dev, entry->vector);
|
dev = entry->dev;
|
||||||
kvm_irqchip_update_msi_route(kvm_state, entry->virq,
|
if (!msix_enabled(dev) && !msi_enabled(dev)) {
|
||||||
msg, entry->dev);
|
continue;
|
||||||
|
}
|
||||||
|
msg = pci_get_msi_message(dev, entry->vector);
|
||||||
|
kvm_irqchip_update_msi_route(kvm_state, entry->virq, msg, dev);
|
||||||
}
|
}
|
||||||
kvm_irqchip_commit_routes(kvm_state);
|
kvm_irqchip_commit_routes(kvm_state);
|
||||||
trace_kvm_x86_update_msi_routes(cnt);
|
trace_kvm_x86_update_msi_routes(cnt);
|
||||||
|
|
Loading…
Reference in a new issue