|
 |
9ae3a8 |
From c7e1a8da5c3576d962c10c7afc11b0146f6b9adc Mon Sep 17 00:00:00 2001
|
|
|
9ae3a8 |
From: Alex Williamson <alex.williamson@redhat.com>
|
|
|
9ae3a8 |
Date: Fri, 29 Sep 2017 21:44:24 +0200
|
|
|
9ae3a8 |
Subject: [PATCH 03/27] hw/vfio/pci: generalize mask/unmask to any IRQ index
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
RH-Author: Alex Williamson <alex.williamson@redhat.com>
|
|
|
9ae3a8 |
Message-id: <20170929214424.16765.25345.stgit@gimli.home>
|
|
|
9ae3a8 |
Patchwork-id: 76761
|
|
|
9ae3a8 |
O-Subject: [RHEL-7.5 qemu-kvm PATCH 03/16] hw/vfio/pci: generalize mask/unmask to any IRQ index
|
|
|
9ae3a8 |
Bugzilla: 1494181
|
|
|
9ae3a8 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Auger Eric <eric.auger@redhat.com>
|
|
|
9ae3a8 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
From: Eric Auger <eric.auger@linaro.org>
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Upstream: 079eb19cbb3079536788dfd58832824804815e48
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
To prepare for platform device introduction, rename vfio_mask_intx
|
|
|
9ae3a8 |
and vfio_unmask_intx into vfio_mask_single_irqindex and respectively
|
|
|
9ae3a8 |
unmask_single_irqindex. Also use a nex index parameter.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
With that name and prototype the function will be usable for other
|
|
|
9ae3a8 |
indexes than VFIO_PCI_INTX_IRQ_INDEX.
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
Signed-off-by: Eric Auger <eric.auger@linaro.org>
|
|
|
9ae3a8 |
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
|
|
|
9ae3a8 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9ae3a8 |
---
|
|
|
9ae3a8 |
hw/misc/vfio.c | 20 ++++++++++----------
|
|
|
9ae3a8 |
1 file changed, 10 insertions(+), 10 deletions(-)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
|
|
|
9ae3a8 |
index a1ca883..ed9b8c4 100644
|
|
|
9ae3a8 |
--- a/hw/misc/vfio.c
|
|
|
9ae3a8 |
+++ b/hw/misc/vfio.c
|
|
|
9ae3a8 |
@@ -288,12 +288,12 @@ static void vfio_disable_irqindex(VFIOPCIDevice *vdev, int index)
|
|
|
9ae3a8 |
/*
|
|
|
9ae3a8 |
* INTx
|
|
|
9ae3a8 |
*/
|
|
|
9ae3a8 |
-static void vfio_unmask_intx(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
+static void vfio_unmask_single_irqindex(VFIOPCIDevice *vdev, int index)
|
|
|
9ae3a8 |
{
|
|
|
9ae3a8 |
struct vfio_irq_set irq_set = {
|
|
|
9ae3a8 |
.argsz = sizeof(irq_set),
|
|
|
9ae3a8 |
.flags = VFIO_IRQ_SET_DATA_NONE | VFIO_IRQ_SET_ACTION_UNMASK,
|
|
|
9ae3a8 |
- .index = VFIO_PCI_INTX_IRQ_INDEX,
|
|
|
9ae3a8 |
+ .index = index,
|
|
|
9ae3a8 |
.start = 0,
|
|
|
9ae3a8 |
.count = 1,
|
|
|
9ae3a8 |
};
|
|
|
9ae3a8 |
@@ -302,12 +302,12 @@ static void vfio_unmask_intx(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
#ifdef CONFIG_KVM /* Unused outside of CONFIG_KVM code */
|
|
|
9ae3a8 |
-static void vfio_mask_intx(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
+static void vfio_mask_single_irqindex(VFIOPCIDevice *vdev, int index)
|
|
|
9ae3a8 |
{
|
|
|
9ae3a8 |
struct vfio_irq_set irq_set = {
|
|
|
9ae3a8 |
.argsz = sizeof(irq_set),
|
|
|
9ae3a8 |
.flags = VFIO_IRQ_SET_DATA_NONE | VFIO_IRQ_SET_ACTION_MASK,
|
|
|
9ae3a8 |
- .index = VFIO_PCI_INTX_IRQ_INDEX,
|
|
|
9ae3a8 |
+ .index = index,
|
|
|
9ae3a8 |
.start = 0,
|
|
|
9ae3a8 |
.count = 1,
|
|
|
9ae3a8 |
};
|
|
|
9ae3a8 |
@@ -376,7 +376,7 @@ static void vfio_eoi(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
vdev->intx.pending = false;
|
|
|
9ae3a8 |
qemu_set_irq(vdev->pdev.irq[vdev->intx.pin], 0);
|
|
|
9ae3a8 |
- vfio_unmask_intx(vdev);
|
|
|
9ae3a8 |
+ vfio_unmask_single_irqindex(vdev, VFIO_PCI_INTX_IRQ_INDEX);
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
static void vfio_enable_intx_kvm(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
@@ -399,7 +399,7 @@ static void vfio_enable_intx_kvm(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
/* Get to a known interrupt state */
|
|
|
9ae3a8 |
qemu_set_fd_handler(irqfd.fd, NULL, NULL, vdev);
|
|
|
9ae3a8 |
- vfio_mask_intx(vdev);
|
|
|
9ae3a8 |
+ vfio_mask_single_irqindex(vdev, VFIO_PCI_INTX_IRQ_INDEX);
|
|
|
9ae3a8 |
vdev->intx.pending = false;
|
|
|
9ae3a8 |
qemu_set_irq(vdev->pdev.irq[vdev->intx.pin], 0);
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -437,7 +437,7 @@ static void vfio_enable_intx_kvm(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
/* Let'em rip */
|
|
|
9ae3a8 |
- vfio_unmask_intx(vdev);
|
|
|
9ae3a8 |
+ vfio_unmask_single_irqindex(vdev, VFIO_PCI_INTX_IRQ_INDEX);
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
vdev->intx.kvm_accel = true;
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -454,7 +454,7 @@ fail_irqfd:
|
|
|
9ae3a8 |
event_notifier_cleanup(&vdev->intx.unmask);
|
|
|
9ae3a8 |
fail:
|
|
|
9ae3a8 |
qemu_set_fd_handler(irqfd.fd, vfio_intx_interrupt, NULL, vdev);
|
|
|
9ae3a8 |
- vfio_unmask_intx(vdev);
|
|
|
9ae3a8 |
+ vfio_unmask_single_irqindex(vdev, VFIO_PCI_INTX_IRQ_INDEX);
|
|
|
9ae3a8 |
#endif
|
|
|
9ae3a8 |
}
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -475,7 +475,7 @@ static void vfio_disable_intx_kvm(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
* Get to a known state, hardware masked, QEMU ready to accept new
|
|
|
9ae3a8 |
* interrupts, QEMU IRQ de-asserted.
|
|
|
9ae3a8 |
*/
|
|
|
9ae3a8 |
- vfio_mask_intx(vdev);
|
|
|
9ae3a8 |
+ vfio_mask_single_irqindex(vdev, VFIO_PCI_INTX_IRQ_INDEX);
|
|
|
9ae3a8 |
vdev->intx.pending = false;
|
|
|
9ae3a8 |
qemu_set_irq(vdev->pdev.irq[vdev->intx.pin], 0);
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
@@ -493,7 +493,7 @@ static void vfio_disable_intx_kvm(VFIOPCIDevice *vdev)
|
|
|
9ae3a8 |
vdev->intx.kvm_accel = false;
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
/* If we've missed an event, let it re-fire through QEMU */
|
|
|
9ae3a8 |
- vfio_unmask_intx(vdev);
|
|
|
9ae3a8 |
+ vfio_unmask_single_irqindex(vdev, VFIO_PCI_INTX_IRQ_INDEX);
|
|
|
9ae3a8 |
|
|
|
9ae3a8 |
DPRINTF("%s(%04x:%02x:%02x.%x) KVM INTx accel disabled\n",
|
|
|
9ae3a8 |
__func__, vdev->host.domain, vdev->host.bus,
|
|
|
9ae3a8 |
--
|
|
|
9ae3a8 |
1.8.3.1
|
|
|
9ae3a8 |
|