[PATCH 1/2] oom: do not live lock on frozen tasks

From: Michal Hocko
Date: Mon Sep 26 2011 - 04:28:44 EST


[Let's add some more people to CC list]

Sorry it took so long but I was quite bussy recently.

On Fri 26-08-11 11:13:40, David Rientjes wrote:
> On Fri, 26 Aug 2011, Michal Hocko wrote:
[...]
> > I am not saying the bonus is necessary, though. It depends on what
> > the freezer is used for (e.g. freeze a process which went wild and
> > debug what went wrong wouldn't welcome that somebody killed it or other
> > (mis)use which relies on D state).
> >
>
> I'd love to be able to do a thaw on a PF_FROZEN task in the oom killer
> followed by a SIGKILL if that task is selected for oom kill without an
> heuristic change. Not sure if that's possible, so we'll wait for Rafael
> to chime in.

We have discussed that with Rafael and it should be safe to do that. See
the patch bellow.
The only place I am not entirely sure about is run_guest
(drivers/lguest/core.c). It seems that the code is able to cope with
signals but it also calls lguest_arch_run_guest after try_to_freeze.
---