From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752663AbbLCSik (ORCPT ); Thu, 3 Dec 2015 13:38:40 -0500 Received: from mga14.intel.com ([192.55.52.115]:53670 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752986AbbLCSgr (ORCPT ); Thu, 3 Dec 2015 13:36:47 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,378,1444719600"; d="scan'208";a="7025554" From: Yunhong Jiang To: alex.williamson@redhat.com, pbonzini@redhat.com Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 3/5] VFIO: Support threaded interrupt handling on VFIO Date: Thu, 3 Dec 2015 10:22:50 -0800 Message-Id: <1449166972-8894-4-git-send-email-yunhong.jiang@linux.intel.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1449166972-8894-1-git-send-email-yunhong.jiang@linux.intel.com> References: <1449166972-8894-1-git-send-email-yunhong.jiang@linux.intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org For VFIO device with MSI interrupt type, it's possible to handle the interrupt on hard interrupt context without invoking the interrupt thread. Handling the interrupt on hard interrupt context reduce the interrupt latency. Signed-off-by: Yunhong Jiang --- drivers/vfio/pci/vfio_pci_intrs.c | 39 ++++++++++++++++++++++++++++++++++----- 1 file changed, 34 insertions(+), 5 deletions(-) diff --git a/drivers/vfio/pci/vfio_pci_intrs.c b/drivers/vfio/pci/vfio_pci_intrs.c index 3b3ba15558b7..108d335c5656 100644 --- a/drivers/vfio/pci/vfio_pci_intrs.c +++ b/drivers/vfio/pci/vfio_pci_intrs.c @@ -236,12 +236,35 @@ static void vfio_intx_disable(struct vfio_pci_device *vdev) kfree(vdev->ctx); } +static irqreturn_t vfio_msihandler(int irq, void *arg) +{ + struct vfio_pci_irq_ctx *ctx = arg; + struct irq_bypass_producer *producer = &ctx->producer; + struct irq_bypass_consumer *consumer; + int ret = IRQ_HANDLED, idx; + + idx = srcu_read_lock(&producer->srcu); + + list_for_each_entry_rcu(consumer, &producer->consumers, sibling) { + /* + * Invoke the thread handler if any consumer would block, but + * finish all consumes. + */ + if (consumer->handle_irq(consumer->irq_context) == -EWOULDBLOCK) + ret = IRQ_WAKE_THREAD; + continue; + } + + srcu_read_unlock(&producer->srcu, idx); + return ret; +} + /* * MSI/MSI-X */ -static irqreturn_t vfio_msihandler(int irq, void *arg) +static irqreturn_t vfio_msihandler_threaded(int irq, void *arg) { - struct eventfd_ctx *trigger = arg; + struct eventfd_ctx *trigger = ((struct vfio_pci_irq_ctx *)arg)->trigger; eventfd_signal(trigger, 1); return IRQ_HANDLED; @@ -318,7 +341,7 @@ static int vfio_msi_set_vector_signal(struct vfio_pci_device *vdev, return -EINVAL; if (vdev->ctx[vector].trigger) { - free_irq(irq, vdev->ctx[vector].trigger); + free_irq(irq, &vdev->ctx[vector]); irq_bypass_unregister_producer(&vdev->ctx[vector].producer); kfree(vdev->ctx[vector].name); eventfd_ctx_put(vdev->ctx[vector].trigger); @@ -353,8 +376,14 @@ static int vfio_msi_set_vector_signal(struct vfio_pci_device *vdev, pci_write_msi_msg(irq, &msg); } - ret = request_irq(irq, vfio_msihandler, 0, - vdev->ctx[vector].name, trigger); + /* + * Currently the primary handler for the thread_irq will be invoked on + * a thread, the IRQF_ONESHOT is a hack for it. + */ + ret = request_threaded_irq(irq, vfio_msihandler, + vfio_msihandler_threaded, + IRQF_ONESHOT, vdev->ctx[vector].name, + &vdev->ctx[vector]); if (ret) { kfree(vdev->ctx[vector].name); eventfd_ctx_put(trigger); -- 1.8.3.1