From owner-freebsd-gecko@FreeBSD.ORG  Fri Aug 19 15:28:46 2011
Return-Path: <owner-freebsd-gecko@FreeBSD.ORG>
Delivered-To: gecko@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 108E9106566C;
	Fri, 19 Aug 2011 15:28:46 +0000 (UTC) (envelope-from avg@FreeBSD.org)
Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140])
	by mx1.freebsd.org (Postfix) with ESMTP id EA2528FC16;
	Fri, 19 Aug 2011 15:28:44 +0000 (UTC)
Received: from odyssey.starpoint.kiev.ua (alpha-e.starpoint.kiev.ua
	[212.40.38.101])
	by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id SAA01722;
	Fri, 19 Aug 2011 18:28:43 +0300 (EEST)
	(envelope-from avg@FreeBSD.org)
Message-ID: <4E4E812B.6000007@FreeBSD.org>
Date: Fri, 19 Aug 2011 18:28:43 +0300
From: Andriy Gapon <avg@FreeBSD.org>
User-Agent: Mozilla/5.0 (X11; FreeBSD amd64;
	rv:5.0) Gecko/20110705 Thunderbird/5.0
MIME-Version: 1.0
To: Alex Dupre <ale@FreeBSD.org>
References: <201108181007.p7IA7PgK032094@repoman.freebsd.org>
	<4E4D6AD3.2050708@FreeBSD.org> <4E4D70FF.2060206@FreeBSD.org>
	<4E4DCFED.4070601@FreeBSD.org> <4E4E05BA.9070707@FreeBSD.org>
	<4E4E7A69.8050406@FreeBSD.org> <4E4E7D5D.4050204@FreeBSD.org>
In-Reply-To: <4E4E7D5D.4050204@FreeBSD.org>
X-Enigmail-Version: 1.2pre
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Cc: gecko@FreeBSD.org, Doug Barton <dougb@FreeBSD.org>
Subject: Re: cvs commit: ports UPDATING ports/mail/enigmail Makefile distinfo
 pkg-message ports/mail/enigmail-thunderbird Makefile
X-BeenThere: freebsd-gecko@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Gecko Rendering Engine issues <freebsd-gecko.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-gecko>,
	<mailto:freebsd-gecko-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-gecko>
List-Post: <mailto:freebsd-gecko@freebsd.org>
List-Help: <mailto:freebsd-gecko-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-gecko>,
	<mailto:freebsd-gecko-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 19 Aug 2011 15:28:46 -0000

on 19/08/2011 18:12 Andriy Gapon said the following:
> on 19/08/2011 17:59 Andriy Gapon said the following:
>> nsIPCService::RunPipe -> nsPipeTransport::Terminate -> nsPipeTransport::Kill ->
>> IPC_WaitProcess == _MD_WaitUnixProcess
> 
> BTW, curiously enough this how IPC_WaitProcess invocation looks in
> nsPipeTransport::Kill:
> 
> // Reap process (to avoid memory leaks in NSPR)
> // **NOTE** This could cause this (UI?) thread to hang
> status = IPC_WaitProcess(mProcess, &mExitCode);

I added some more printfs and here is very interesting stuff:

_MD_InitProcesses!
PR_CreateThread(WaitPidDaemonThread)
ForkAndExec, path = /usr/local/bin/gpg2
ForkAndExec, pid = 32094
ForkAndExec: numProcs 0 -> 1
waiting for children
ProcessReapedChildInternal, pid = 31969
ProcessReapedChildInternal: FindPidTable returned NULL
no children
before inputStream->Close()
before pipeTrans->Terminate()
before IPC_WaitProcess
_MD_WaitUnixProcess, pid = 32094
_MD_WaitUnixProcess: FindPidTable returned NULL

So apparently there was some "rogue" child process with pid 31969 which screwed up
the accounting of children and thus WaitPidDaemonThread is not aware that it
should call wait() to wait for pid 32094.
Apparently that rogue child process was not created via
PR_CreateProcess/_MD_CreateUnixProcess, otherwise it would be accounted for via
numProcs.
Hmm...

-- 
Andriy Gapon