Re: perf_event_pid() (Was: [PATCH] pids: make task_tgid_nr_ns() safe)

2017-08-21 Thread Peter Zijlstra
On Mon, Aug 21, 2017 at 05:36:27PM +0200, Oleg Nesterov wrote: > On 08/21, Oleg Nesterov wrote: > > > > This was reported many times, and this was even mentioned in commit > > 52ee2dfdd4f5 "pids: refactor vnr/nr_ns helpers to make them safe" but > > somehow nobody bothered to fix the obvious

Re: perf_event_pid() (Was: [PATCH] pids: make task_tgid_nr_ns() safe)

2017-08-21 Thread Peter Zijlstra
On Mon, Aug 21, 2017 at 05:36:27PM +0200, Oleg Nesterov wrote: > On 08/21, Oleg Nesterov wrote: > > > > This was reported many times, and this was even mentioned in commit > > 52ee2dfdd4f5 "pids: refactor vnr/nr_ns helpers to make them safe" but > > somehow nobody bothered to fix the obvious

perf_event_pid() (Was: [PATCH] pids: make task_tgid_nr_ns() safe)

2017-08-21 Thread Oleg Nesterov
On 08/21, Oleg Nesterov wrote: > > This was reported many times, and this was even mentioned in commit > 52ee2dfdd4f5 "pids: refactor vnr/nr_ns helpers to make them safe" but > somehow nobody bothered to fix the obvious problem: task_tgid_nr_ns() > is not safe because task->group_leader points to

perf_event_pid() (Was: [PATCH] pids: make task_tgid_nr_ns() safe)

2017-08-21 Thread Oleg Nesterov
On 08/21, Oleg Nesterov wrote: > > This was reported many times, and this was even mentioned in commit > 52ee2dfdd4f5 "pids: refactor vnr/nr_ns helpers to make them safe" but > somehow nobody bothered to fix the obvious problem: task_tgid_nr_ns() > is not safe because task->group_leader points to