Commit db2a0dd7 authored by Yaowei Bai's avatar Yaowei Bai Committed by Linus Torvalds

mm/oom_kill.c: introduce is_sysrq_oom helper

Introduce is_sysrq_oom helper function indicating oom kill triggered
by sysrq to improve readability.

No functional changes.
Signed-off-by: default avatarYaowei Bai <bywxiaobai@163.com>
Acked-by: default avatarDavid Rientjes <rientjes@google.com>
Acked-by: default avatarMichal Hocko <mhocko@suse.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 5bc23a0c
...@@ -118,6 +118,15 @@ struct task_struct *find_lock_task_mm(struct task_struct *p) ...@@ -118,6 +118,15 @@ struct task_struct *find_lock_task_mm(struct task_struct *p)
return t; return t;
} }
/*
* order == -1 means the oom kill is required by sysrq, otherwise only
* for display purposes.
*/
static inline bool is_sysrq_oom(struct oom_control *oc)
{
return oc->order == -1;
}
/* return true if the task is not adequate as candidate victim task. */ /* return true if the task is not adequate as candidate victim task. */
static bool oom_unkillable_task(struct task_struct *p, static bool oom_unkillable_task(struct task_struct *p,
struct mem_cgroup *memcg, const nodemask_t *nodemask) struct mem_cgroup *memcg, const nodemask_t *nodemask)
...@@ -265,7 +274,7 @@ enum oom_scan_t oom_scan_process_thread(struct oom_control *oc, ...@@ -265,7 +274,7 @@ enum oom_scan_t oom_scan_process_thread(struct oom_control *oc,
* Don't allow any other task to have access to the reserves. * Don't allow any other task to have access to the reserves.
*/ */
if (test_tsk_thread_flag(task, TIF_MEMDIE)) { if (test_tsk_thread_flag(task, TIF_MEMDIE)) {
if (oc->order != -1) if (!is_sysrq_oom(oc))
return OOM_SCAN_ABORT; return OOM_SCAN_ABORT;
} }
if (!task->mm) if (!task->mm)
...@@ -278,7 +287,7 @@ enum oom_scan_t oom_scan_process_thread(struct oom_control *oc, ...@@ -278,7 +287,7 @@ enum oom_scan_t oom_scan_process_thread(struct oom_control *oc,
if (oom_task_origin(task)) if (oom_task_origin(task))
return OOM_SCAN_SELECT; return OOM_SCAN_SELECT;
if (task_will_free_mem(task) && oc->order != -1) if (task_will_free_mem(task) && !is_sysrq_oom(oc))
return OOM_SCAN_ABORT; return OOM_SCAN_ABORT;
return OOM_SCAN_OK; return OOM_SCAN_OK;
...@@ -629,7 +638,7 @@ void check_panic_on_oom(struct oom_control *oc, enum oom_constraint constraint, ...@@ -629,7 +638,7 @@ void check_panic_on_oom(struct oom_control *oc, enum oom_constraint constraint,
return; return;
} }
/* Do not panic for oom kills triggered by sysrq */ /* Do not panic for oom kills triggered by sysrq */
if (oc->order == -1) if (is_sysrq_oom(oc))
return; return;
dump_header(oc, NULL, memcg); dump_header(oc, NULL, memcg);
panic("Out of memory: %s panic_on_oom is enabled\n", panic("Out of memory: %s panic_on_oom is enabled\n",
...@@ -709,7 +718,7 @@ bool out_of_memory(struct oom_control *oc) ...@@ -709,7 +718,7 @@ bool out_of_memory(struct oom_control *oc)
p = select_bad_process(oc, &points, totalpages); p = select_bad_process(oc, &points, totalpages);
/* Found nothing?!?! Either we hang forever, or we panic. */ /* Found nothing?!?! Either we hang forever, or we panic. */
if (!p && oc->order != -1) { if (!p && !is_sysrq_oom(oc)) {
dump_header(oc, NULL, NULL); dump_header(oc, NULL, NULL);
panic("Out of memory and no killable processes...\n"); panic("Out of memory and no killable processes...\n");
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment