Messages in this thread | | | Date | Wed, 23 Aug 2017 13:30:04 +0100 | From | Roman Gushchin <> | Subject | Re: [v5 1/4] mm, oom: refactor the oom_kill_process() function |
| |
On Tue, Aug 22, 2017 at 01:06:55PM -0400, Johannes Weiner wrote: > On Mon, Aug 14, 2017 at 07:32:09PM +0100, Roman Gushchin wrote: > > @@ -817,67 +817,12 @@ static bool task_will_free_mem(struct task_struct *task) > > return ret; > > } > > > > -static void oom_kill_process(struct oom_control *oc, const char *message) > > +static void __oom_kill_process(struct task_struct *victim) > > oom_kill_task()?
Not sure, as it kills all tasks which are sharing mm with the given task. Also, it will be confusing to have oom_kill_process() and oom_kill_task(), where the actual difference is in how much verbose they are, and if it's allowed to perfer a child process.
| |