Date: Mon, 30 May 2011 16:00:41 -0300
From: Arnaldo Carvalho de Melo <acme@infradead.org>
To: Frederic Weisbecker <fweisbec@gmail.com>
Cc: David Ahern <dsahern@gmail.com>,
        Akihiro Nagai <akihiro.nagai.hw@hitachi.com>,
        Ingo Molnar <mingo@elte.hu>, Peter Zijlstra <peterz@infradead.org>,
        linux-kernel@vger.kernel.org,
        Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>,
        pp-manager@sdl.hitachi.co.jp, David Miller <davem@davemloft.net>
Subject: Re: [PATCH -tip v4 0/7] perf: Introduce branch sub commands
Message-ID: <20110530190041.GE14097@ghostprotocols.net>
References: <20110526050246.30011.86048.stgit@localhost6.localdomain6>
 <20110526132833.GC21815@nowhere>
 <4DDE7EAD.7000603@gmail.com>
 <4DE39C31.1070600@hitachi.com>
 <4DE3B724.3020203@gmail.com>
 <20110530161147.GA1741@somewhere>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20110530161147.GA1741@somewhere>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1759
Lines: 40

Em Mon, May 30, 2011 at 06:11:49PM +0200, Frederic Weisbecker escreveu:
> On Mon, May 30, 2011 at 09:26:28AM -0600, David Ahern wrote:
> > On 05/30/2011 07:31 AM, Akihiro Nagai wrote:
> > >> The sample address can be converted to symbols and the output can be
> > >> added to perf-script rather easily. Attached is an example. I was going
> > >> to submit it back in April and got distracted. I'll rebase, move the
> > >> addr->sym conversion to a function and submit later today.
> > > OK.
> > > I agreed with implementing it on perf script.
> > > I'd like to try it.
> > 
> > Updated patch. It applies on top of:
> >   http://lkml.org/lkml/2011/5/27/385
> > 
> > Hopefully for BTS you only have to change the sample_addr_correlates_sym
> > function.
> > 
> > I still need to look into out why some of the addresses for page-faults
> > are not resolving to symbols.
> 
> Perhaps it only resolves instruction faults because you only resolve with
> MAP__FUNCTION. You can try MAP__VARIABLE as well when that fails.
> 
> But that is going to only help with faults on global vars, ie: only
> a few part of them. Further information would require some dwarf processing.
> 
> Your patch looks good though. Mind resending it with appropriate title/changelog
> and signed-off-by tag?

Just for reference: I thought about removing this split but David Miller
told me that there are architectures where we can't do that, i.e. there
is no guarantee of an unified addr space for variables and code, such as
SPARC.

- Arnaldo
--
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/