Bastian Blank
2008-Mar-26 11:01 UTC
[Xen-devel] [PATCH 2/3] UNTESTED: xend - Use os.execvp for xenstored startup
# HG changeset patch # User Bastian Blank <waldi@debian.org> # 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@debian.org> 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@lists.xensource.com http://lists.xensource.com/xen-devel