[libvirt] [PATCH v2 3/4] qemu: process: Simplify acces to individual disk when reconnecting

Peter Krempa pkrempa at redhat.com
Thu Oct 5 12:06:19 UTC 2017


Add a helper variable so that we don't have to access the disk via 3
indirections.
---
 src/qemu/qemu_process.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index f29228bc1..4353ad5ec 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -6969,21 +6969,21 @@ qemuProcessReconnect(void *opaque)
      * qemu_driver->sharedDevices.
      */
     for (i = 0; i < obj->def->ndisks; i++) {
+        virDomainDiskDefPtr disk = obj->def->disks[i];
         virDomainDeviceDef dev;

-        if (virStorageTranslateDiskSourcePool(conn, obj->def->disks[i]) < 0)
+        if (virStorageTranslateDiskSourcePool(conn, disk) < 0)
             goto error;

         /* XXX we should be able to restore all data from XML in the future.
          * This should be the only place that calls qemuDomainDetermineDiskChain
          * with @report_broken == false to guarantee best-effort domain
          * reconnect */
-        if (qemuDomainDetermineDiskChain(driver, obj, obj->def->disks[i],
-                                         true, false) < 0)
+        if (qemuDomainDetermineDiskChain(driver, obj, disk, true, false) < 0)
             goto error;

         dev.type = VIR_DOMAIN_DEVICE_DISK;
-        dev.data.disk = obj->def->disks[i];
+        dev.data.disk = disk;
         if (qemuAddSharedDevice(driver, &dev, obj->def->name) < 0)
             goto error;
     }
-- 
2.14.1




More information about the libvir-list mailing list