From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Daniel P. Berrange" Subject: Re: [libvirt] [Xen-devel] [PATCH 00/12] libxl: fork: SIGCHLD flexibility Date: Thu, 30 Jan 2014 12:18:05 +0000 Message-ID: <20140130121805.GI3139@redhat.com> References: <1389975845-1195-1-git-send-email-ian.jackson@eu.citrix.com> <52D9AECF.6050309@suse.com> <52DD678F.3070504@suse.com> <21214.37402.648941.864060@mariner.uk.xensource.com> <52DF57E2.2090602@suse.com> <52E09513.6060603@suse.com> <21216.62800.746512.422459@mariner.uk.xensource.com> <52E1EB97.4080007@suse.com> <21218.24466.92095.134875@mariner.uk.xensource.com> <52E70A58.2060002@suse.com> Reply-To: "Daniel P. Berrange" Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <52E70A58.2060002@suse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com To: Jim Fehlig Cc: LibVir , xen-devel@lists.xensource.com, Ian Jackson , Ian Campbell List-Id: xen-devel@lists.xenproject.org On Mon, Jan 27, 2014 at 06:39:36PM -0700, Jim Fehlig wrote: > [Adding libvirt list...] > > Ian Jackson wrote: > > Jim Fehlig writes ("Re: [Xen-devel] [PATCH 00/12] libxl: fork: SIGCHLD flexibility"): BTW, I meant to ask before - what is the SIGCHLD reference about in the subject line ? libvirt drivers that live inside libvirtd should never use or rely on the SIGCHLD signal at all. All VM processes started by libvirtd ought to be fully daemonized so that their parent is pid 1 / init. This ensures that the libvirtd daemon can be restarted without all the VMs getting reaped. Once the VMs are reparented to init, then libvirt or library code it uses has no way of ever receiving SIGCHLD. I'm unclear if the libvirt libxl driver is currently daemonizing the processes associated with VMs it starts, but if not, it really should do. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|