Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754262AbZIOOIr (ORCPT ); Tue, 15 Sep 2009 10:08:47 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754234AbZIOOIo (ORCPT ); Tue, 15 Sep 2009 10:08:44 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.124]:36589 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754229AbZIOOIn (ORCPT ); Tue, 15 Sep 2009 10:08:43 -0400 Subject: Re: [PATCH 2/4] virtual block device driver (ramzswap) From: Steven Rostedt Reply-To: rostedt@goodmis.org To: Pekka Enberg Cc: Nitin Gupta , Andrew Morton , Hugh Dickins , Ed Tomlinson , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-mm-cc@laptop.org, Ingo Molnar , =?ISO-8859-1?Q?Fr=E9d=E9ric?= Weisbecker In-Reply-To: <1253022207.4754.1.camel@penberg-laptop> References: <200909100215.36350.ngupta@vflare.org> <200909100249.26284.ngupta@vflare.org> <84144f020909141310y164b2d1ak44dd6945d35e6ec@mail.gmail.com> <84144f020909150030h1f9d8062sc39057b55a7ba6c0@mail.gmail.com> <1253020471.20020.76.camel@gandalf.stny.rr.com> <1253022207.4754.1.camel@penberg-laptop> Content-Type: text/plain Organization: Kihon Technologies Inc. Date: Tue, 15 Sep 2009 10:08:45 -0400 Message-Id: <1253023725.20020.103.camel@gandalf.stny.rr.com> Mime-Version: 1.0 X-Mailer: Evolution 2.26.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1509 Lines: 40 On Tue, 2009-09-15 at 16:43 +0300, Pekka Enberg wrote: > Hi Steve, > > On Tue, 2009-09-15 at 09:14 -0400, Steven Rostedt wrote: > > > >>> + > > > >>> + trace_mark(ramzswap_lock_wait, "ramzswap_lock_wait"); > > > >>> + mutex_lock(&rzs->lock); > > > >>> + trace_mark(ramzswap_lock_acquired, "ramzswap_lock_acquired"); > > > >> > > > >> Hmm? What's this? I don't think you should be doing ad hoc > > > >> trace_mark() in driver code. > > > > > > > > This is not ad hoc. It is to see contention over this lock which I believe is a > > > > major bottleneck even on dual-cores. I need to keep this to measure improvements > > > > as I gradually make this locking more fine grained (using per-cpu buffer etc). > > > > > > It is ad hoc. Talk to the ftrace folks how to do it properly. I'd keep > > > those bits out-of-tree until the issue is resolved, really. > > > > Yes, trace_mark is deprecated. You want to use TRACE_EVENT. See how gfs2 > > does it in: > > > > fs/gfs2/gfs2_trace.h > > > > and it is well documented in > > samples/trace_events/trace-events-samples.[ch] > > Does it really make sense to add special-case tracing in driver code to > profile lock contention for a _single mutex_? No that's what LOCKSTAT is for ;-) -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/