Re: [PATCH v4] ptrace: Fix fork event messages across pid namespaces

2014-04-07 Thread Oleg Nesterov
On 04/03, Matthew Dempsky wrote: > > On Thu, Apr 3, 2014 at 8:44 AM, Oleg Nesterov wrote: > > > Some notes for potential future changes... > > > > - I do not not see any potential user of ptrace_event_pid() outside > > of fork.c, so perhaps this helper should not be exported. > >

Re: [PATCH v4] ptrace: Fix fork event messages across pid namespaces

2014-04-03 Thread Matthew Dempsky
On Thu, Apr 3, 2014 at 8:44 AM, Oleg Nesterov wrote: > On 04/02, Matthew Dempsky wrote: >> >> When tracing a process in another pid namespace, it's important for >> fork event messages to contain the child's pid as seen from the >> tracer's pid namespace, not the parent's. Otherwise, the tracer w

Re: [PATCH v4] ptrace: Fix fork event messages across pid namespaces

2014-04-03 Thread Oleg Nesterov
On 04/03, Oleg Nesterov wrote: > > Some notes for potential future changes... > > - I do not not see any potential user of ptrace_event_pid() outside > of fork.c, so perhaps this helper should not be exported. > > In fact I wouldn't mind if you send v5 which moves it into fork

Re: [PATCH v4] ptrace: Fix fork event messages across pid namespaces

2014-04-03 Thread Oleg Nesterov
On 04/02, Matthew Dempsky wrote: > > When tracing a process in another pid namespace, it's important for > fork event messages to contain the child's pid as seen from the > tracer's pid namespace, not the parent's. Otherwise, the tracer won't > be able to correlate the fork event with later SIGTRA