From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [GIT PULL] adaptive spinning mutexes Date: Wed, 14 Jan 2009 11:36:38 -0800 Message-ID: <20090114113638.c818fcf8.akpm@linux-foundation.org> References: <1231774622.4371.96.camel@laptop> <1231859742.442.128.camel@twins> <1231863710.7141.3.camel@twins> <1231864854.7141.8.camel@twins> <1231867314.7141.16.camel@twins> <1231952436.14825.28.camel@laptop> <20090114183319.GA18630@elte.hu> <20090114105300.66bd014d.akpm@linux-foundation.org> <20090114190008.GA13203@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: torvalds@linux-foundation.org, a.p.zijlstra@chello.nl, paulmck@linux.vnet.ibm.com, ghaskins@novell.com, matthew@wil.cx, andi@firstfloor.org, chris.mason@oracle.com, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-btrfs@vger.kernel.org, tglx@linutronix.de, npiggin@suse.de, pmorreale@novell.com, SDietrich@novell.com, dmitry.adamushko@gmail.com, hannes@cmpxchg.org To: Ingo Molnar Return-path: In-Reply-To: <20090114190008.GA13203@elte.hu> Sender: linux-btrfs-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Wed, 14 Jan 2009 20:00:08 +0100 Ingo Molnar wrote: > > * Andrew Morton wrote: > > > On Wed, 14 Jan 2009 19:33:19 +0100 Ingo Molnar wrote: > > > > > Please pull the adaptive-mutexes-for-linus git tree > > > > > > > > - It seems a major shortcoming that the feature is disabled if > > CONFIG_DEBUG_MUTEXES=y. It means that lots of people won't test it. ^^^? > > - When people hit performance/latency oddities, it would be nice if > > they had a /proc knob with which they could disable this feature at > > runtime. > > > > This would also be useful for comparative performance testing. > > Yeah. From my other mail: > > > > We still have the /sys/debug/sched_features tunable under > > > CONFIG_SCHED_DEBUG=y, so should this cause any performance regressions > > > somewhere, it can be pinned down and blamed back on this change > > > easily, without bisection and without rebooting the box. > > This kind of easy knob was included early on - this is how all those spin > versus no-spin numbers were done. Do people enable CONFIG_SCHED_DEBUG? CONFIG_DEBUG_MUTEXES=n, CONFIG_SCHED_DEBUG=y is getting to be a pretty small subset?