Re: [PATCH v3] mm, oom: Fix race when selecting process to kill

2013-11-09 Thread Oleg Nesterov
On 11/08, Sameer Nanda wrote: > > @@ -413,12 +413,20 @@ void oom_kill_process(struct task_struct *p, gfp_t > gfp_mask, int order, > DEFAULT_RATELIMIT_BURST); > @@ -456,10 +463,18 @@ void oom_kill_process(struct task_struct *p, gfp_t > gfp_mask, int

Re: [PATCH v3] mm, oom: Fix race when selecting process to kill

2013-11-09 Thread Oleg Nesterov
On 11/08, Sameer Nanda wrote: @@ -413,12 +413,20 @@ void oom_kill_process(struct task_struct *p, gfp_t gfp_mask, int order, DEFAULT_RATELIMIT_BURST); @@ -456,10 +463,18 @@ void oom_kill_process(struct task_struct *p, gfp_t gfp_mask, int order,

[PATCH v3] mm, oom: Fix race when selecting process to kill

2013-11-08 Thread Sameer Nanda
The selection of the process to be killed happens in two spots: first in select_bad_process and then a further refinement by looking for child processes in oom_kill_process. Since this is a two step process, it is possible that the process selected by select_bad_process may get a SIGKILL just

[PATCH v3] mm, oom: Fix race when selecting process to kill

2013-11-08 Thread Sameer Nanda
The selection of the process to be killed happens in two spots: first in select_bad_process and then a further refinement by looking for child processes in oom_kill_process. Since this is a two step process, it is possible that the process selected by select_bad_process may get a SIGKILL just