From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753090Ab0CASRl (ORCPT ); Mon, 1 Mar 2010 13:17:41 -0500 Received: from bombadil.infradead.org ([18.85.46.34]:37120 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751232Ab0CASRj (ORCPT ); Mon, 1 Mar 2010 13:17:39 -0500 Subject: Re: [PATCH tip/core/rcu 0/2] more lockdep-RCU and RCU_FAST_NO_HZ fixes From: Peter Zijlstra To: paulmck@linux.vnet.ibm.com Cc: Ingo Molnar , linux-kernel@vger.kernel.org, laijs@cn.fujitsu.com, dipankar@in.ibm.com, akpm@linux-foundation.org, mathieu.desnoyers@polymtl.ca, josh@joshtriplett.org, dvhltc@us.ibm.com, niv@us.ibm.com, tglx@linutronix.de, rostedt@goodmis.org, Valdis.Kletnieks@vt.edu, dhowells@redhat.com In-Reply-To: <20100301160813.GA6758@linux.vnet.ibm.com> References: <20100227225240.GA16515@linux.vnet.ibm.com> <20100228085505.GA2067@elte.hu> <20100228163218.GD6846@linux.vnet.ibm.com> <20100301121045.GA30901@elte.hu> <20100301160813.GA6758@linux.vnet.ibm.com> Content-Type: text/plain; charset="UTF-8" Date: Mon, 01 Mar 2010 18:58:10 +0100 Message-ID: <1267466290.1579.29.camel@laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.28.2 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2010-03-01 at 08:08 -0800, Paul E. McKenney wrote: > On Mon, Mar 01, 2010 at 01:10:45PM +0100, Ingo Molnar wrote: > > > > FYI, even with your patch applied i'm getting this in -tip testing: > > > > [ 0.000000] Memory: 914996k/1047744k available (15146k kernel code, 452k absent, 131584k reserved, 12516k data, 2552k init) > > [ 0.000000] SLUB: Genslabs=13, HWalign=64, Order=0-3, MinObjects=0, CPUs=1, Nodes=1 > > [ 0.000000] > > [ 0.000000] =================================================== > > [ 0.000000] [ INFO: suspicious rcu_dereference_check() usage. ] > > [ 0.000000] --------------------------------------------------- > > [ 0.000000] include/linux/cgroup.h:492 invoked rcu_dereference_check() without protection! > > [ 0.000000] > > [ 0.000000] other info that might help us debug this: > > [ 0.000000] > > [ 0.000000] 1 lock held by swapper/0: > > [ 0.000000] #0: (&rq->lock){......}, at: [] init_idle+0x31/0x1ee > > [ 0.000000] > > [ 0.000000] stack backtrace: > > [ 0.000000] Pid: 0, comm: swapper Not tainted 2.6.33-tip+ #10563 > > [ 0.000000] Call Trace: > > [ 0.000000] [] lockdep_rcu_dereference+0xa1/0xb0 > > [ 0.000000] [] init_idle+0x141/0x1ee > > [ 0.000000] [] sched_init+0x43a/0x4b6 > > [ 0.000000] [] start_kernel+0x1b3/0x49e > > [ 0.000000] [] x86_64_start_reservations+0x120/0x124 > > [ 0.000000] [] x86_64_start_kernel+0x14e/0x15d > > [ 0.000000] Hierarchical RCU implementation. > > [ 0.000000] RCU-based detection of stalled CPUs is enabled. > > [ 0.000000] NR_IRQS:4352 > > > > Config attached. > > > > The sha1 is: > > > > b5fabe1: Merge branch 'perf/urgent' > > > > i.e. your latest fix is included: > > > > 90a6501: sched, rcu: Fix rcu_dereference() for RCU-lockdep > > Sigh! I clearly need a more organized approach for handling this very > early boot stuff. Fix is in progress, please accept my apologies for > the hassle! add: system_state != SYSTEM_RUNNING, to all the default rcu_read_lock*_held thingies?