From mboxrd@z Thu Jan  1 00:00:00 1970
From: Roger Pau Monne <roger.pau@citrix.com>
Subject: Re: [PATCH 9/9] init/NetBSD: move xenbackendd to xend
	init script
Date: Tue, 17 Jul 2012 17:23:35 +0100
Message-ID: <50059187.8080802@citrix.com>
References: <1342002232-75531-1-git-send-email-roger.pau@citrix.com>
	<1342002232-75531-10-git-send-email-roger.pau@citrix.com>
	<20485.36379.841767.872741@mariner.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <20485.36379.841767.872741@mariner.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: Christoph Egger <Christoph.Egger@amd.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

Ian Jackson wrote:
> Roger Pau Monne writes ("[PATCH 9/9] init/NetBSD: move xenbackendd to xend init script"):
>> xenbackendd is not needed by the xl toolstack, so move it's launch to
>> the xend script.
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
> 
> on the basis that it doesn't touch non-BSD code.  But:
> 
>> +	printf "Stopping xenbackendd, xend\n"
>> +	xb_pid=$(check_pidfile ${XENBACKENDD_PIDFILE} ${SBINDIR}/xenbackendd)
>> +	xend_pid=`ps x -o pid,command|grep ${SBINDIR}/xend|awk '{ print $1 }'`
>> +	if test -n "$xb_pid";
>> +	then
>> +		kill -${sig_stop:-TERM} $xb_pid
> 
> This is pretty horrid.  Does BSD not have better ways to manage
> daemons ?

Yes, but xend seems to spawn two processes that regenerate, so you have
to kill them with one shot, or it will be useless. "xend stop" doesn't
work either, but since xend is deprecated I think it's rather useless to
try to fix this properly now.