9ae3a8
From 474ecfa2ca12dee5c07c974a166c27652b9c9a93 Mon Sep 17 00:00:00 2001
9ae3a8
From: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
Date: Fri, 29 Sep 2017 21:45:50 +0200
9ae3a8
Subject: [PATCH 13/27] vfio: Wrap VFIO_DEVICE_GET_REGION_INFO
9ae3a8
9ae3a8
RH-Author: Alex Williamson <alex.williamson@redhat.com>
9ae3a8
Message-id: <20170929214550.16765.34651.stgit@gimli.home>
9ae3a8
Patchwork-id: 76771
9ae3a8
O-Subject: [RHEL-7.5 qemu-kvm PATCH 13/16] vfio: Wrap VFIO_DEVICE_GET_REGION_INFO
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
Upstream: 469002263a295ec471c1498c3b456ccd9f85a841
9ae3a8
9ae3a8
In preparation for supporting capability chains on regions, wrap
9ae3a8
ioctl(VFIO_DEVICE_GET_REGION_INFO) so we don't duplicate the code for
9ae3a8
each caller.
9ae3a8
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 | 95 +++++++++++++++++++++++++++++++++++-----------------------
9ae3a8
 1 file changed, 58 insertions(+), 37 deletions(-)
9ae3a8
9ae3a8
diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
9ae3a8
index e2dc3f5..57a0065 100644
9ae3a8
--- a/hw/misc/vfio.c
9ae3a8
+++ b/hw/misc/vfio.c
9ae3a8
@@ -292,6 +292,8 @@ static uint32_t vfio_pci_read_config(PCIDevice *pdev, uint32_t addr, int len);
9ae3a8
 static void vfio_pci_write_config(PCIDevice *pdev, uint32_t addr,
9ae3a8
                                   uint32_t val, int len);
9ae3a8
 static void vfio_mmap_set_enabled(VFIOPCIDevice *vdev, bool enabled);
9ae3a8
+static int vfio_get_region_info(VFIODevice *vbasedev, int index,
9ae3a8
+                                struct vfio_region_info **info);
9ae3a8
 
