diff options
| author | Michal Hocko <mhocko@kernel.org> | 2016-10-08 01:57:32 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-10-08 03:46:26 +0200 |
| commit | 5870c2e1d78b043b69de3199469c056ca3b05102 (patch) | |
| tree | bf1d5f294d1f7816fad3f2a82d1ba253c3ac90c3 /mm | |
| parent | mm: memcontrol: add sanity checks for memcg->id.ref on get/put (diff) | |
| download | linux-5870c2e1d78b043b69de3199469c056ca3b05102.tar.xz linux-5870c2e1d78b043b69de3199469c056ca3b05102.zip | |
mm/oom_kill.c: fix task_will_free_mem() comment
Attempt to demystify the task_will_free_mem() loop.
Cc: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
| -rw-r--r-- | mm/oom_kill.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/mm/oom_kill.c b/mm/oom_kill.c index ef175518f05f..463cdd22d4e0 100644 --- a/mm/oom_kill.c +++ b/mm/oom_kill.c @@ -803,8 +803,9 @@ static bool task_will_free_mem(struct task_struct *task) return true; /* - * This is really pessimistic but we do not have any reliable way - * to check that external processes share with our mm + * Make sure that all tasks which share the mm with the given tasks + * are dying as well to make sure that a) nobody pins its mm and + * b) the task is also reapable by the oom reaper. */ rcu_read_lock(); for_each_process(p) { |