On 06/25, Tetsuo Handa wrote: > > Michal Hocko wrote: > > --- a/mm/oom_kill.c > > +++ b/mm/oom_kill.c > > @@ -295,7 +295,8 @@ enum oom_scan_t oom_scan_process_thread(struct oom_control *oc, > > if (test_bit(MMF_OOM_REAPED, &p->mm->flags)) > > ret = OOM_SCAN_CONTINUE; > > task_unlock(p); > > - } > > + } else if (task->state == EXIT_ZOMBIE) ^^^^^ you meant exit_state ;) > > + ret = OOM_SCAN_CONTINUE; > > I think EXIT_ZOMBIE is too late, for it is exit_notify() stage from do_exit() > which sets EXIT_ZOMBIE state. Yes, and in any case nobody but exit/wait/ptrace code should ever look at ->exit_state, not to mention EXIT_ZOMBIE/DEAD/WHATEVER codes. Oleg. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>