From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail143.messagelabs.com (mail143.messagelabs.com [216.82.254.35]) by kanga.kvack.org (Postfix) with ESMTP id B08326B01B2 for ; Thu, 17 Jun 2010 05:24:50 -0400 (EDT) Received: from d03relay01.boulder.ibm.com (d03relay01.boulder.ibm.com [9.17.195.226]) by e37.co.us.ibm.com (8.14.4/8.13.1) with ESMTP id o5H9N3tj017196 for ; Thu, 17 Jun 2010 03:23:03 -0600 Received: from d03av02.boulder.ibm.com (d03av02.boulder.ibm.com [9.17.195.168]) by d03relay01.boulder.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id o5H9OmbL131992 for ; Thu, 17 Jun 2010 03:24:49 -0600 Received: from d03av02.boulder.ibm.com (loopback [127.0.0.1]) by d03av02.boulder.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id o5H9Om2v004875 for ; Thu, 17 Jun 2010 03:24:48 -0600 Date: Thu, 17 Jun 2010 14:54:42 +0530 From: Balbir Singh Subject: Re: [BUGFIX][PATCH -mm] fix bad call of memcg_oom_recover at cancel move. Message-ID: <20100617092442.GJ4306@balbir.in.ibm.com> Reply-To: balbir@linux.vnet.ibm.com References: <20100617172034.00ea8835.kamezawa.hiroyu@jp.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <20100617172034.00ea8835.kamezawa.hiroyu@jp.fujitsu.com> Sender: owner-linux-mm@kvack.org To: KAMEZAWA Hiroyuki Cc: "linux-mm@kvack.org" , "akpm@linux-foundation.org" , "nishimura@mxp.nes.nec.co.jp" , "linux-kernel@vger.kernel.org" List-ID: * KAMEZAWA Hiroyuki [2010-06-17 17:20:34]: > From: KAMEZAWA Hiroyuki > > When cgroup_cancel_attach() is called via cgroup_attach_task(), > mem_cgroup_clear_mc() can be called even when any migration > was done. In such case, mc.to and mc.from is NULL. > > But, memcg-clean-up-waiting-move-acct-v2.patch > doesn't handle this correctly and pass NULL to memcg_oom_recover. > fix it. > > BUG: unable to handle kernel paging request at 000000000000114c > IP: [] memcg_oom_recover+0x9/0x30 > PGD 61ce4b067 PUD 613ea0067 PMD 0 > Oops: 0000 [#1] SMP > > Call Trace: > [] mem_cgroup_clear_mc+0x119/0x1c0 > [] mem_cgroup_cancel_attach+0xe/0x10 > [] cgroup_attach_task+0x26c/0x2c0 > [] cgroup_tasks_write+0x67/0x1c0 > [] ? might_fault+0xa5/0xb0 > [] ? might_fault+0x5c/0xb0 > [] cgroup_file_write+0x2d2/0x330 > [] ? print_lock_contention_bug+0x22/0xf0 > [] ? security_file_permission+0x1f/0x80 > [] vfs_write+0xc8/0x190 > [] sys_write+0x51/0x90 > [] system_call_fastpath+0x16/0x1b > Code: 20 48 39 43 20 41 bc f0 ff ff ff 75 c7 45 88 ae 48 11 00 00 45 31 e4 eb bb 66 0f 1f 84 00 00 00 00 00 55 48 89 e5 0f 1f 44 00 00 <8b> 87 4c 11 00 00 85 c0 75 05 c9 c3 0f 1f 00 48 89 f9 31 d2 be > RIP [] memcg_oom_recover+0x9/0x30 > > Signed-off-by: KAMEZAWA Hiroyuki > --- > mm/memcontrol.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > Index: mmotm-2.6.35-0611/mm/memcontrol.c > =================================================================== > --- mmotm-2.6.35-0611.orig/mm/memcontrol.c > +++ mmotm-2.6.35-0611/mm/memcontrol.c > @@ -4485,8 +4485,10 @@ static void mem_cgroup_clear_mc(void) > mc.to = NULL; > mc.moving_task = NULL; > spin_unlock(&mc.lock); > - memcg_oom_recover(from); > - memcg_oom_recover(to); > + if (from) > + memcg_oom_recover(from); > + if (to) > + memcg_oom_recover(to); > wake_up_all(&mc.waitq); May I recommend the following change instead Don't crash on a null memcg being passed, check if memcg is NULL and handle the condition gracefully Signed-off-by: Balbir Singh --- mm/memcontrol.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index c6ece0a..d71c488 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c @@ -1370,7 +1370,7 @@ static void memcg_wakeup_oom(struct mem_cgroup *mem) static void memcg_oom_recover(struct mem_cgroup *mem) { - if (mem->oom_kill_disable && atomic_read(&mem->oom_lock)) + if (mem && mem->oom_kill_disable && atomic_read(&mem->oom_lock)) memcg_wakeup_oom(mem); } -- 1.7.0.1 -- Three Cheers, Balbir -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org