cryptospore / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone
9ae3a8
From 725ad24f771adcc8aeacaf27b21a6d95cab41e69 Mon Sep 17 00:00:00 2001
9ae3a8
From: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
Date: Fri, 29 Sep 2017 21:45:02 +0200
9ae3a8
Subject: [PATCH 07/27] hw/vfio/pci: Introduce VFIORegion
9ae3a8
9ae3a8
RH-Author: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
Message-id: <20170929214502.16765.46217.stgit@gimli.home>
9ae3a8
Patchwork-id: 76765
9ae3a8
O-Subject: [RHEL-7.5 qemu-kvm PATCH 07/16] hw/vfio/pci: Introduce VFIORegion
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: a664477db8dac84cc046e9d79701eefda1d58703
9ae3a8
9ae3a8
This structure is going to be shared by VFIOPCIDevice and
9ae3a8
VFIOPlatformDevice. VFIOBAR includes it.
9ae3a8
9ae3a8
vfio_eoi becomes an ops of VFIODevice specialized by parent device.
9ae3a8
This makes possible to transform vfio_bar_write/read into generic
9ae3a8
vfio_region_write/read that will be used by VFIOPlatformDevice too.
9ae3a8
9ae3a8
vfio_mmap_bar becomes vfio_map_region
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 | 188 +++++++++++++++++++++++++++++----------------------------
9ae3a8
 1 file changed, 97 insertions(+), 91 deletions(-)
9ae3a8
9ae3a8
diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
9ae3a8
index 3e559ed..92414b9 100644
9ae3a8
--- a/hw/misc/vfio.c
9ae3a8
+++ b/hw/misc/vfio.c
9ae3a8
@@ -84,15 +84,19 @@ typedef struct VFIOQuirk {
9ae3a8
     } data;
9ae3a8
 } VFIOQuirk;
9ae3a8
 
