Re: [patch v2] oom: thaw threads if oom killed thread is frozen before deferring

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
sorry for the late reply but I was on vacation last week.

On Sat 08-10-11 03:23:15, KOSAKI Motohiro wrote:
> 2011/10/7 David Rientjes <rientjes@xxxxxxxxxx>:
> > If a thread has been oom killed and is frozen, thaw it before returning
> > to the page allocator.  Otherwise, it can stay frozen indefinitely and
> > no memory will be freed.
> >
> > Reported-by: Michal Hocko <mhocko@xxxxxxx>
> > Signed-off-by: David Rientjes <rientjes@xxxxxxxxxx>
> > ---
> >  v2: adds the missing header file include, the resend patch was based on a
> >     previous patch from Michal that is no longer needed if this is
> >     applied.
> 
> Looks ok to me.
> Michal, do you agree this patch?

The patch looks good but we still need other 2 patches
(http://comments.gmane.org/gmane.linux.kernel.mm/68578), right?

Anyway, I thought that we agreed on the other approach suggested by
Tejun (make frozen tasks oom killable without thawing). Even in that
case we want the first patch
(http://permalink.gmane.org/gmane.linux.kernel.mm/68576).
-- 
Michal Hocko
SUSE Labs
SUSE LINUX s.r.o.
Lihovarska 1060/12
190 00 Praha 9    
Czech Republic

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]