Jesse Keating 7a32965
From 03657519851cd180983db4bd0c38eaeed4aa2962 Mon Sep 17 00:00:00 2001
Jesse Keating 7a32965
From: Kyle McMartin <kyle@treachery.i.jkkm.org>
Jesse Keating 7a32965
Date: Mon, 11 Jan 2010 08:25:12 -0500
Jesse Keating 7a32965
Subject: linux-2.6-debug-vm-would-have-oomkilled.patch
Jesse Keating 7a32965
Jesse Keating 7a32965
---
Jesse Keating 7a32965
 kernel/sysctl.c |    8 ++++++++
Jesse Keating 7a32965
 mm/oom_kill.c   |    7 +++++++
Jesse Keating 7a32965
 2 files changed, 15 insertions(+), 0 deletions(-)
Jesse Keating 7a32965
Jesse Keating 7a32965
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
Jesse Keating 7a32965
index 8a68b24..72a4ff1 100644
Jesse Keating 7a32965
--- a/kernel/sysctl.c
Jesse Keating 7a32965
+++ b/kernel/sysctl.c
Jesse Keating 7a32965
@@ -71,6 +71,7 @@ extern int sysctl_overcommit_ratio;
Jesse Keating 7a32965
 extern int sysctl_panic_on_oom;
Jesse Keating 7a32965
 extern int sysctl_oom_kill_allocating_task;
Jesse Keating 7a32965
 extern int sysctl_oom_dump_tasks;
Jesse Keating 7a32965
+extern int sysctl_would_have_oomkilled;
Jesse Keating 7a32965
 extern int max_threads;
Jesse Keating 7a32965
 extern int core_uses_pid;
Jesse Keating 7a32965
 extern int suid_dumpable;
Jesse Keating 7a32965
@@ -973,6 +974,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,
Jesse Keating 7a32965
+		.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
Jesse Keating 7a32965
index f52481b..a892f07 100644
Jesse Keating 7a32965
--- a/mm/oom_kill.c
Jesse Keating 7a32965
+++ b/mm/oom_kill.c
Jesse Keating 7a32965
@@ -31,6 +31,7 @@
Jesse Keating 7a32965
 int sysctl_panic_on_oom;
Jesse Keating 7a32965
 int sysctl_oom_kill_allocating_task;
Jesse Keating 7a32965
 int sysctl_oom_dump_tasks;
Jesse Keating 7a32965
+int sysctl_would_have_oomkilled;
Jesse Keating 7a32965
 static DEFINE_SPINLOCK(zone_scan_lock);
Jesse Keating 7a32965
 /* #define DEBUG */
Jesse Keating 7a32965
 
Jesse Keating 7a32965
@@ -396,6 +397,12 @@ static void __oom_kill_task(struct task_struct *p, int verbose)
Jesse Keating 7a32965
 		return;
Jesse Keating 7a32965
 	}
Jesse Keating 7a32965
 
Jesse Keating 7a32965
+	if (sysctl_would_have_oomkilled == 1) {
Jesse Keating 7a32965
+		printk(KERN_ERR "Would have killed process %d (%s). But continuing instead.\n",
Jesse Keating 7a32965
+				task_pid_nr(p), p->comm);
Jesse Keating 7a32965
+		return;
Jesse Keating 7a32965
+	}
Jesse Keating 7a32965
+
Jesse Keating 7a32965
 	if (verbose)
Jesse Keating 7a32965
 		printk(KERN_ERR "Killed process %d (%s) "
Jesse Keating 7a32965
 		       "vsz:%lukB, anon-rss:%lukB, file-rss:%lukB\n",
Jesse Keating 7a32965
-- 
Jesse Keating 7a32965
1.6.5.2
Jesse Keating 7a32965