WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

To: xen-devel@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-devel] [PATCH 2/3] UNTESTED: xend - Use os.execvp for xenstored startup
From: Bastian Blank <bastian@xxxxxxxxxxxx>
Date: Wed, 26 Mar 2008 12:01:10 +0100
Delivery-date: Wed, 26 Mar 2008 04:01:34 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.13 (2006-08-11)
# 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

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-devel] [PATCH 2/3] UNTESTED: xend - Use os.execvp for xenstored startup, Bastian Blank <=