[libvirt] [PATCH 1/3] qemu: Remove compiled out localhost migration support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Pretty sure this would deadlock now that we have proper locking, so
remove the code.

Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
---
 src/qemu/qemu_driver.c |   12 ------------
 1 files changed, 0 insertions(+), 12 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index e8606c8..7eed356 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -6298,20 +6298,8 @@ qemudDomainMigratePrepare2 (virConnectPtr dconn,
     /* Target domain name, maybe renamed. */
     dname = dname ? dname : def->name;
 
-#if 1
     /* Ensure the name and UUID don't already exist in an active VM */
     vm = virDomainFindByUUID(&driver->domains, def->uuid);
-#else
-    /* For TESTING ONLY you can change #if 1 -> #if 0 above and use
-     * this code which lets you do localhost migrations.  You must still
-     * supply a fresh 'dname' but this code assigns a random UUID.
-     */
-    if (virUUIDGenerate (def->uuid) == -1) {
-        qemudReportError (dconn, NULL, NULL, VIR_ERR_OPERATION_FAILED,
-            _("could not generate random UUID"));
-        goto cleanup;
-    }
-#endif
 
     if (!vm) vm = virDomainFindByName(&driver->domains, dname);
     if (vm) {
-- 
1.6.5.1

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]