[Xen-devel] [PATCH 2/3] UNTESTED: xend - Use os.execvp for xenstored startup

[Bastian Blank <bastian@xxxxxxxxxxxx>]

# HG changeset patch
# User Bastian Blank <waldi@xxxxxxxxxx>
# Date 1206528849 -3600
# Node ID 784e9c500e4e6b2d6a76b639d695c8eaa5c99b62
# Parent  5e0412c5f5798b5c0acdd4057c91b3820d6c4afd
Use os.execvp for xenstored startup.

A similar patch is used since long time in the Debian package.

Signed-off-by: Bastian Blank <waldi@xxxxxxxxxx>

diff -r 5e0412c5f579 -r 784e9c500e4e tools/misc/xend
--- a/tools/misc/xend   Wed Mar 26 11:54:09 2008 +0100
+++ b/tools/misc/xend   Wed Mar 26 11:54:09 2008 +0100
@@ -95,11 +95,11 @@ def start_xenstored():
         f.close()
     except:
         pass
-    XENSTORED_TRACE = os.getenv("XENSTORED_TRACE")
-    cmd = "xenstored --pid-file /var/run/xenstore.pid"
-    if XENSTORED_TRACE:
-        cmd += " -T /var/log/xen/xenstored-trace.log"
-    s,o = commands.getstatusoutput(cmd)
+    args = ['xenstored', "--pid-file", pidfname]
+    if os.getenv("XENSTORED_TRACE"):
+        args.extend(["-T", "/var/log/xen/xenstored-trace.log"])
+    if os.fork() == 0:
+        os.execvp('xenstored', args)
 
 def start_consoled():
     if os.fork() == 0:
-- 
There are always alternatives.
                -- Spock, "The Galileo Seven", stardate 2822.3

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