9ae3a8
 /*
9ae3a8
  * Common VFIO interrupt disable
9ae3a8
@@ -1176,26 +1178,26 @@ static const MemoryRegionOps vfio_region_ops = {
9ae3a8
 
9ae3a8
 static void vfio_pci_load_rom(VFIOPCIDevice *vdev)
9ae3a8
 {
9ae3a8
-    struct vfio_region_info reg_info = {
9ae3a8
-        .argsz = sizeof(reg_info),
9ae3a8
-        .index = VFIO_PCI_ROM_REGION_INDEX
9ae3a8
-    };
9ae3a8
+    struct vfio_region_info *reg_info;
9ae3a8
     uint64_t size;
9ae3a8
     off_t off = 0;
9ae3a8
     size_t bytes;
9ae3a8
 
9ae3a8
-    if (ioctl(vdev->vbasedev.fd, VFIO_DEVICE_GET_REGION_INFO, ®_info)) {
9ae3a8
+    if (vfio_get_region_info(&vdev->vbasedev,
9ae3a8
+                             VFIO_PCI_ROM_REGION_INDEX, ®_info)) {
9ae3a8
         error_report("vfio: Error getting ROM info: %m");
9ae3a8
         return;
9ae3a8
     }
9ae3a8
 
9ae3a8
     DPRINTF("Device %s ROM:\n", vdev->vbasedev.name);
9ae3a8
     DPRINTF("  size: 0x%lx, offset: 0x%lx, flags: 0x%lx\n",
9ae3a8
-            (unsigned long)reg_info.size, (unsigned long)reg_info.offset,
9ae3a8
-            (unsigned long)reg_info.flags);
9ae3a8
+            (unsigned long)reg_info->size, (unsigned long)reg_info->offset,
9ae3a8
+            (unsigned long)reg_info->flags);
9ae3a8
+
9ae3a8
+    vdev->rom_size = size = reg_info->size;
9ae3a8
+    vdev->rom_offset = reg_info->offset;
9ae3a8
 
9ae3a8
-    vdev->rom_size = size = reg_info.size;
9ae3a8
-    vdev->rom_offset = reg_info.offset;
9ae3a8
+    g_free(reg_info);
9ae3a8
 
9ae3a8
     if (!vdev->rom_size) {
9ae3a8
         vdev->rom_read_failed = true;
9ae3a8
@@ -3483,7 +3485,7 @@ static int vfio_get_device(VFIOGroup *group, const char *name,
9ae3a8
                            VFIOPCIDevice *vdev)
9ae3a8
 {
9ae3a8
     struct vfio_device_info dev_info = { .argsz = sizeof(dev_info) };
9ae3a8
-    struct vfio_region_info reg_info = { .argsz = sizeof(reg_info) };
9ae3a8
+    struct vfio_region_info *reg_info;
9ae3a8
     struct vfio_irq_info irq_info = { .argsz = sizeof(irq_info) };
9ae3a8
     int ret, i;
9ae3a8
 
9ae3a8
@@ -3529,9 +3531,7 @@ static int vfio_get_device(VFIOGroup *group, const char *name,
9ae3a8
     }
9ae3a8
 
9ae3a8
     for (i = VFIO_PCI_BAR0_REGION_INDEX; i < VFIO_PCI_ROM_REGION_INDEX; i++) {
9ae3a8
-        reg_info.index = i;
9ae3a8
-
9ae3a8
-        ret = ioctl(vdev->vbasedev.fd, VFIO_DEVICE_GET_REGION_INFO, ®_info);
9ae3a8
+        ret = vfio_get_region_info(&vdev->vbasedev, i, ®_info);
9ae3a8
         if (ret) {
9ae3a8
             error_report("vfio: Error getting region %d info: %m", i);
9ae3a8
             goto error;
9ae3a8
@@ -3539,20 +3539,21 @@ static int vfio_get_device(VFIOGroup *group, const char *name,
9ae3a8
 
9ae3a8
         DPRINTF("Device %s region %d:\n", name, i);
9ae3a8
         DPRINTF("  size: 0x%lx, offset: 0x%lx, flags: 0x%lx\n",
9ae3a8
-                (unsigned long)reg_info.size, (unsigned long)reg_info.offset,
9ae3a8
-                (unsigned long)reg_info.flags);
9ae3a8
+                (unsigned long)reg_info->size, (unsigned long)reg_info->offset,
9ae3a8
+                (unsigned long)reg_info->flags);
9ae3a8
 
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.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
-    reg_info.index = VFIO_PCI_CONFIG_REGION_INDEX;
9ae3a8
+        g_free(reg_info);
9ae3a8
+    }
9ae3a8
 
9ae3a8
-    ret = ioctl(vdev->vbasedev.fd, VFIO_DEVICE_GET_REGION_INFO, ®_info);
9ae3a8
+    ret = vfio_get_region_info(&vdev->vbasedev,
9ae3a8
+                               VFIO_PCI_CONFIG_REGION_INDEX, ®_info);
9ae3a8
     if (ret) {
9ae3a8
         error_report("vfio: Error getting config info: %m");
9ae3a8
         goto error;
9ae3a8
@@ -3560,41 +3561,43 @@ static int vfio_get_device(VFIOGroup *group, const char *name,
9ae3a8
 
9ae3a8
     DPRINTF("Device %s config:\n", name);
9ae3a8
     DPRINTF("  size: 0x%lx, offset: 0x%lx, flags: 0x%lx\n",
9ae3a8
-            (unsigned long)reg_info.size, (unsigned long)reg_info.offset,
9ae3a8
-            (unsigned long)reg_info.flags);
9ae3a8
+            (unsigned long)reg_info->size, (unsigned long)reg_info->offset,
9ae3a8
+            (unsigned long)reg_info->flags);
9ae3a8
 
9ae3a8
-    vdev->config_size = reg_info.size;
9ae3a8
+    vdev->config_size = reg_info->size;
9ae3a8
     if (vdev->config_size == PCI_CONFIG_SPACE_SIZE) {
9ae3a8
         vdev->pdev.cap_present &= ~QEMU_PCI_CAP_EXPRESS;
9ae3a8
     }
9ae3a8
-    vdev->config_offset = reg_info.offset;
9ae3a8
+    vdev->config_offset = reg_info->offset;
9ae3a8
+
9ae3a8
+    g_free(reg_info);
9ae3a8
 
9ae3a8
     if ((vdev->features & VFIO_FEATURE_ENABLE_VGA) &&
9ae3a8
         dev_info.num_regions > VFIO_PCI_VGA_REGION_INDEX) {
9ae3a8
-        struct vfio_region_info vga_info = {
9ae3a8
-            .argsz = sizeof(vga_info),
9ae3a8
-            .index = VFIO_PCI_VGA_REGION_INDEX,
9ae3a8
-         };
9ae3a8
-
9ae3a8
-        ret = ioctl(vdev->vbasedev.fd, VFIO_DEVICE_GET_REGION_INFO, &vga_info);
9ae3a8
+        ret = vfio_get_region_info(&vdev->vbasedev,
9ae3a8
+                                   VFIO_PCI_VGA_REGION_INDEX, ®_info);
9ae3a8
         if (ret) {
9ae3a8
             error_report(
9ae3a8
                 "vfio: Device does not support requested feature x-vga");
9ae3a8
             goto error;
9ae3a8
         }
9ae3a8
 
9ae3a8
-        if (!(vga_info.flags & VFIO_REGION_INFO_FLAG_READ) ||
9ae3a8
-            !(vga_info.flags & VFIO_REGION_INFO_FLAG_WRITE) ||
9ae3a8
-            vga_info.size < 0xbffff + 1) {
9ae3a8
+        if (!(reg_info->flags & VFIO_REGION_INFO_FLAG_READ) ||
9ae3a8
+            !(reg_info->flags & VFIO_REGION_INFO_FLAG_WRITE) ||
9ae3a8
+            reg_info->size < 0xbffff + 1) {
9ae3a8
             error_report("vfio: Unexpected VGA info, flags 0x%lx, size 0x%lx",
9ae3a8
-                         (unsigned long)vga_info.flags,
9ae3a8
-                         (unsigned long)vga_info.size);
9ae3a8
+                         (unsigned long)reg_info->flags,
9ae3a8
+                         (unsigned long)reg_info->size);
9ae3a8
+            g_free(reg_info);
9ae3a8
+            ret = -1;
9ae3a8
             goto error;
9ae3a8
         }
9ae3a8
 
9ae3a8
-        vdev->vga.fd_offset = vga_info.offset;
9ae3a8
+        vdev->vga.fd_offset = reg_info->offset;
9ae3a8
         vdev->vga.fd = vdev->vbasedev.fd;
9ae3a8
 
9ae3a8
+        g_free(reg_info);
9ae3a8
+
9ae3a8
         vdev->vga.region[QEMU_PCI_VGA_MEM].offset = QEMU_PCI_VGA_MEM_BASE;
9ae3a8
         vdev->vga.region[QEMU_PCI_VGA_MEM].nr = QEMU_PCI_VGA_MEM;
9ae3a8
         QLIST_INIT(&vdev->vga.region[QEMU_PCI_VGA_MEM].quirks);
9ae3a8
@@ -3647,6 +3650,24 @@ static void vfio_put_device(VFIOPCIDevice *vdev)
9ae3a8
     }
9ae3a8
 }
9ae3a8
 
9ae3a8
+static int vfio_get_region_info(VFIODevice *vbasedev, int index,
9ae3a8
+                                struct vfio_region_info **info)
9ae3a8
+{
9ae3a8
+    size_t argsz = sizeof(struct vfio_region_info);
9ae3a8
+
9ae3a8
+    *info = g_malloc0(argsz);
9ae3a8
+
9ae3a8
+    (*info)->index = index;
9ae3a8
+    (*info)->argsz = argsz;
9ae3a8
+
9ae3a8
+    if (ioctl(vbasedev->fd, VFIO_DEVICE_GET_REGION_INFO, *info)) {
9ae3a8
+        g_free(*info);
9ae3a8
+        return -errno;
9ae3a8
+    }
9ae3a8
+
9ae3a8
+    return 0;
9ae3a8
+}
9ae3a8
+
9ae3a8
 static void vfio_err_notifier_handler(void *opaque)
9ae3a8
 {
9ae3a8
     VFIOPCIDevice *vdev = opaque;
9ae3a8
-- 
9ae3a8
1.8.3.1
9ae3a8