qemu-kvm: Do not advertise MSI caps on lacking KVM support

As suggested by Michael Tsirkin: Move the check for GSI routing from
kvm_msi_message_add to the MSI/MSI-X initalization. If it fails (and KVM
is in in-kernel irqchip mode), do not advertise MSI at all.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>

3 files changed, 7 insertions, 5 deletions

diff --git a/hw/msi.c b/hw/msi.c
index 725b2b7e8..ddcfedd6e 100644
--- a/hw/msi.c
+++ b/hw/msi.c
@@ -208,6 +208,10 @@ int msi_init(struct PCIDevice *dev, uint8_t offset,
                    " 64bit %d mask %d\n",
                    offset, nr_vectors, msi64bit, msi_per_vector_mask);
 
+    if (kvm_enabled() && kvm_irqchip_in_kernel() && !kvm_has_gsi_routing()) {
+        return -ENOTSUP;
+    }
+
     assert(!(nr_vectors & (nr_vectors - 1)));   /* power of 2 */
     assert(nr_vectors > 0);
     assert(nr_vectors <= PCI_MSI_VECTORS_MAX);
diff --git a/hw/msix.c b/hw/msix.c
index 9cee086fd..d371940ea 100644
--- a/hw/msix.c
+++ b/hw/msix.c
@@ -350,8 +350,10 @@ int msix_init(struct PCIDevice *dev, unsigned short nentries,
 {
     int ret;
     /* Nothing to do if MSI is not supported by interrupt controller */
-    if (!msix_supported)
+    if (!msix_supported ||
+        (kvm_enabled() && kvm_irqchip_in_kernel() && !kvm_has_gsi_routing())) {
         return -ENOTSUP;
+    }
 
     if (nentries > MSIX_MAX_ENTRIES)
         return -EINVAL;
diff --git a/qemu-kvm.c b/qemu-kvm.c
index e8c2009c4..b9511465d 100644
--- a/qemu-kvm.c
+++ b/qemu-kvm.c
@@ -986,10 +986,6 @@ int kvm_msi_message_add(KVMMsiMessage *msg)
     struct kvm_irq_routing_entry e;
     int ret;
 
-    if (!kvm_has_gsi_routing()) {
-        return -EOPNOTSUPP;
-    }
-
     ret = kvm_get_irq_route_gsi();
     if (ret < 0) {
         return ret;