Re: [PATCH] proc_oom_score: remove tasklist_lock and pid_alive()

2016-06-09 Thread Michal Hocko
On Wed 08-06-16 23:19:21, Oleg Nesterov wrote: > This was needed before to ensure that ->signal != 0 and do_each_thread() > is safe, see the commit b95c35e76b29b for details. > > Today tsk->signal can't go away and for_each_thread(tsk) is always safe. > > Signed-off-by: Oleg Nesterov

Re: [PATCH] proc_oom_score: remove tasklist_lock and pid_alive()

2016-06-09 Thread Michal Hocko
On Wed 08-06-16 23:19:21, Oleg Nesterov wrote: > This was needed before to ensure that ->signal != 0 and do_each_thread() > is safe, see the commit b95c35e76b29b for details. > > Today tsk->signal can't go away and for_each_thread(tsk) is always safe. > > Signed-off-by: Oleg Nesterov Acked-by:

Re: [PATCH] proc_oom_score: remove tasklist_lock and pid_alive()

2016-06-08 Thread David Rientjes
On Wed, 8 Jun 2016, Oleg Nesterov wrote: > This was needed before to ensure that ->signal != 0 and do_each_thread() > is safe, see the commit b95c35e76b29b for details. > > Today tsk->signal can't go away and for_each_thread(tsk) is always safe. > > Signed-off-by: Oleg Nesterov

Re: [PATCH] proc_oom_score: remove tasklist_lock and pid_alive()

2016-06-08 Thread David Rientjes
On Wed, 8 Jun 2016, Oleg Nesterov wrote: > This was needed before to ensure that ->signal != 0 and do_each_thread() > is safe, see the commit b95c35e76b29b for details. > > Today tsk->signal can't go away and for_each_thread(tsk) is always safe. > > Signed-off-by: Oleg Nesterov Acked-by:

[PATCH] proc_oom_score: remove tasklist_lock and pid_alive()

2016-06-08 Thread Oleg Nesterov
This was needed before to ensure that ->signal != 0 and do_each_thread() is safe, see the commit b95c35e76b29b for details. Today tsk->signal can't go away and for_each_thread(tsk) is always safe. Signed-off-by: Oleg Nesterov --- fs/proc/base.c | 7 ++- 1 file changed, 2

[PATCH] proc_oom_score: remove tasklist_lock and pid_alive()

2016-06-08 Thread Oleg Nesterov
This was needed before to ensure that ->signal != 0 and do_each_thread() is safe, see the commit b95c35e76b29b for details. Today tsk->signal can't go away and for_each_thread(tsk) is always safe. Signed-off-by: Oleg Nesterov --- fs/proc/base.c | 7 ++- 1 file changed, 2 insertions(+), 5