[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090501160931.1dbd2abc.akpm@linux-foundation.org>
Date: Fri, 1 May 2009 16:09:31 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: "Rafael J. Wysocki" <rjw@...k.pl>
Cc: pavel@....cz, torvalds@...ux-foundation.org, jens.axboe@...cle.com,
alan-jenkins@...fmail.co.uk, linux-kernel@...r.kernel.org,
kernel-testers@...r.kernel.org, linux-pm@...ts.linux-foundation.org
Subject: Re: [PATCH 1/3] PM: Disable OOM killer during system-wide power
transitions
On Sat, 2 May 2009 00:27:30 +0200
"Rafael J. Wysocki" <rjw@...k.pl> wrote:
> From: Rafael J. Wysocki <rjw@...k.pl>
>
> The OOM killer is not particularly useful during system-wide power
> transitions, so do not use it if such a transition is in progress.
>
so... I think what you've done here is to arrange for the page
allocator to return NULL if we're hibernating rather than oom-killing,
yes?
Does the same apply to suspending? If so, why?
I think this is an OK change, as long as the only thing which is
allocating memory is hibernation itself. If random processes are still
doing random memory allocations at this time then their failed memory
allocation could be just as fatal as an oom-killing. Moreso if they're
s/bin/init or whatever.
So is it the case that pm_transition_in_progress() is only true during
the highly-constrained hibernation process? After everything is frozen?
If so, there are alternatives - the calling process could set
PF_DONT_KILL_ANYONE_FOR_ME, or could pass __GFP_DONT_KILL_ANYONE_FOR_ME.
Those might be worse alternatives, dunno - I'm just asking probing
questions ;)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists