From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38757) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VORy2-00075J-EH for qemu-devel@nongnu.org; Tue, 24 Sep 2013 08:43:55 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VORxx-00040S-3m for qemu-devel@nongnu.org; Tue, 24 Sep 2013 08:43:50 -0400 Received: from mail-ea0-x22a.google.com ([2a00:1450:4013:c01::22a]:63582) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VORxw-00040J-TK for qemu-devel@nongnu.org; Tue, 24 Sep 2013 08:43:45 -0400 Received: by mail-ea0-f170.google.com with SMTP id h14so2489772eak.1 for ; Tue, 24 Sep 2013 05:43:43 -0700 (PDT) Date: Tue, 24 Sep 2013 14:43:40 +0200 From: Stefan Hajnoczi Message-ID: <20130924124340.GA12579@stefanha-thinkpad.redhat.com> References: <1379947014-14216-1-git-send-email-borntraeger@de.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1379947014-14216-1-git-send-email-borntraeger@de.ibm.com> Subject: Re: [Qemu-devel] [PATCH] tracing: start trace processing thread in final child process List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Christian Borntraeger Cc: qemu-devel@nongnu.org, =?iso-8859-1?Q?Llu=EDs?= , Michael Mueller On Mon, Sep 23, 2013 at 04:36:54PM +0200, Christian Borntraeger wrote: > From: Michael Mueller > > When running with trace backend e.g. "simple" the writer thread needs to be > implemented in the same process context as the trace points that will be > processed. Under libvirtd control, qemu gets first started in daemonized > mode to privide its capabilities. Creating the writer thread in the initial > process context then leads to a dead lock because the thread gets termined > together with the initial parent. (-daemonize) > > Signed-off-by: Michael Mueller > Signed-off-by: Christian Borntraeger > [minor whitespace fixes] > --- > vl.c | 12 ++++++++++-- > 1 file changed, 10 insertions(+), 2 deletions(-) Thanks, applied to my tracing tree: https://github.com/stefanha/qemu/commits/tracing Stefan