lkml.org 
[lkml]   [2014]   [Aug]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[Patch] freezer: check OOM kill signal while being frozen
Date
There is a race condition between OOM killer and freezer when
they try to operate on the same process, something like below:

Process A Process B Process C
trigger oom
B=oom_scan_process_thread()
cgroup_freezer_freeze(B)
...
try_to_freeze()
stay in D state
oom_kill_process(B)

In this case, process A triggers OOM and kernel selects process B
as the victim, right before being killed process B was frozen by
process C therefore went to D state, then kernel sent SIGKILL but
it is already too late as process B will never care about pending
signals any more. Fix this straightly by checking fatal pending signal
from OOM killer, so that the frozen process will recover itself
and then be killed finally.

Cc: David Rientjes <rientjes@google.com>
Cc: Michal Hocko <mhocko@suse.cz>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Tejun Heo <tj@kernel.org>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
---
diff --git a/kernel/freezer.c b/kernel/freezer.c
index aa6a8aa..c6d189d 100644
--- a/kernel/freezer.c
+++ b/kernel/freezer.c
@@ -68,7 +68,9 @@ bool __refrigerator(bool check_kthr_stop)
spin_lock_irq(&freezer_lock);
current->flags |= PF_FROZEN;
if (!freezing(current) ||
- (check_kthr_stop && kthread_should_stop()))
+ (check_kthr_stop && kthread_should_stop()) ||
+ (test_tsk_thread_flag(current, TIF_MEMDIE) &&
+ fatal_signal_pending(current)))
current->flags &= ~PF_FROZEN;
spin_unlock_irq(&freezer_lock);


\
 
 \ /
  Last update: 2014-08-09 03:01    [W:2.046 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site