lkml.org 
[lkml]   [2014]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[RFC 3/4] OOM, PM: handle pm freezer as an OOM victim correctly
Date
PM freezer doesn't check whether it has been killed by OOM killer
after it disables OOM killer which means that it continues with the
suspend even though it should die as soon as possible. This has been
the case ever since PM suspend disables OOM killer and I suppose
it has ignored OOM even before.

This is not harmful though. The allocation which triggers OOM will
retry the allocation after a process is killed and the next attempt
will fail because the OOM killer will be disabled at the time so
there is no risk of an endless loop because the OOM victim doesn't
die.

But this is a correctness issue because no task should ignore OOM.
As suggested by Tejun, oom_killer_lock will return a success status
now. If the current task is pending fatal signals or TIF_MEMDIE is set
after oom_sem is taken then the caller should bail out and this is what
freeze_processes does with this patch.

Signed-off-by: Michal Hocko <mhocko@suse.cz>
---
include/linux/oom.h | 5 ++++-
kernel/power/process.c | 5 ++++-
mm/oom_kill.c | 12 +++++++++++-
3 files changed, 19 insertions(+), 3 deletions(-)

diff --git a/include/linux/oom.h b/include/linux/oom.h
index 8ca73c0b07df..8f4f634cc5b3 100644
--- a/include/linux/oom.h
+++ b/include/linux/oom.h
@@ -92,10 +92,13 @@ extern void oom_killer_enable(void);

/** oom_killer_lock - locks global OOM killer.
*
+ * Returns true on success and fails if the OOM killer couldn't be
+ * locked (e.g. because the current task has been killed before).
+ *
* This function should be used with an extreme care. No allocations
* are allowed with the lock held.
*/
-extern void oom_killer_lock(void);
+extern bool oom_killer_lock(void);

/** oom_killer_unlock - unlocks global OOM killer.
*/
diff --git a/kernel/power/process.c b/kernel/power/process.c
index 5c5da0fe54dd..49d8d84ccd6e 100644
--- a/kernel/power/process.c
+++ b/kernel/power/process.c
@@ -127,7 +127,10 @@ int freeze_processes(void)
* getting frozen to make sure none of them gets killed after
* try_to_freeze_tasks is done.
*/
- oom_killer_lock()
+ if (!oom_killer_lock()) {
+ usermodehelper_enable();
+ return -EBUSY;
+ }

/* Make sure this task doesn't get frozen */
current->flags |= PF_SUSPEND_TASK;
diff --git a/mm/oom_kill.c b/mm/oom_kill.c
index 0a061803be09..39a591092ca0 100644
--- a/mm/oom_kill.c
+++ b/mm/oom_kill.c
@@ -601,9 +601,19 @@ void oom_zonelist_unlock(struct zonelist *zonelist, gfp_t gfp_mask)
bool oom_killer_disabled __read_mostly;
static DECLARE_RWSEM(oom_sem);

-void oom_killer_lock(void)
+bool oom_killer_lock(void)
{
+ bool ret = true;
+
down_write(&oom_sem);
+
+ /* We might have been killed while waiting for the oom_sem. */
+ if (fatal_signal_pending(current) || test_thread_flag(TIF_MEMDIE)) {
+ up_write(&oom_sem);
+ ret = false;
+ }
+
+ return ret;
}

void oom_killer_unlock(void)
--
2.1.1


\
 
 \ /
  Last update: 2014-11-12 20:41    [W:0.188 / U:8.632 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site