9ae3a8
-typedef struct VFIOBAR {
9ae3a8
-    off_t fd_offset; /* offset of BAR within device fd */
9ae3a8
-    int fd; /* device fd, allows us to pass VFIOBAR as opaque data */
9ae3a8
+typedef struct VFIORegion {
9ae3a8
+    struct VFIODevice *vbasedev;
9ae3a8
+    off_t fd_offset; /* offset of region within device fd */
9ae3a8
     MemoryRegion mem; /* slow, read/write access */
9ae3a8
     MemoryRegion mmap_mem; /* direct mapped access */
9ae3a8
     void *mmap;
9ae3a8
     size_t size;
9ae3a8
     uint32_t flags; /* VFIO region flags (rd/wr/mmap) */
9ae3a8
-    uint8_t nr; /* cache the BAR number for debug */
9ae3a8
+    uint8_t nr; /* cache the region number for debug */
9ae3a8
+} VFIORegion;
9ae3a8
+
9ae3a8
+typedef struct VFIOBAR {
9ae3a8
+    VFIORegion region;
9ae3a8
     bool ioport;
9ae3a8
     bool mem64;
9ae3a8
     QLIST_HEAD(, VFIOQuirk) quirks;
9ae3a8
@@ -194,6 +198,7 @@ typedef struct VFIODevice {
9ae3a8
 struct VFIODeviceOps {
9ae3a8
     void (*vfio_compute_needs_reset)(VFIODevice *vdev);
9ae3a8
     int (*vfio_hot_reset_multi)(VFIODevice *vdev);
9ae3a8
+    void (*vfio_eoi)(VFIODevice *vdev);
9ae3a8
 };
9ae3a8
 
9ae3a8
 typedef struct VFIOPCIDevice {
9ae3a8
@@ -383,8 +388,10 @@ static void vfio_intx_interrupt(void *opaque)
9ae3a8
     }
9ae3a8
 }
9ae3a8
 
9ae3a8
-static void vfio_eoi(VFIOPCIDevice *vdev)
9ae3a8
+static void vfio_eoi(VFIODevice *vbasedev)
9ae3a8
 {
9ae3a8
+    VFIOPCIDevice *vdev = container_of(vbasedev, VFIOPCIDevice, vbasedev);
9ae3a8
+
9ae3a8
     if (!vdev->intx.pending) {
9ae3a8
         return;
9ae3a8
     }
9ae3a8
@@ -394,7 +401,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_single_irqindex(&vdev->vbasedev, VFIO_PCI_INTX_IRQ_INDEX);
9ae3a8
+    vfio_unmask_single_irqindex(vbasedev, VFIO_PCI_INTX_IRQ_INDEX);
9ae3a8
 }
9ae3a8
 
9ae3a8
 static void vfio_enable_intx_kvm(VFIOPCIDevice *vdev)
9ae3a8
@@ -549,7 +556,7 @@ static void vfio_update_irq(PCIDevice *pdev)
9ae3a8
     vfio_enable_intx_kvm(vdev);
9ae3a8
 
9ae3a8
     /* Re-enable the interrupt in cased we missed an EOI */
9ae3a8
-    vfio_eoi(vdev);
9ae3a8
+    vfio_eoi(&vdev->vbasedev);
9ae3a8
 }
9ae3a8
 
9ae3a8
 static int vfio_enable_intx(VFIOPCIDevice *vdev)
9ae3a8
@@ -1086,10 +1093,11 @@ static void vfio_update_msi(VFIOPCIDevice *vdev)
9ae3a8
 /*
9ae3a8
  * IO Port/MMIO - Beware of the endians, VFIO is always little endian
9ae3a8
  */
9ae3a8
-static void vfio_bar_write(void *opaque, hwaddr addr,
9ae3a8
-                           uint64_t data, unsigned size)
9ae3a8
+static void vfio_region_write(void *opaque, hwaddr addr,
9ae3a8
+                              uint64_t data, unsigned size)
9ae3a8
 {
9ae3a8
-    VFIOBAR *bar = opaque;
9ae3a8
+    VFIORegion *region = opaque;
9ae3a8
+    VFIODevice *vbasedev = region->vbasedev;
9ae3a8
     union {
9ae3a8
         uint8_t byte;
9ae3a8
         uint16_t word;
9ae3a8
@@ -1112,21 +1120,15 @@ static void vfio_bar_write(void *opaque, hwaddr addr,
9ae3a8
         break;
9ae3a8
     }
9ae3a8
 
9ae3a8
-    if (pwrite(bar->fd, &buf, size, bar->fd_offset + addr) != size) {
9ae3a8
-        error_report("%s(,0x%"HWADDR_PRIx", 0x%"PRIx64", %d) failed: %m",
9ae3a8
-                     __func__, addr, data, size);
9ae3a8
+    if (pwrite(vbasedev->fd, &buf, size, region->fd_offset + addr) != size) {
9ae3a8
+        error_report("%s(%s:region%d+0x%"HWADDR_PRIx", 0x%"PRIx64
9ae3a8
+                     ",%d) failed: %m",
9ae3a8
+                     __func__, vbasedev->name, region->nr,
9ae3a8
+                     addr, data, size);
9ae3a8
     }
9ae3a8
 
9ae3a8
-#ifdef DEBUG_VFIO
9ae3a8
-    {
9ae3a8
-        VFIOPCIDevice *vdev = container_of(bar, VFIOPCIDevice, bars[bar->nr]);
9ae3a8
-
9ae3a8
-        DPRINTF("%s(%04x:%02x:%02x.%x:BAR%d+0x%"HWADDR_PRIx", 0x%"PRIx64
9ae3a8
-                ", %d)\n", __func__, vdev->host.domain, vdev->host.bus,
9ae3a8
-                vdev->host.slot, vdev->host.function, bar->nr, addr,
9ae3a8
-                data, size);
9ae3a8
-    }
9ae3a8
-#endif
9ae3a8
+    DPRINTF("%s(%s:BAR%d+0x%"HWADDR_PRIx", 0x%"PRIx64", %d)\n", __func__,
9ae3a8
+            vbasedev->name, region->nr, addr, data, size);
9ae3a8
 
9ae3a8
     /*
9ae3a8
      * A read or write to a BAR always signals an INTx EOI.  This will
9ae3a8
@@ -1136,13 +1138,14 @@ static void vfio_bar_write(void *opaque, hwaddr addr,
9ae3a8
      * which access will service the interrupt, so we're potentially
9ae3a8
      * getting quite a few host interrupts per guest interrupt.
9ae3a8
      */
9ae3a8
-    vfio_eoi(container_of(bar, VFIOPCIDevice, bars[bar->nr]));
9ae3a8
+    vbasedev->ops->vfio_eoi(vbasedev);
9ae3a8
 }
9ae3a8
 
9ae3a8
-static uint64_t vfio_bar_read(void *opaque,
9ae3a8
-                              hwaddr addr, unsigned size)
9ae3a8
+static uint64_t vfio_region_read(void *opaque,
9ae3a8
+                                 hwaddr addr, unsigned size)
9ae3a8
 {
9ae3a8
-    VFIOBAR *bar = opaque;
9ae3a8
+    VFIORegion *region = opaque;
9ae3a8
+    VFIODevice *vbasedev = region->vbasedev;
9ae3a8
     union {
9ae3a8
         uint8_t byte;
9ae3a8
         uint16_t word;
9ae3a8
@@ -1151,9 +1154,10 @@ static uint64_t vfio_bar_read(void *opaque,
9ae3a8
     } buf;
9ae3a8
     uint64_t data = 0;
9ae3a8
 
9ae3a8
-    if (pread(bar->fd, &buf, size, bar->fd_offset + addr) != size) {
9ae3a8
-        error_report("%s(,0x%"HWADDR_PRIx", %d) failed: %m",
9ae3a8
-                     __func__, addr, size);
9ae3a8
+    if (pread(vbasedev->fd, &buf, size, region->fd_offset + addr) != size) {
9ae3a8
+        error_report("%s(%s:region%d+0x%"HWADDR_PRIx", %d) failed: %m",
9ae3a8
+                     __func__, vbasedev->name, region->nr,
9ae3a8
+                     addr, size);
9ae3a8
         return (uint64_t)-1;
9ae3a8
     }
9ae3a8
 
9ae3a8
@@ -1172,26 +1176,18 @@ static uint64_t vfio_bar_read(void *opaque,
9ae3a8
         break;
9ae3a8
     }
9ae3a8
 
9ae3a8
-#ifdef DEBUG_VFIO
9ae3a8
-    {
9ae3a8
-        VFIOPCIDevice *vdev = container_of(bar, VFIOPCIDevice, bars[bar->nr]);
9ae3a8
-
9ae3a8
-        DPRINTF("%s(%04x:%02x:%02x.%x:BAR%d+0x%"HWADDR_PRIx
9ae3a8
-                ", %d) = 0x%"PRIx64"\n", __func__, vdev->host.domain,
9ae3a8
-                vdev->host.bus, vdev->host.slot, vdev->host.function,
9ae3a8
-                bar->nr, addr, size, data);
9ae3a8
-    }
9ae3a8
-#endif
9ae3a8
+    DPRINTF("%s(%s:BAR%d+0x%"HWADDR_PRIx", %d) = 0x%"PRIx64"\n", __func__,
9ae3a8
+            vbasedev->name, region->nr, addr, size, data);
9ae3a8
 
9ae3a8
     /* Same as write above */
9ae3a8
-    vfio_eoi(container_of(bar, VFIOPCIDevice, bars[bar->nr]));
9ae3a8
+    vbasedev->ops->vfio_eoi(vbasedev);
9ae3a8
 
9ae3a8
     return data;
9ae3a8
 }
9ae3a8
 
9ae3a8
-static const MemoryRegionOps vfio_bar_ops = {
9ae3a8
-    .read = vfio_bar_read,
9ae3a8
-    .write = vfio_bar_write,
9ae3a8
+static const MemoryRegionOps vfio_region_ops = {
9ae3a8
+    .read = vfio_region_read,
9ae3a8
+    .write = vfio_region_write,
9ae3a8
     .endianness = DEVICE_LITTLE_ENDIAN,
9ae3a8
 };
9ae3a8
 
9ae3a8
@@ -1505,8 +1501,8 @@ static uint64_t vfio_generic_window_quirk_read(void *opaque,
9ae3a8
                 vdev->host.bus, vdev->host.slot, vdev->host.function,
9ae3a8
                 quirk->data.bar, addr, size, data);
9ae3a8
     } else {
9ae3a8
-        data = vfio_bar_read(&vdev->bars[quirk->data.bar],
9ae3a8
-                             addr + quirk->data.base_offset, size);
9ae3a8
+        data = vfio_region_read(&vdev->bars[quirk->data.bar].region,
9ae3a8
+                                addr + quirk->data.base_offset, size);
9ae3a8
     }
9ae3a8
 
9ae3a8
     return data;
9ae3a8
@@ -1556,7 +1552,7 @@ static void vfio_generic_window_quirk_write(void *opaque, hwaddr addr,
9ae3a8
         return;
9ae3a8
     }
9ae3a8
 
9ae3a8
-    vfio_bar_write(&vdev->bars[quirk->data.bar],
9ae3a8
+    vfio_region_write(&vdev->bars[quirk->data.bar].region,
9ae3a8
                    addr + quirk->data.base_offset, data, size);
9ae3a8
 }
9ae3a8
 
9ae3a8
@@ -1590,7 +1586,8 @@ static uint64_t vfio_generic_quirk_read(void *opaque,
9ae3a8
                 vdev->host.bus, vdev->host.slot, vdev->host.function,
9ae3a8
                 quirk->data.bar, addr + base, size, data);
9ae3a8
     } else {
9ae3a8
-        data = vfio_bar_read(&vdev->bars[quirk->data.bar], addr + base, size);
9ae3a8
+        data = vfio_region_read(&vdev->bars[quirk->data.bar].region,
9ae3a8
+                                addr + base, size);
9ae3a8
     }
9ae3a8
 
9ae3a8
     return data;
9ae3a8
@@ -1619,7 +1616,8 @@ static void vfio_generic_quirk_write(void *opaque, hwaddr addr,
9ae3a8
                 vdev->host.domain, vdev->host.bus, vdev->host.slot,
9ae3a8
                 vdev->host.function, quirk->data.bar, addr + base, data, size);
9ae3a8
     } else {
9ae3a8
-        vfio_bar_write(&vdev->bars[quirk->data.bar], addr + base, data, size);
9ae3a8
+        vfio_region_write(&vdev->bars[quirk->data.bar].region,
9ae3a8
+                          addr + base, data, size);
9ae3a8
     }
9ae3a8
 }
9ae3a8
 
9ae3a8
@@ -1672,7 +1670,7 @@ static void vfio_vga_probe_ati_3c3_quirk(VFIOPCIDevice *vdev)
9ae3a8
      * As long as the BAR is >= 256 bytes it will be aligned such that the
9ae3a8
      * lower byte is always zero.  Filter out anything else, if it exists.
9ae3a8
      */
9ae3a8
-    if (!vdev->bars[4].ioport || vdev->bars[4].size < 256) {
9ae3a8
+    if (!vdev->bars[4].ioport || vdev->bars[4].region.size < 256) {
9ae3a8
         return;
9ae3a8
     }
9ae3a8
 
9ae3a8
@@ -1725,7 +1723,7 @@ static void vfio_probe_ati_bar4_window_quirk(VFIOPCIDevice *vdev, int nr)
9ae3a8
     memory_region_init_io(&quirk->mem,
9ae3a8
                           &vfio_generic_window_quirk, quirk,
9ae3a8
                           "vfio-ati-bar4-window-quirk", 8);
9ae3a8
-    memory_region_add_subregion_overlap(&vdev->bars[nr].mem,
9ae3a8
+    memory_region_add_subregion_overlap(&vdev->bars[nr].region.mem,
9ae3a8
                           quirk->data.base_offset, &quirk->mem, 1);
9ae3a8
 
9ae3a8
     QLIST_INSERT_HEAD(&vdev->bars[nr].quirks, quirk, next);
9ae3a8
@@ -1759,7 +1757,7 @@ static void vfio_probe_ati_bar2_4000_quirk(VFIOPCIDevice *vdev, int nr)
9ae3a8
     memory_region_init_io(&quirk->mem, &vfio_generic_quirk, quirk,
9ae3a8
                           "vfio-ati-bar2-4000-quirk",
9ae3a8
                           TARGET_PAGE_ALIGN(quirk->data.address_mask + 1));
9ae3a8
-    memory_region_add_subregion_overlap(&vdev->bars[nr].mem,
9ae3a8
+    memory_region_add_subregion_overlap(&vdev->bars[nr].region.mem,
9ae3a8
                           quirk->data.address_match & TARGET_PAGE_MASK,
9ae3a8
                           &quirk->mem, 1);
9ae3a8
 
9ae3a8
@@ -1878,7 +1876,7 @@ static void vfio_vga_probe_nvidia_3d0_quirk(VFIOPCIDevice *vdev)
9ae3a8
     VFIOQuirk *quirk;
9ae3a8
 
9ae3a8
     if (pci_get_word(pdev->config + PCI_VENDOR_ID) != PCI_VENDOR_ID_NVIDIA ||
9ae3a8
-        !vdev->bars[1].size) {
9ae3a8
+        !vdev->bars[1].region.size) {
9ae3a8
         return;
9ae3a8
     }
9ae3a8
 
9ae3a8
@@ -1986,7 +1984,8 @@ static void vfio_probe_nvidia_bar5_window_quirk(VFIOPCIDevice *vdev, int nr)
9ae3a8
     memory_region_init_io(&quirk->mem,
9ae3a8
                           &vfio_nvidia_bar5_window_quirk, quirk,
9ae3a8
                           "vfio-nvidia-bar5-window-quirk", 16);
9ae3a8
-    memory_region_add_subregion_overlap(&vdev->bars[nr].mem, 0, &quirk->mem, 1);
9ae3a8
+    memory_region_add_subregion_overlap(&vdev->bars[nr].region.mem,
9ae3a8
+                                        0, &quirk->mem, 1);
9ae3a8
 
9ae3a8
     QLIST_INSERT_HEAD(&vdev->bars[nr].quirks, quirk, next);
9ae3a8
 
9ae3a8
@@ -2024,7 +2023,7 @@ static void vfio_probe_nvidia_bar0_88000_quirk(VFIOPCIDevice *vdev, int nr)
9ae3a8
     memory_region_init_io(&quirk->mem, &vfio_generic_quirk,
9ae3a8
                           quirk, "vfio-nvidia-bar0-88000-quirk",
9ae3a8
                           TARGET_PAGE_ALIGN(quirk->data.address_mask + 1));
9ae3a8
-    memory_region_add_subregion_overlap(&vdev->bars[nr].mem,
9ae3a8
+    memory_region_add_subregion_overlap(&vdev->bars[nr].region.mem,
9ae3a8
                           quirk->data.address_match & TARGET_PAGE_MASK,
9ae3a8
                           &quirk->mem, 1);
9ae3a8
 
9ae3a8
@@ -2050,7 +2049,8 @@ static void vfio_probe_nvidia_bar0_1800_quirk(VFIOPCIDevice *vdev, int nr)
9ae3a8
 
9ae3a8
     /* Log the chipset ID */
9ae3a8
     DPRINTF("Nvidia NV%02x\n",
9ae3a8
-            (unsigned int)(vfio_bar_read(&vdev->bars[0], 0, 4) >> 20) & 0xff);
9ae3a8
+            (unsigned int)(vfio_region_read(&vdev->bars[0].region, 0, 4) >> 20)
9ae3a8
+            & 0xff);
9ae3a8
 
9ae3a8
     quirk = g_malloc0(sizeof(*quirk));
9ae3a8
     quirk->vdev = vdev;
9ae3a8
@@ -2062,7 +2062,7 @@ static void vfio_probe_nvidia_bar0_1800_quirk(VFIOPCIDevice *vdev, int nr)
9ae3a8
     memory_region_init_io(&quirk->mem, &vfio_generic_quirk, quirk,
9ae3a8
                           "vfio-nvidia-bar0-1800-quirk",
9ae3a8
                           TARGET_PAGE_ALIGN(quirk->data.address_mask + 1));
9ae3a8
-    memory_region_add_subregion_overlap(&vdev->bars[nr].mem,
9ae3a8
+    memory_region_add_subregion_overlap(&vdev->bars[nr].region.mem,
9ae3a8
                           quirk->data.address_match & TARGET_PAGE_MASK,
9ae3a8
                           &quirk->mem, 1);
9ae3a8
 
9ae3a8
@@ -2118,7 +2118,7 @@ static void vfio_bar_quirk_teardown(VFIOPCIDevice *vdev, int nr)
9ae3a8
 
9ae3a8
     while (!QLIST_EMPTY(&bar->quirks)) {
9ae3a8
         VFIOQuirk *quirk = QLIST_FIRST(&bar->quirks);
9ae3a8
-        memory_region_del_subregion(&bar->mem, &quirk->mem);
9ae3a8
+        memory_region_del_subregion(&bar->region.mem, &quirk->mem);
9ae3a8
         memory_region_destroy(&quirk->mem);
9ae3a8
         QLIST_REMOVE(quirk, next);
9ae3a8
         g_free(quirk);
9ae3a8
@@ -2495,7 +2495,7 @@ static int vfio_early_setup_msix(VFIOPCIDevice *vdev)
9ae3a8
      * specific quirk if the device is known or we have a broken configuration.
9ae3a8
      */
9ae3a8
     if (vdev->msix->pba_offset >=
9ae3a8
-        vdev->bars[vdev->msix->pba_bar].size) {
9ae3a8
+        vdev->bars[vdev->msix->pba_bar].region.size) {
9ae3a8
 
9ae3a8
         PCIDevice *pdev = &vdev->pdev;
9ae3a8
         uint16_t vendor = pci_get_word(pdev->config + PCI_VENDOR_ID);
9ae3a8
@@ -2524,9 +2524,9 @@ static int vfio_setup_msix(VFIOPCIDevice *vdev, int pos)
9ae3a8
     int ret;
9ae3a8
 
9ae3a8
     ret = msix_init(&vdev->pdev, vdev->msix->entries,
9ae3a8
-                    &vdev->bars[vdev->msix->table_bar].mem,
9ae3a8
+                    &vdev->bars[vdev->msix->table_bar].region.mem,
9ae3a8
                     vdev->msix->table_bar, vdev->msix->table_offset,
9ae3a8
-                    &vdev->bars[vdev->msix->pba_bar].mem,
9ae3a8
+                    &vdev->bars[vdev->msix->pba_bar].region.mem,
9ae3a8
                     vdev->msix->pba_bar, vdev->msix->pba_offset, pos);
9ae3a8
     if (ret < 0) {
9ae3a8
         if (ret == -ENOTSUP) {
9ae3a8
@@ -2544,8 +2544,9 @@ static void vfio_teardown_msi(VFIOPCIDevice *vdev)
9ae3a8
     msi_uninit(&vdev->pdev);
9ae3a8
 
9ae3a8
     if (vdev->msix) {
9ae3a8
-        msix_uninit(&vdev->pdev, &vdev->bars[vdev->msix->table_bar].mem,
9ae3a8
-                    &vdev->bars[vdev->msix->pba_bar].mem);
9ae3a8
+        msix_uninit(&vdev->pdev,
9ae3a8
+                    &vdev->bars[vdev->msix->table_bar].region.mem,
9ae3a8
+                    &vdev->bars[vdev->msix->pba_bar].region.mem);
9ae3a8
     }
9ae3a8
 }
9ae3a8
 
9ae3a8
@@ -2559,11 +2560,11 @@ static void vfio_mmap_set_enabled(VFIOPCIDevice *vdev, bool enabled)
9ae3a8
     for (i = 0; i < PCI_ROM_SLOT; i++) {
9ae3a8
         VFIOBAR *bar = &vdev->bars[i];
9ae3a8
 
9ae3a8
-        if (!bar->size) {
9ae3a8
+        if (!bar->region.size) {
9ae3a8
             continue;
9ae3a8
         }
9ae3a8
 
9ae3a8
-        memory_region_set_enabled(&bar->mmap_mem, enabled);
9ae3a8
+        memory_region_set_enabled(&bar->region.mmap_mem, enabled);
9ae3a8
         if (vdev->msix && vdev->msix->table_bar == i) {
9ae3a8
             memory_region_set_enabled(&vdev->msix->mmap_mem, enabled);
9ae3a8
         }
9ae3a8
@@ -2574,45 +2575,47 @@ static void vfio_unmap_bar(VFIOPCIDevice *vdev, int nr)
9ae3a8
 {
9ae3a8
     VFIOBAR *bar = &vdev->bars[nr];
9ae3a8
 
9ae3a8
-    if (!bar->size) {
9ae3a8
+    if (!bar->region.size) {
9ae3a8
         return;
9ae3a8
     }
9ae3a8
 
9ae3a8
     vfio_bar_quirk_teardown(vdev, nr);
9ae3a8
 
9ae3a8
-    memory_region_del_subregion(&bar->mem, &bar->mmap_mem);
9ae3a8
-    munmap(bar->mmap, memory_region_size(&bar->mmap_mem));
9ae3a8
-    memory_region_destroy(&bar->mmap_mem);
9ae3a8
+    memory_region_del_subregion(&bar->region.mem, &bar->region.mmap_mem);
9ae3a8
+    munmap(bar->region.mmap, memory_region_size(&bar->region.mmap_mem));
9ae3a8
+    memory_region_destroy(&bar->region.mmap_mem);
9ae3a8
 
9ae3a8
     if (vdev->msix && vdev->msix->table_bar == nr) {
9ae3a8
-        memory_region_del_subregion(&bar->mem, &vdev->msix->mmap_mem);
9ae3a8
+        memory_region_del_subregion(&bar->region.mem, &vdev->msix->mmap_mem);
9ae3a8
         munmap(vdev->msix->mmap, memory_region_size(&vdev->msix->mmap_mem));
9ae3a8
         memory_region_destroy(&vdev->msix->mmap_mem);
9ae3a8
     }
9ae3a8
 
9ae3a8
-    memory_region_destroy(&bar->mem);
9ae3a8
+    memory_region_destroy(&bar->region.mem);
9ae3a8
 }
9ae3a8
 
9ae3a8
-static int vfio_mmap_bar(VFIOPCIDevice *vdev, VFIOBAR *bar,
9ae3a8
-                         MemoryRegion *mem, MemoryRegion *submem,
9ae3a8
-                         void **map, size_t size, off_t offset,
9ae3a8
-                         const char *name)
9ae3a8
+static int vfio_mmap_region(Object *obj, VFIORegion *region,
9ae3a8
+                            MemoryRegion *mem, MemoryRegion *submem,
9ae3a8
+                            void **map, size_t size, off_t offset,
9ae3a8
+                            const char *name)
9ae3a8
 {
9ae3a8
     int ret = 0;
9ae3a8
+    VFIODevice *vbasedev = region->vbasedev;
9ae3a8
 
9ae3a8
-    if (VFIO_ALLOW_MMAP && size && bar->flags & VFIO_REGION_INFO_FLAG_MMAP) {
9ae3a8
+    if (VFIO_ALLOW_MMAP && size && region->flags &
9ae3a8
+        VFIO_REGION_INFO_FLAG_MMAP) {
9ae3a8
         int prot = 0;
9ae3a8
 
9ae3a8
-        if (bar->flags & VFIO_REGION_INFO_FLAG_READ) {
9ae3a8
+        if (region->flags & VFIO_REGION_INFO_FLAG_READ) {
9ae3a8
             prot |= PROT_READ;
9ae3a8
         }
9ae3a8
 
9ae3a8
-        if (bar->flags & VFIO_REGION_INFO_FLAG_WRITE) {
9ae3a8
+        if (region->flags & VFIO_REGION_INFO_FLAG_WRITE) {
9ae3a8
             prot |= PROT_WRITE;
9ae3a8
         }
9ae3a8
 
9ae3a8
         *map = mmap(NULL, size, prot, MAP_SHARED,
9ae3a8
-                    bar->fd, bar->fd_offset + offset);
9ae3a8
+                    vbasedev->fd, region->fd_offset + offset);
9ae3a8
         if (*map == MAP_FAILED) {
9ae3a8
             *map = NULL;
9ae3a8
             ret = -errno;
9ae3a8
@@ -2635,7 +2638,7 @@ empty_region:
9ae3a8
 static void vfio_map_bar(VFIOPCIDevice *vdev, int nr)
9ae3a8
 {
9ae3a8
     VFIOBAR *bar = &vdev->bars[nr];
9ae3a8
-    uint64_t size = bar->size;
9ae3a8
+    uint64_t size = bar->region.size;
9ae3a8
     char name[64];
9ae3a8
     uint32_t pci_bar;
9ae3a8
     uint8_t type;
9ae3a8
@@ -2665,9 +2668,9 @@ static void vfio_map_bar(VFIOPCIDevice *vdev, int nr)
9ae3a8
                                     ~PCI_BASE_ADDRESS_MEM_MASK);
9ae3a8
 
9ae3a8
     /* A "slow" read/write mapping underlies all BARs */
9ae3a8
-    memory_region_init_io(&bar->mem, &vfio_bar_ops,
9ae3a8
+    memory_region_init_io(&bar->region.mem, &vfio_region_ops,
9ae3a8
                           bar, name, size);
9ae3a8
-    pci_register_bar(&vdev->pdev, nr, type, &bar->mem);
9ae3a8
+    pci_register_bar(&vdev->pdev, nr, type, &bar->region.mem);
9ae3a8
 
9ae3a8
     /*
9ae3a8
      * We can't mmap areas overlapping the MSIX vector table, so we
9ae3a8
@@ -2678,8 +2681,9 @@ static void vfio_map_bar(VFIOPCIDevice *vdev, int nr)
9ae3a8
     }
9ae3a8
 
9ae3a8
     strncat(name, " mmap", sizeof(name) - strlen(name) - 1);
9ae3a8
-    if (vfio_mmap_bar(vdev, bar, &bar->mem,
9ae3a8
-                      &bar->mmap_mem, &bar->mmap, size, 0, name)) {
9ae3a8
+    if (vfio_mmap_region(OBJECT(vdev), &bar->region, &bar->region.mem,
9ae3a8
+                      &bar->region.mmap_mem, &bar->region.mmap,
9ae3a8
+                      size, 0, name)) {
9ae3a8
         error_report("%s unsupported. Performance may be slow", name);
9ae3a8
     }
9ae3a8
 
9ae3a8
@@ -2689,10 +2693,11 @@ static void vfio_map_bar(VFIOPCIDevice *vdev, int nr)
9ae3a8
         start = TARGET_PAGE_ALIGN((uint64_t)vdev->msix->table_offset +
9ae3a8
                                   (vdev->msix->entries * PCI_MSIX_ENTRY_SIZE));
9ae3a8
 
9ae3a8
-        size = start < bar->size ? bar->size - start : 0;
9ae3a8
+        size = start < bar->region.size ? bar->region.size - start : 0;
9ae3a8
         strncat(name, " msix-hi", sizeof(name) - strlen(name) - 1);
9ae3a8
         /* VFIOMSIXInfo contains another MemoryRegion for this mapping */
9ae3a8
-        if (vfio_mmap_bar(vdev, bar, &bar->mem, &vdev->msix->mmap_mem,
9ae3a8
+        if (vfio_mmap_region(OBJECT(vdev), &bar->region, &bar->region.mem,
9ae3a8
+                          &vdev->msix->mmap_mem,
9ae3a8
                           &vdev->msix->mmap, size, start, name)) {
9ae3a8
             error_report("%s unsupported. Performance may be slow", name);
9ae3a8
         }
9ae3a8
@@ -3281,6 +3286,7 @@ static void vfio_pci_compute_needs_reset(VFIODevice *vbasedev)
9ae3a8
 static VFIODeviceOps vfio_pci_ops = {
9ae3a8
     .vfio_compute_needs_reset = vfio_pci_compute_needs_reset,
9ae3a8
     .vfio_hot_reset_multi = vfio_pci_hot_reset_multi,
9ae3a8
+    .vfio_eoi = vfio_eoi,
9ae3a8
 };
9ae3a8
 
9ae3a8
 static void vfio_reset_handler(void *opaque)
9ae3a8
@@ -3607,11 +3613,11 @@ static int vfio_get_device(VFIOGroup *group, const char *name,
9ae3a8
                 (unsigned long)reg_info.size, (unsigned long)reg_info.offset,
9ae3a8
                 (unsigned long)reg_info.flags);
9ae3a8
 
9ae3a8
-        vdev->bars[i].flags = reg_info.flags;
9ae3a8
-        vdev->bars[i].size = reg_info.size;
9ae3a8
-        vdev->bars[i].fd_offset = reg_info.offset;
9ae3a8
-        vdev->bars[i].fd = vdev->vbasedev.fd;
9ae3a8
-        vdev->bars[i].nr = i;
9ae3a8
+        vdev->bars[i].region.vbasedev = &vdev->vbasedev;
9ae3a8
+        vdev->bars[i].region.flags = reg_info.flags;
9ae3a8
+        vdev->bars[i].region.size = reg_info.size;
9ae3a8
+        vdev->bars[i].region.fd_offset = reg_info.offset;
9ae3a8
+        vdev->bars[i].region.nr = i;
9ae3a8
         QLIST_INIT(&vdev->bars[i].quirks);
9ae3a8
     }
9ae3a8
 
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8