On Mon, Jul 30, 2018 at 11:00:59AM -0700, Roman Gushchin wrote: > oom_kill_process() consists of two logical parts: the first one is > responsible for considering task's children as a potential victim and > printing the debug information. The second half is responsible for > sending SIGKILL to all tasks sharing the mm struct with the given victim. > > This commit splits oom_kill_process() with an intention to re-use the the > second half: __oom_kill_process(). > > The cgroup-aware OOM killer will kill multiple tasks belonging to the > victim cgroup. We don't need to print the debug information for the each > task, as well as play with task selection (considering task's children), > so we can't use the existing oom_kill_process(). > > Link: http://lkml.kernel.org/r/20171130152824.1591-2-guro@xxxxxx > Signed-off-by: Roman Gushchin <guro@xxxxxx> > Acked-by: Michal Hocko <mhocko@xxxxxxxx> > Acked-by: Johannes Weiner <hannes@xxxxxxxxxxx> > Acked-by: David Rientjes <rientjes@xxxxxxxxxx> > Cc: Vladimir Davydov <vdavydov.dev@xxxxxxxxx> > Cc: Tetsuo Handa <penguin-kernel@xxxxxxxxxxxxxxxxxxx> > Cc: David Rientjes <rientjes@xxxxxxxxxx> > Cc: Tejun Heo <tj@xxxxxxxxxx> > Cc: Michal Hocko <mhocko@xxxxxxxxxx> > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> This is pretty straight-forward. Acked-by: Johannes Weiner <hannes@xxxxxxxxxxx>
