[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090622122615.GA28503@elte.hu>
Date: Mon, 22 Jun 2009 14:26:15 +0200
From: Ingo Molnar <mingo@...e.hu>
To: KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
Cc: "linux-mm@...ck.org" <linux-mm@...ck.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"balbir@...ux.vnet.ibm.com" <balbir@...ux.vnet.ibm.com>,
"nishimura@....nes.nec.co.jp" <nishimura@....nes.nec.co.jp>,
"menage@...gle.com" <menage@...gle.com>
Subject: Re: [RFC][PATCH] cgroup: fix permanent wait in rmdir
* KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com> wrote:
> KAMEZAWA Hiroyuki wrote:
> > Ingo Molnar wrote:
> >>
> >> FYI, there's a new cgroup related list corruption warning/crash that
> >> i've seen a lot of times in latest -tip tests:
> >>
> >> [ 478.555544] ------------[ cut here ]------------
> >> [ 478.556523] WARNING: at lib/list_debug.c:26 __list_add+0x70/0xa0()
> >> [ 478.556523] Hardware name:
> >> [ 478.556523] list_add corruption. next->prev should be prev
> >> (ffff88003e640448), but was ffff88003fa1a6e8. (next=ffff88003fa1a8a0).
> >> [ 478.556523] Modules linked in:
> >> [ 478.556523] Pid: 470, comm: kswapd0 Not tainted 2.6.30-tip #10989
> >> [ 478.556523] Call Trace:
> >> [ 478.556523] [<ffffffff81306150>] ? __list_add+0x70/0xa0
> >> [ 478.556523] [<ffffffff810598dc>] warn_slowpath_common+0x8c/0xc0
> >> [ 478.556523] [<ffffffff81059999>] warn_slowpath_fmt+0x69/0x70
> >> [ 478.556523] [<ffffffff81086e3b>] ? __lock_acquired+0x18b/0x2b0
> >> [ 478.556523] [<ffffffff811022f0>] ? page_check_address+0x110/0x1a0
> >> [ 478.556523] [<ffffffff812ebcf2>] ? cpumask_any_but+0x42/0xb0
> >> [ 478.556523] [<ffffffff8108c528>] ? __lock_release+0x38/0x90
> >> [ 478.556523] [<ffffffff811024e1>] ? page_referenced_one+0x91/0x120
> >> [ 478.556523] [<ffffffff81306150>] __list_add+0x70/0xa0
> >> [ 478.556523] [<ffffffff8111dc63>] mem_cgroup_add_lru_list+0x63/0x70
> >> [ 478.556523] [<ffffffff810eaee4>] move_active_pages_to_lru+0xf4/0x180
> >> [ 478.556523] [<ffffffff810eb758>] ? shrink_active_list+0x1f8/0x2a0
> >> [ 478.556523] [<ffffffff810eb758>] ? shrink_active_list+0x1f8/0x2a0
> >> [ 478.556523] [<ffffffff810eb794>] shrink_active_list+0x234/0x2a0
> >> [ 478.556523] [<ffffffff810ec3c3>] shrink_zone+0x173/0x1f0
> >> [ 478.556523] [<ffffffff810ece0a>] balance_pgdat+0x4da/0x4e0
> >> [ 478.556523] [<ffffffff810eb240>] ? isolate_pages_global+0x0/0x60
> >> [ 478.556523] [<ffffffff810ed3b6>] kswapd+0x106/0x150
> >> [ 478.556523] [<ffffffff810752f0>] ? autoremove_wake_function+0x0/0x40
> >> [ 478.556523] [<ffffffff810ed2b0>] ? kswapd+0x0/0x150
> >> [ 478.556523] [<ffffffff8107516e>] kthread+0x9e/0xb0
> >> [ 478.556523] [<ffffffff8100d2ba>] child_rip+0xa/0x20
> >> [ 478.556523] [<ffffffff8100cc40>] ? restore_args+0x0/0x30
> >> [ 478.556523] [<ffffffff81075085>] ? kthreadd+0xb5/0x100
> >> [ 478.556523] [<ffffffff810750d0>] ? kthread+0x0/0xb0
> >> [ 478.556523] [<ffffffff8100d2b0>] ? child_rip+0x0/0x20
> >> [ 478.556523] ---[ end trace 9f3122957c34141e ]---
> >> [ 484.923530] ------------[ cut here ]------------
> >> [ 484.924525] WARNING: at lib/list_debug.c:26 __list_add+0x70/0xa0()
> >> [ 484.924525] Hardware name:
> >> [ 484.924525] list_add corruption. next->prev should be prev
> >> (ffff88003e640448), but was ffff88003fa192e8. (next=ffff88003fa14d88).
> >> [ 484.941152] Modules linked in:
> >> [ 484.941152] Pid: 470, comm: kswapd0 Tainted: G W 2.6.30-tip
> >> #10989
> >> [ 484.941152] Call Trace:
> >> [ 484.941152] [<ffffffff81306150>] ? __list_add+0x70/0xa0
> >> [ 484.941152] [<ffffffff810598dc>] warn_slowpath_common+0x8c/0xc0
> >> [ 484.941152] [<ffffffff81059999>] warn_slowpath_fmt+0x69/0x70
> >> [ 484.941152] [<ffffffff81086e3b>] ? __lock_acquired+0x18b/0x2b0
> >> [ 484.941152] [<ffffffff811022f0>] ? page_check_address+0x110/0x1a0
> >> [ 484.941152] [<ffffffff812ebcf2>] ? cpumask_any_but+0x42/0xb0
> >> [ 484.941152] [<ffffffff8108c528>] ? __lock_release+0x38/0x90
> >> [ 484.941152] [<ffffffff811024e1>] ? page_referenced_one+0x91/0x120
> >> [ 484.941152] [<ffffffff81306150>] __list_add+0x70/0xa0
> >> [ 484.941152] [<ffffffff8111dc63>] mem_cgroup_add_lru_list+0x63/0x70
> >> [ 484.941152] [<ffffffff810eaee4>] move_active_pages_to_lru+0xf4/0x180
> >> [ 484.941152] [<ffffffff810eb758>] ? shrink_active_list+0x1f8/0x2a0
> >> [ 484.941152] [<ffffffff810eb758>] ? shrink_active_list+0x1f8/0x2a0
> >> [ 484.941152] [<ffffffff810eb794>] shrink_active_list+0x234/0x2a0
> >> [ 484.941152] [<ffffffff810ec3c3>] shrink_zone+0x173/0x1f0
> >> [ 484.941152] [<ffffffff810ece0a>] balance_pgdat+0x4da/0x4e0
> >> [ 484.941152] [<ffffffff810eb240>] ? isolate_pages_global+0x0/0x60
> >> [ 484.941152] [<ffffffff810ed3b6>] kswapd+0x106/0x150
> >> [ 484.941152] [<ffffffff810752f0>] ? autoremove_wake_function+0x0/0x40
> >> [ 484.941152] [<ffffffff810ed2b0>] ? kswapd+0x0/0x150
> >> [ 484.941152] [<ffffffff8107516e>] kthread+0x9e/0xb0
> >> [ 484.941152] [<ffffffff8100d2ba>] child_rip+0xa/0x20
> >> [ 484.941152] [<ffffffff8100cc40>] ? restore_args+0x0/0x30
> >> [ 484.941152] [<ffffffff81075085>] ? kthreadd+0xb5/0x100
> >> [ 484.941152] [<ffffffff810750d0>] ? kthread+0x0/0xb0
> >> [ 484.941152] [<ffffffff8100d2b0>] ? child_rip+0x0/0x20
> >> [ 484.941152] ---[ end trace 9f3122957c34141f ]---
> >> [ 485.365631] ------------[ cut here ]------------
> >> [ 485.368029] WARNING: at lib/list_debug.c:26 __list_add+0x70/0xa0()
> >>
> >> has this been reported before? Is there a fix for it i missed?
> >>
> > Hmm, looks new one.
> >
> > Maybe related to my latest post which modifies __isolate_lru_page()
> > memcg: fix lru rotation in isolate_pages
> >
> > I'll dig tomorrow, sorry.
> >
> Ah, while I test 2.6.30-git18 (includes above patch), I don't see
> above stack dump (with LIST_DEBUG=y) under quick memory pressure
> test...
Note, it still occurs even with latest -git (f234012).
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists