diff options
author | Anton Vorontsov <anton.vorontsov@linaro.org> | 2012-02-06 20:29:47 +0400 |
---|---|---|
committer | Simon Shields <keepcalm444@gmail.com> | 2016-06-13 14:47:34 +1000 |
commit | 02f83a692461160e7652c4ecaaa632f2339e0f53 (patch) | |
tree | b7ca39fe29a97f95ba59181570036b64fa52b239 /drivers | |
parent | e6edd4723129b31402ad2da3d579abfaf5286f4d (diff) | |
download | kernel_samsung_smdk4412-02f83a692461160e7652c4ecaaa632f2339e0f53.tar.gz kernel_samsung_smdk4412-02f83a692461160e7652c4ecaaa632f2339e0f53.tar.bz2 kernel_samsung_smdk4412-02f83a692461160e7652c4ecaaa632f2339e0f53.zip |
staging: android/lowmemorykiller: Better mm handling
LMK should not directly check for task->mm. The reason is that the
process' threads may exit or detach its mm via use_mm(), but other
threads may still have a valid mm. To catch this we use
find_lock_task_mm(), which walks up all threads and returns an
appropriate task (with lock held).
Suggested-by: Oleg Nesterov <oleg@redhat.com>
Reviewed-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org>
Acked-by: KOSAKI Motohiro <kosaki.motohiro@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/android/lowmemorykiller.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/drivers/staging/android/lowmemorykiller.c b/drivers/staging/android/lowmemorykiller.c index d091c801a3d..f3219f40892 100644 --- a/drivers/staging/android/lowmemorykiller.c +++ b/drivers/staging/android/lowmemorykiller.c @@ -99,7 +99,7 @@ task_notify_func(struct notifier_block *self, unsigned long val, void *data) static int lowmem_shrink(struct shrinker *s, struct shrink_control *sc) { - struct task_struct *p; + struct task_struct *tsk; #ifdef ENHANCED_LMK_ROUTINE struct task_struct *selected[LOWMEM_DEATHPENDING_DEPTH] = {NULL,}; #else @@ -180,17 +180,19 @@ static int lowmem_shrink(struct shrinker *s, struct shrink_control *sc) #endif rcu_read_lock(); - for_each_process(p) { - struct mm_struct *mm; + for_each_process(tsk) { + struct task_struct *p; struct signal_struct *sig; int oom_adj; #ifdef ENHANCED_LMK_ROUTINE int is_exist_oom_task = 0; #endif - task_lock(p); - mm = p->mm; + p = find_lock_task_mm(tsk); + if (!p) + continue; + sig = p->signal; - if (!mm || !sig) { + if (!sig) { task_unlock(p); continue; } @@ -199,7 +201,7 @@ static int lowmem_shrink(struct shrinker *s, struct shrink_control *sc) task_unlock(p); continue; } - tasksize = get_mm_rss(mm); + tasksize = get_mm_rss(p->mm); task_unlock(p); if (tasksize <= 0) continue; |