[Xen-devel] [PATCH] xend: destroy restored domain when its device doesn't exist

[Kouya Shimura <kouya@xxxxxxxxxxxxxx>]

A migrated domain keeps on running even though its disk doesn't exist.
This situation must be undesirable.

Signed-off-by: Kouya Shimura <kouya@xxxxxxxxxxxxxx>

diff -r 36159ea86bbd tools/python/xen/xend/XendCheckpoint.py
--- a/tools/python/xen/xend/XendCheckpoint.py   Tue Jan 19 13:59:28 2010 +0000
+++ b/tools/python/xen/xend/XendCheckpoint.py   Tue Jan 26 13:49:28 2010 +0900
@@ -337,19 +337,18 @@ def restore(xd, fd, dominfo = None, paus
 
         try:
             dominfo.waitForDevices() # Wait for backends to set up
-        except Exception, exn:
-            log.exception(exn)
-
-        if lock:
-            XendDomain.instance().domains_lock.acquire()
+        finally:
+            if lock:
+                XendDomain.instance().domains_lock.acquire()
 
         if not paused:
             dominfo.unpause()
 
         return dominfo
-    except:
+    except Exception, exn:
         dominfo.destroy()
-        raise
+        log.exception(exn)
+        raise exn
 
 
 class RestoreInputHandler:

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel