kvm: Conditionally register IRQ bypass consumer
authorAlex Williamson <alex.williamson@redhat.com>
Thu, 5 May 2016 17:58:35 +0000 (11:58 -0600)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 11 May 2016 20:37:55 +0000 (22:37 +0200)
If we don't support a mechanism for bypassing IRQs, don't register as
a consumer.  This eliminates meaningless dev_info()s when the connect
fails between producer and consumer, such as on AMD systems where
kvm_x86_ops->update_pi_irte is not implemented

Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/x86.c
include/linux/kvm_host.h
virt/kvm/eventfd.c

index 57a62d1e2f5d387fbe7af49df47ae2cff20778f5..6c774cdf553c69e600e6b90fe90aed2d77d081c4 100644 (file)
@@ -8358,19 +8358,21 @@ bool kvm_arch_has_noncoherent_dma(struct kvm *kvm)
 }
 EXPORT_SYMBOL_GPL(kvm_arch_has_noncoherent_dma);
 
+bool kvm_arch_has_irq_bypass(void)
+{
+       return kvm_x86_ops->update_pi_irte != NULL;
+}
+
 int kvm_arch_irq_bypass_add_producer(struct irq_bypass_consumer *cons,
                                      struct irq_bypass_producer *prod)
 {
        struct kvm_kernel_irqfd *irqfd =
                container_of(cons, struct kvm_kernel_irqfd, consumer);
 
-       if (kvm_x86_ops->update_pi_irte) {
-               irqfd->producer = prod;
-               return kvm_x86_ops->update_pi_irte(irqfd->kvm,
-                               prod->irq, irqfd->gsi, 1);
-       }
+       irqfd->producer = prod;
 
-       return -EINVAL;
+       return kvm_x86_ops->update_pi_irte(irqfd->kvm,
+                                          prod->irq, irqfd->gsi, 1);
 }
 
 void kvm_arch_irq_bypass_del_producer(struct irq_bypass_consumer *cons,
@@ -8380,11 +8382,6 @@ void kvm_arch_irq_bypass_del_producer(struct irq_bypass_consumer *cons,
        struct kvm_kernel_irqfd *irqfd =
                container_of(cons, struct kvm_kernel_irqfd, consumer);
 
-       if (!kvm_x86_ops->update_pi_irte) {
-               WARN_ON(irqfd->producer != NULL);
-               return;
-       }
-
        WARN_ON(irqfd->producer != prod);
        irqfd->producer = NULL;
 
index 352889d6e322c44304f50bb016f7a5d71c220aea..92a0229044fb0c8148f4d5c227de5b3856610fba 100644 (file)
@@ -1185,6 +1185,7 @@ static inline void kvm_vcpu_set_dy_eligible(struct kvm_vcpu *vcpu, bool val)
 #endif /* CONFIG_HAVE_KVM_CPU_RELAX_INTERCEPT */
 
 #ifdef CONFIG_HAVE_KVM_IRQ_BYPASS
+bool kvm_arch_has_irq_bypass(void);
 int kvm_arch_irq_bypass_add_producer(struct irq_bypass_consumer *,
                           struct irq_bypass_producer *);
 void kvm_arch_irq_bypass_del_producer(struct irq_bypass_consumer *,
index 46dbc0a7dfc144ee10a2a1c53b6ab03bbf0f66c0..e469b60124718e60740e24857d15ae7b8a503552 100644 (file)
@@ -408,15 +408,17 @@ kvm_irqfd_assign(struct kvm *kvm, struct kvm_irqfd *args)
         */
        fdput(f);
 #ifdef CONFIG_HAVE_KVM_IRQ_BYPASS
-       irqfd->consumer.token = (void *)irqfd->eventfd;
-       irqfd->consumer.add_producer = kvm_arch_irq_bypass_add_producer;
-       irqfd->consumer.del_producer = kvm_arch_irq_bypass_del_producer;
-       irqfd->consumer.stop = kvm_arch_irq_bypass_stop;
-       irqfd->consumer.start = kvm_arch_irq_bypass_start;
-       ret = irq_bypass_register_consumer(&irqfd->consumer);
-       if (ret)
-               pr_info("irq bypass consumer (token %p) registration fails: %d\n",
+       if (kvm_arch_has_irq_bypass()) {
+               irqfd->consumer.token = (void *)irqfd->eventfd;
+               irqfd->consumer.add_producer = kvm_arch_irq_bypass_add_producer;
+               irqfd->consumer.del_producer = kvm_arch_irq_bypass_del_producer;
+               irqfd->consumer.stop = kvm_arch_irq_bypass_stop;
+               irqfd->consumer.start = kvm_arch_irq_bypass_start;
+               ret = irq_bypass_register_consumer(&irqfd->consumer);
+               if (ret)
+                       pr_info("irq bypass consumer (token %p) registration fails: %d\n",
                                irqfd->consumer.token, ret);
+       }
 #endif
 
        return 0;