diff options
author | 2009-12-15 16:47:12 -0800 | |
---|---|---|
committer | 2009-12-16 07:20:07 -0800 | |
commit | d31f56dbf8bafaacb0c617f9a6f137498d5c7aed (patch) | |
tree | 88d095c2208d27362e58ff7431407040ead9d848 /kernel/fork.c | |
parent | memcg: cleanup mem_cgroup_move_parent() (diff) | |
download | wireguard-linux-d31f56dbf8bafaacb0c617f9a6f137498d5c7aed.tar.xz wireguard-linux-d31f56dbf8bafaacb0c617f9a6f137498d5c7aed.zip |
memcg: avoid oom-killing innocent task in case of use_hierarchy
task_in_mem_cgroup(), which is called by select_bad_process() to check
whether a task can be a candidate for being oom-killed from memcg's limit,
checks "curr->use_hierarchy"("curr" is the mem_cgroup the task belongs
to).
But this check return true(it's false positive) when:
<some path>/aa use_hierarchy == 0 <- hitting limit
<some path>/aa/00 use_hierarchy == 1 <- the task belongs to
This leads to killing an innocent task in aa/00. This patch is a fix for
this bug. And this patch also fixes the arg for
mem_cgroup_print_oom_info(). We should print information of mem_cgroup
which the task being killed, not current, belongs to.
Signed-off-by: Daisuke Nishimura <nishimura@mxp.nes.nec.co.jp>
Acked-by: Balbir Singh <balbir@linux.vnet.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/fork.c')
0 files changed, 0 insertions, 0 deletions