Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753702Ab1DAPNW (ORCPT ); Fri, 1 Apr 2011 11:13:22 -0400 Received: from mail-wy0-f174.google.com ([74.125.82.174]:63268 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751942Ab1DAPNU (ORCPT ); Fri, 1 Apr 2011 11:13:20 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=sF6XodOCDz3IOG3CEMfVkFcHILkUeJhJxXiB0TAb1z5zSH29sd+as8pi17wdWyQmsy XmclCYQGYoXwjwSJ63dzehIpBcvJwzX+A/J5oDnL9ib+EkyjL51IBOeccErLlILe/a8g NrCW8pk8dCD+pIHnJINLHBT+0Bzx9QxF0ZqiI= Date: Fri, 1 Apr 2011 17:13:16 +0200 From: Frederic Weisbecker To: David Ahern Cc: Akihiro Nagai , Arnaldo Carvalho de Melo , Ingo Molnar , Peter Zijlstra , linux-kernel@vger.kernel.org, Masami Hiramatsu , 2nddept-manager@sdl.hitachi.co.jp, Paul Mackerras Subject: Re: [PATCH -tip v3 3/6] perf branch trace: print pid and command Message-ID: <20110401151313.GC2335@nowhere> References: <20110324113137.20235.42265.stgit@localhost6.localdomain6> <20110324113209.20235.61900.stgit@localhost6.localdomain6> <4D8B79E6.2050603@gmail.com> <4D8C6B1B.70409@hitachi.com> <4D8CAE74.9080805@gmail.com> <4D906450.1040809@hitachi.com> <4D909BBB.5020500@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4D909BBB.5020500@gmail.com> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2036 Lines: 43 On Mon, Mar 28, 2011 at 08:31:23AM -0600, David Ahern wrote: > On 03/28/11 04:34, Akihiro Nagai wrote: > >> from is sample->ip? to is sample->addr? In the above example > >> 0x39d3015260 is the value from sample->addr, 1526f is sample->ip which > >> resolves to _dl_next_ld_env_entry from /lib64/ld-2.13.so. > > Yes. > > In this example, resolved address is only sample->ip (branch from). > > We need the resolved address of sample->addr (branch to) too, because > > both of them are addresses of execution code. > > Ok, now I understand. In that case add conversion of sample->addr to > symbols to perf-script. I agree that we should rather use perf script for branch dumps. Sorry Akihiro, I think we suggested you to create this dedicated perf branch by the past. But then perf script became the vanilla dump tool in the middle and it seems more suitable today. We can still create a perf branch later in order to produce some more advanced post-processing tools. But for sample dumps perf script (which starts to show itself as a misnomer BTW) seems to be the right place. So, in this context, if we have PERF_SAMPLE_ADDR, we are going to always follow the ip with a "=>" and then resolve the address, etc... Yeah it makes sense for this default mode. But what about when we'll want a function graph kind of output? This will require a totally different layout. Also PERF_SAMPLE_ADDR may be used for different context in the future. Perhaps we want a kind of per evsel callback that makes its own interpretation of the pid/tid/dso/sym/etc... options asked by the user? But well, we can start simple and make and just do the => trick if we have PERF_SAMPLE_ADDR and resolve addr if the user asked the sym. Then when we have the graph output, have these per evsel display callbacks. -- 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/