From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30764 invoked by alias); 2 Dec 2009 21:53:13 -0000 Received: (qmail 30748 invoked by uid 22791); 2 Dec 2009 21:53:12 -0000 X-SWARE-Spam-Status: No, hits=-2.5 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 02 Dec 2009 21:53:07 +0000 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id nB2Lr6LP028949 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 2 Dec 2009 16:53:06 -0500 Received: from [10.16.2.46] (dhcp-100-2-46.bos.redhat.com [10.16.2.46]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id nB2Lr1dd015015; Wed, 2 Dec 2009 16:53:02 -0500 Message-ID: <4B16E1C0.7000401@redhat.com> Date: Wed, 02 Dec 2009 21:53:00 -0000 From: Masami Hiramatsu User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.4pre) Gecko/20091014 Fedora/3.0-2.8.b4.fc11 Thunderbird/3.0b4 MIME-Version: 1.0 To: Ingo Molnar CC: Frederic Weisbecker , lkml , Steven Rostedt , Jim Keniston , Ananth N Mavinakayanahalli , Christoph Hellwig , "Frank Ch. Eigler" , Jason Baron , "K.Prasad" , Peter Zijlstra , Srikar Dronamraju , Arnaldo Carvalho de Melo , systemtap , DLE Subject: Re: [PATCH -tip 0/9] perf-probe updates References: <20091201001912.10235.79411.stgit@harusame> <20091201072902.GA4587@elte.hu> In-Reply-To: <20091201072902.GA4587@elte.hu> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes Mailing-List: contact systemtap-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: systemtap-owner@sourceware.org X-SW-Source: 2009-q4/txt/msg00782.txt.bz2 Ingo Molnar wrote: > > * Masami Hiramatsu wrote: > >> Hi, >> >> Here are bugfixes and updates for perf-probe and kprobe-tracer. I've >> fixed some minor bugs and added --list option and simple probe naming. > > Applied, thanks Masami! > >> TODO: >> - Support build-id checking. >> - Support --del option to remove probes. >> - Support --line option to show which lines user can probe. >> - Support lazy string matching. > > ok, cool! > > One other small detail i noticed wrt. probe naming. Right now if we > insert a single probe into a function it gets named schedule_0: > > # perf probe schedule > Could not open vmlinux/module file. Try to use symbols. > Added new event: p:probe/schedule_0 schedule+0 > > the next one gets named schedule_1, schedule_2, etc. > > It would be nice to special-case the first one and name it 'schedule'. > Most of the time people insert a single probe into a function, so the _0 > postfix is extra and in most cases unnecessary typing for them. Sure, that's reasonable. > > Another small detail is that i dont think we should emit this line: > > Could not open vmlinux/module file. Try to use symbols. > > when we can create a probe successfully - it's just unnecessary noise, > the user does not care how we pulled it off, as long as we were able to > get a reliable symbol address and the insertion worked fine. OK, I'll make it a debug message. > A third detail is this line: > > Added new event: p:probe/schedule_0 schedule+0 > > If that is pasted to perf stat directly it wont work because the syntax > is probe:schedule_0. So i'd suggest to print something like: > > Added new event: probe/schedule_0 (on schedule+0) Sure, perf always use ':' for event group separator, so Added new event: probe:schedule (on schedule+0 [with ...(args)]) > > Perhaps even print another line: > > You can now use it on all perf tools, such as: > > perf probe -e probe/schedule_0 -a sleep 1 > perf record -e probe/schedule_0 -a sleep 1 OK. Thank you for good advice! > > ... to show people how to make use of it. > > Thanks, > > Ingo -- Masami Hiramatsu Software Engineer Hitachi Computer Products (America), Inc. Software Solutions Division e-mail: mhiramat@redhat.com