Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Aug 2014 19:02:41 +0200
From:      Mateusz Guzik <mjguzik@gmail.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Mateusz Guzik <mjg@freebsd.org>
Subject:   Re: svn commit: r270444 - in head/sys: kern sys
Message-ID:  <20140825170241.GA23088@dft-labs.eu>
In-Reply-To: <1724027.iWxFDWcg2R@ralph.baldwin.cx>
References:  <201408240904.s7O949sI083660@svn.freebsd.org> <1724027.iWxFDWcg2R@ralph.baldwin.cx>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Aug 25, 2014 at 10:23:19AM -0400, John Baldwin wrote:
> On Sunday, August 24, 2014 09:04:09 AM Mateusz Guzik wrote:
> > Author: mjg
> > Date: Sun Aug 24 09:04:09 2014
> > New Revision: 270444
> > URL: http://svnweb.freebsd.org/changeset/base/270444
> > 
> > Log:
> >   Fix getppid for traced processes.
> > 
> >   Traced processes always have the tracer set as the parent.
> >   Utilize proc_realparent to obtain the right process when needed.
> 
> Are you sure this won't break things?  I know of several applications that 
> expect a debugger to be the parent when attached and change behavior as a 
> result (e.g. inserting a breakpoint on an assertion failure rather than 
> generating a core).
> 

Well, this is what linux and solaris do.

I don't feel strongly about this change. If you really want I'm happy to
revert.

-- 
Mateusz Guzik <mjguzik gmail.com>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140825170241.GA23088>