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

2013-11-14 Thread Sameer Nanda
On Thu, Nov 14, 2013 at 5:43 AM, dserrg wrote: > (sorry for html) > > Why do we even bother with locking? > Why not just merge my original patch? (The link is in Vladimir's message) > It provides much more elegant (and working!) solution for this problem. As Oleg alluded to in that thread, that p

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

2013-11-13 Thread Oleg Nesterov
On 11/13, Sameer Nanda wrote: > > 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 > selec

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

2013-11-13 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 befor