From: Osier Yang <jyang@redhat.com>
Date: Fri, 19 Aug 2011 13:46:22 +0000 (+0800)
Subject: libxl: Allow to undefine a running domain.
X-Git-Tag: v0.9.5-rc1~204
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0f407570b446aa68ab958686da76746ec4c18972;p=thirdparty%2Flibvirt.git

libxl: Allow to undefine a running domain.

Undefining a running domain will convert it to trasient, but keep
the domain still running.
---

diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
index 516148f740..d6e0c28f22 100644
--- a/src/libxl/libxl_driver.c
+++ b/src/libxl/libxl_driver.c
@@ -2802,12 +2802,6 @@ libxlDomainUndefineFlags(virDomainPtr dom,
         goto cleanup;
     }
 
-    if (virDomainObjIsActive(vm)) {
-        libxlError(VIR_ERR_OPERATION_INVALID,
-                   "%s", _("cannot undefine active domain"));
-        goto cleanup;
-    }
-
     if (!vm->persistent) {
         libxlError(VIR_ERR_OPERATION_INVALID,
                    "%s", _("cannot undefine transient domain"));
@@ -2841,8 +2835,13 @@ libxlDomainUndefineFlags(virDomainPtr dom,
     event = virDomainEventNewFromObj(vm, VIR_DOMAIN_EVENT_UNDEFINED,
                                      VIR_DOMAIN_EVENT_UNDEFINED_REMOVED);
 
-    virDomainRemoveInactive(&driver->domains, vm);
-    vm = NULL;
+    if (virDomainObjIsActive(vm)) {
+        vm->persistent = 0;
+    } else {
+        virDomainRemoveInactive(&driver->domains, vm);
+        vm = NULL;
+    }
+
     ret = 0;
 
   cleanup: