[PATCH] select_bad_process(): cleanup 'releasing' check
authorOleg Nesterov <oleg@tv-sign.ru>
Fri, 29 Sep 2006 09:01:12 +0000 (02:01 -0700)
committerLinus Torvalds <torvalds@g5.osdl.org>
Fri, 29 Sep 2006 16:18:21 +0000 (09:18 -0700)
No logic changes, but imho easier to read.

Signed-off-by: Oleg Nesterov <oleg@tv-sign.ru>
Acked-by: Nick Piggin <npiggin@suse.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
mm/oom_kill.c

index 423dcae323a54cee90817df40cf4fdcf26b1b1dd..991bf0cf47780650f417b07778b5ae7d4ed9963a 100644 (file)
@@ -204,7 +204,6 @@ static struct task_struct *select_bad_process(unsigned long *ppoints)
        do_posix_clock_monotonic_gettime(&uptime);
        do_each_thread(g, p) {
                unsigned long points;
-               int releasing;
 
                /*
                 * skip kernel threads and tasks which have already released
@@ -226,16 +225,15 @@ static struct task_struct *select_bad_process(unsigned long *ppoints)
                 * the process of exiting and releasing its resources.
                 * Otherwise we could get an OOM deadlock.
                 */
-               releasing = test_tsk_thread_flag(p, TIF_MEMDIE) ||
-                                               p->flags & PF_EXITING;
-               if (releasing) {
-                       if (p->flags & PF_EXITING && p == current) {
-                               chosen = p;
-                               *ppoints = ULONG_MAX;
-                               break;
-                       }
-                       return ERR_PTR(-1UL);
+               if ((p->flags & PF_EXITING) && p == current) {
+                       chosen = p;
+                       *ppoints = ULONG_MAX;
+                       break;
                }
+               if ((p->flags & PF_EXITING) ||
+                               test_tsk_thread_flag(p, TIF_MEMDIE))
+                       return ERR_PTR(-1UL);
+
                if (p->oomkilladj == OOM_DISABLE)
                        continue;
 
@@ -245,6 +243,7 @@ static struct task_struct *select_bad_process(unsigned long *ppoints)
                        *ppoints = points;
                }
        } while_each_thread(g, p);
+
        return chosen;
 }