]> xenbits.xensource.com Git - libvirt.git/commitdiff
qemu: process: Move TLS setup for storage source to qemuProcessPrepareDomainStorage
authorPeter Krempa <pkrempa@redhat.com>
Tue, 3 Oct 2017 11:16:05 +0000 (13:16 +0200)
committerPeter Krempa <pkrempa@redhat.com>
Thu, 5 Oct 2017 07:45:59 +0000 (09:45 +0200)
src/qemu/qemu_domain.c
src/qemu/qemu_domain.h
src/qemu/qemu_process.c

index bf2ce29bfc4eba2448f333d78a9a0c7315e2039b..d3d5dbac66238e6a0c67e35df676041fda4bb704 100644 (file)
@@ -7720,29 +7720,6 @@ qemuDomainPrepareDiskSourceTLS(virStorageSourcePtr src,
 }
 
 
-/* qemuProcessPrepareDiskSource:
- * @def: live domain definition
- * @driver: qemu driver
- *
- * Returns 0 on success, -1 on failure
- */
-int
-qemuDomainPrepareDiskSource(virDomainDefPtr def,
-                            virQEMUDriverConfigPtr cfg)
-{
-    size_t i;
-
-    for (i = 0; i < def->ndisks; i++) {
-        if (qemuDomainPrepareDiskSourceTLS(def->disks[i]->src,
-                                           def->disks[i]->info.alias,
-                                           cfg) < 0)
-            return -1;
-    }
-
-    return 0;
-}
-
-
 int
 qemuDomainPrepareShmemChardev(virDomainShmemDefPtr shmem)
 {
index 23285967fabeb28e491226f08a74352c171f9d90..f92841ceb9394858c355bcb9885e4c8ab96e0197 100644 (file)
@@ -873,11 +873,6 @@ qemuDomainPrepareDiskSourceTLS(virStorageSourcePtr src,
                                virQEMUDriverConfigPtr cfg)
     ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(3);
 
-int
-qemuDomainPrepareDiskSource(virDomainDefPtr def,
-                            virQEMUDriverConfigPtr cfg)
-    ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
-
 int qemuDomainPrepareShmemChardev(virDomainShmemDefPtr shmem)
     ATTRIBUTE_NONNULL(1);
 
index f160933e245ac2d9379298f03b14dfb7b027cc93..8fa53f7e2d366c162998cbc2ec23a2de4ba6d051 100644 (file)
@@ -5288,6 +5288,7 @@ static int
 qemuProcessPrepareDomainStorage(virConnectPtr conn,
                                 virQEMUDriverPtr driver,
                                 virDomainObjPtr vm,
+                                virQEMUDriverConfigPtr cfg,
                                 unsigned int flags)
 {
     size_t i;
@@ -5304,6 +5305,9 @@ qemuProcessPrepareDomainStorage(virConnectPtr conn,
             /* disk source was dropped */
             continue;
         }
+
+        if (qemuDomainPrepareDiskSourceTLS(disk->src, disk->info.alias, cfg) < 0)
+            return -1;
     }
 
     return 0;
@@ -5387,7 +5391,7 @@ qemuProcessPrepareDomain(virConnectPtr conn,
         goto cleanup;
 
     VIR_DEBUG("Setting up storage");
-    if (qemuProcessPrepareDomainStorage(conn, driver, vm, flags) < 0)
+    if (qemuProcessPrepareDomainStorage(conn, driver, vm, cfg, flags) < 0)
         goto cleanup;
 
     /* Drop possibly missing disks from the definition. */
@@ -5398,10 +5402,6 @@ qemuProcessPrepareDomain(virConnectPtr conn,
     if (qemuDomainMasterKeyCreate(vm) < 0)
         goto cleanup;
 
-    VIR_DEBUG("Prepare disk source backends for TLS");
-    if (qemuDomainPrepareDiskSource(vm->def, cfg) < 0)
-        goto cleanup;
-
     VIR_DEBUG("Prepare chardev source backends for TLS");
     qemuDomainPrepareChardevSource(vm->def, cfg);