Messages in this thread | | | Date | Fri, 17 Oct 2014 00:22:37 +0200 | From | Oleg Nesterov <> | Subject | Re: + freezer-check-oom-kill-while-being-frozen.patch added to -mm tree |
| |
On 10/16, Cong Wang wrote: > > On Thu, Oct 16, 2014 at 2:35 PM, Oleg Nesterov <oleg@redhat.com> wrote: > > > > If a task B is already frozen, it sleeps in D state. > > > > If OOM selects B as a victim after that, it won't be woken by > > SIGKILL, thus it obviously can't call should_thaw_current() and > > notice TIF_MEMDIE. > > I see your point now, it would be more clear if you can just quote > the patch instead of changelog. > > So are you saying the loop in __refrigerator() is useless?
No.
> Since > it will always stay in asleep after schedule()?
Not always. But it will stay asleep in this particular case.
> > Btw, I also do not understand the cgroup_freezing() check in > > should_thaw_current(), but this is another story. > > I hate to repeat the previous discussion. Maybe you can just follow > the link I gave to you? :)
May be, but this thread is huge. Will try tomorrow to read it tomorrow, but you know, I hope that someone else from cc list can copy-and-paste the relevant part of this discussion, or give me the link to some specific email. To me the comment should be more clear in any case, but perhaps it is just me who can't understand it immediately.
Oleg.
| |