Re: [PATCH 1/5] mm: Add __GFP_NO_OOM_KILL flag

From: Rafael J. Wysocki
Date: Fri May 08 2009 - 09:43:37 EST


On Friday 08 May 2009, KAMEZAWA Hiroyuki wrote:
> On Fri, 8 May 2009 01:11:30 +0200
> "Rafael J. Wysocki" <rjw@xxxxxxx> wrote:
>
> > + for_each_populated_zone(zone)
> > + zone_set_flag(zone, ZONE_OOM_LOCKED);
>
> > + for_each_populated_zone(zone)
> > + zone_clear_flag(zone, ZONE_OOM_LOCKED);
> > +
>
> Isn't it better to make above 2 be functions and move to mm/oom_kill.c ?

Hmm, OK. I'll do it.

Well, in fact snapshot.c is all about memory management, so perhaps it's a good
idea to move it into mm as a whole. ;-)

Best,
Rafael
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/