We mark all VFIO devices as inactive after we've detached them from the
domain, so that step doesn't need to be performed afterwards.
---
 src/util/virhostdev.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/util/virhostdev.c b/src/util/virhostdev.c
index 1042ab3..b350de0 100644
--- a/src/util/virhostdev.c
+++ b/src/util/virhostdev.c
@@ -894,12 +894,10 @@ 
virHostdevExpandPCIDeviceListIOMMUGroups(virPCIDeviceListPtr list,
 static void
 virHostdevReattachPCIDevice(virPCIDevicePtr dev, virHostdevManagerPtr mgr)
 {
-    /* If the device is not managed and was attached to guest
-     * successfully, it must have been inactive.
-     */
+    /* We don't need to do anything for unmanaged devices */
     if (!virPCIDeviceGetManaged(dev)) {
-        if (virPCIDeviceListAdd(mgr->inactivePCIHostdevs, dev) < 0)
-            virPCIDeviceFree(dev);
+        VIR_DEBUG("Skipping reattach of unmanaged PCI device %s",
+                  virPCIDeviceGetName(dev));
         return;
     }
 
-- 
2.5.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to