svn commit: r270745 - in head: bin/ps sys/compat/freebsd32 sys/kern sys/sys

[Mateusz Guzik]

On Fri, Aug 29, 2014 at 01:17:45PM -0400, John Baldwin wrote:
> On Thursday, August 28, 2014 08:41:12 AM Mateusz Guzik wrote:
> > Author: mjg
> > Date: Thu Aug 28 08:41:11 2014
> > New Revision: 270745
> > URL: http://svnweb.freebsd.org/changeset/base/270745
> > 
> > Log:
> >   Return real parent pid in kinfo (used by e.g. ps)
> > 
> >   Add a separate field which exports tracer pid and add a new keyword
> >   ("tracer") for ps to display it.
> > 
> >   This is a follow up to r270444.
> 
> Note that proctree_lock is now needed in several more places such as dtrace 
> (which uses fill_kinfo_proc()) and the ELF coredump code 
> (elf(N)_note_procstat_proc() calls kern_proc_out()).

Weird, I was sure I checked all callers. Indeed missed them, will fix
that up later.

Maybe we should reconsider an incomplete approach with storing the
pointer, which would require only proc lock (i.e. what was required
prior to any changes)

The hack I had was:
https://people.freebsd.org/~mjg/patches/ptrace-hack.diff

-- 
Mateusz Guzik <mjguzik gmail.com>