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

Rafael J. Wysocki
2009-05-08T13:42:47Z

On Friday 08 May 2009, KAMEZAWA Hiroyuki wrote:
> On Fri, 8 May 2009 01:11:30 +0200
> "Rafael J. Wysocki" <rjw-KKrjLPT3xs0@public.gmane.org> 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