Kyle McMartin 311bf8c
From beb764ac03e52eba1a654afb4273fab1f9de3cff Mon Sep 17 00:00:00 2001
Kyle McMartin 311bf8c
From: Kyle McMartin <kyle@mcmartin.ca>
Kyle McMartin 311bf8c
Date: Mon, 29 Nov 2010 20:59:14 -0500
Kyle McMartin 311bf8c
Subject: [PATCH] linux-2.6-debug-vm-would_have_oomkilled
Jesse Keating 7a32965
Jesse Keating 7a32965
---
Kyle McMartin 311bf8c
 include/linux/oom.h |    1 +
Kyle McMartin 311bf8c
 kernel/sysctl.c     |    7 +++++++
Kyle McMartin 311bf8c
 mm/oom_kill.c       |    8 ++++++++
Kyle McMartin 311bf8c
 3 files changed, 16 insertions(+), 0 deletions(-)
Jesse Keating 7a32965
Kyle McMartin 311bf8c
diff --git a/include/linux/oom.h b/include/linux/oom.h
Kyle McMartin 311bf8c
index 5e3aa83..79a27b4 100644
Kyle McMartin 311bf8c
--- a/include/linux/oom.h
Kyle McMartin 311bf8c
+++ b/include/linux/oom.h
Kyle McMartin 311bf8c
@@ -72,5 +72,6 @@ extern struct task_struct *find_lock_task_mm(struct task_struct *p);
Kyle McMartin 311bf8c
 extern int sysctl_oom_dump_tasks;
Kyle McMartin 311bf8c
 extern int sysctl_oom_kill_allocating_task;
Kyle McMartin 311bf8c
 extern int sysctl_panic_on_oom;
Kyle McMartin 311bf8c
+extern int sysctl_would_have_oomkilled;
Kyle McMartin 311bf8c
 #endif /* __KERNEL__*/
Kyle McMartin 311bf8c
 #endif /* _INCLUDE_LINUX_OOM_H */
Jesse Keating 7a32965
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
Kyle McMartin 311bf8c
index 5abfa15..a0fed6d 100644
Jesse Keating 7a32965
--- a/kernel/sysctl.c
Jesse Keating 7a32965
+++ b/kernel/sysctl.c
Kyle McMartin 311bf8c
@@ -1000,6 +1000,13 @@ static struct ctl_table vm_table[] = {
Jesse Keating 7a32965
 		.proc_handler	= proc_dointvec,
Jesse Keating 7a32965
 	},
Jesse Keating 7a32965
 	{
Jesse Keating 7a32965
+		.procname	= "would_have_oomkilled",
Jesse Keating 7a32965
+		.data		= &sysctl_would_have_oomkilled,
Jesse Keating 7a32965
+		.maxlen		= sizeof(sysctl_would_have_oomkilled),
Jesse Keating 7a32965
+		.mode		= 0644,
Kyle McMartin 311bf8c
+		.proc_handler	= proc_dointvec,
Jesse Keating 7a32965
+	},
Jesse Keating 7a32965
+	{
Jesse Keating 7a32965
 		.procname	= "overcommit_ratio",
Jesse Keating 7a32965
 		.data		= &sysctl_overcommit_ratio,
Jesse Keating 7a32965
 		.maxlen		= sizeof(sysctl_overcommit_ratio),
Jesse Keating 7a32965
diff --git a/mm/oom_kill.c b/mm/oom_kill.c
Kyle McMartin 311bf8c
index 7dcca55..281ac39 100644
Jesse Keating 7a32965
--- a/mm/oom_kill.c
Jesse Keating 7a32965
+++ b/mm/oom_kill.c
Kyle McMartin 311bf8c
@@ -35,6 +35,7 @@
Jesse Keating 7a32965
 int sysctl_panic_on_oom;
Jesse Keating 7a32965
 int sysctl_oom_kill_allocating_task;
Kyle McMartin 311bf8c
 int sysctl_oom_dump_tasks = 1;
Jesse Keating 7a32965
+int sysctl_would_have_oomkilled;
Jesse Keating 7a32965
 static DEFINE_SPINLOCK(zone_scan_lock);
Jesse Keating 7a32965
 
Kyle McMartin 311bf8c
 #ifdef CONFIG_NUMA
Kyle McMartin 311bf8c
@@ -477,6 +478,13 @@ static int oom_kill_process(struct task_struct *p, gfp_t gfp_mask, int order,
Jesse Keating 7a32965
 	}
Jesse Keating 7a32965
 
Kyle McMartin 311bf8c
 	task_lock(p);
Kyle McMartin 311bf8c
+	if (sysctl_would_have_oomkilled) {
Kyle McMartin 311bf8c
+		printk(KERN_ERR "%s: would have killed process %d (%s), but continuing instead...\n",
Kyle McMartin 311bf8c
+			__func__, task_pid_nr(p), p->comm);
Kyle McMartin 311bf8c
+		task_unlock(p);
Kyle McMartin 311bf8c
+		return 0;
Jesse Keating 7a32965
+	}
Jesse Keating 7a32965
+
Kyle McMartin 311bf8c
 	pr_err("%s: Kill process %d (%s) score %d or sacrifice child\n",
Kyle McMartin 311bf8c
 		message, task_pid_nr(p), p->comm, points);
Kyle McMartin 311bf8c
 	task_unlock(p);
Jesse Keating 7a32965
-- 
Kyle McMartin 311bf8c
1.7.3.2
Jesse Keating 7a32965