Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-23 Thread Jiri Olsa
On Thu, Oct 23, 2014 at 12:33:22PM -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Oct 23, 2014 at 05:13:06PM +0200, Jiri Olsa escreveu: > > On Thu, Oct 23, 2014 at 11:26:34AM -0300, Arnaldo Carvalho de Melo wrote: > > > That is why I thought it would be a compromise to put what he did, it > > > w

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-23 Thread Arnaldo Carvalho de Melo
Em Thu, Oct 23, 2014 at 05:13:06PM +0200, Jiri Olsa escreveu: > On Thu, Oct 23, 2014 at 11:26:34AM -0300, Arnaldo Carvalho de Melo wrote: > > That is why I thought it would be a compromise to put what he did, it > > would not make the existing situation that much worse, work needs to be > > done in

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-23 Thread Jiri Olsa
On Thu, Oct 23, 2014 at 11:26:34AM -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Oct 23, 2014 at 04:12:13PM +0200, Jiri Olsa escreveu: > > On Thu, Oct 23, 2014 at 10:37:24AM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Wed, Oct 22, 2014 at 10:46:59AM -0700, Sukadev Bhattiprolu escreveu: > > >

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-23 Thread Arnaldo Carvalho de Melo
Em Thu, Oct 23, 2014 at 04:12:13PM +0200, Jiri Olsa escreveu: > On Thu, Oct 23, 2014 at 10:37:24AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Wed, Oct 22, 2014 at 10:46:59AM -0700, Sukadev Bhattiprolu escreveu: > > > Jiri Olsa [jo...@redhat.com] wrote: > > > | > + goto out; >

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-23 Thread Jiri Olsa
On Thu, Oct 23, 2014 at 10:37:24AM -0300, Arnaldo Carvalho de Melo wrote: > Em Wed, Oct 22, 2014 at 10:46:59AM -0700, Sukadev Bhattiprolu escreveu: > > Jiri Olsa [jo...@redhat.com] wrote: > > | > + goto out; > > | > + } > > | > + dso->dwfl = dwfl; >

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-23 Thread Arnaldo Carvalho de Melo
Em Wed, Oct 22, 2014 at 10:46:59AM -0700, Sukadev Bhattiprolu escreveu: > Jiri Olsa [jo...@redhat.com] wrote: > | > + goto out; > | > + } > | > + dso->dwfl = dwfl; > | > | so by this we get powerpc arch code sharing dw handle via dso object, > | but we have lot of g

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-22 Thread Sukadev Bhattiprolu
Jiri Olsa [jo...@redhat.com] wrote: | > + goto out; | > + } | > + dso->dwfl = dwfl; | | so by this we get powerpc arch code sharing dw handle via dso object, | but we have lot of generic code too ;-) Well, this applies to powerpc... | | could you make this

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-22 Thread Jiri Olsa
On Tue, Oct 21, 2014 at 05:09:58PM -0700, Sukadev Bhattiprolu wrote: > Arnaldo Carvalho de Melo [a...@kernel.org] wrote: > | Em Tue, Oct 21, 2014 at 11:56:10AM -0700, Sukadev Bhattiprolu escreveu: > | > >From 773a3608a0cd2daf02e244cb9ffbf5bb6a0e724e Mon Sep 17 00:00:00 2001 > | > From: Sukadev Bhat

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-21 Thread Sukadev Bhattiprolu
Arnaldo Carvalho de Melo [a...@kernel.org] wrote: | Em Tue, Oct 21, 2014 at 11:56:10AM -0700, Sukadev Bhattiprolu escreveu: | > >From 773a3608a0cd2daf02e244cb9ffbf5bb6a0e724e Mon Sep 17 00:00:00 2001 | > From: Sukadev Bhattiprolu | > Date: Tue, 21 Oct 2014 13:20:22 -0500 | > Subject: [PATCH 1/1] p

Re: [PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-21 Thread Arnaldo Carvalho de Melo
Em Tue, Oct 21, 2014 at 11:56:10AM -0700, Sukadev Bhattiprolu escreveu: > >From 773a3608a0cd2daf02e244cb9ffbf5bb6a0e724e Mon Sep 17 00:00:00 2001 > From: Sukadev Bhattiprolu > Date: Tue, 21 Oct 2014 13:20:22 -0500 > Subject: [PATCH 1/1] perf/powerpc: Cache DWARF debug info Jiri, isn't his related

[PATCH] perf/powerpc: Cache the DWARF debug info

2014-10-21 Thread Sukadev Bhattiprolu
>From 773a3608a0cd2daf02e244cb9ffbf5bb6a0e724e Mon Sep 17 00:00:00 2001 From: Sukadev Bhattiprolu Date: Tue, 21 Oct 2014 13:20:22 -0500 Subject: [PATCH 1/1] perf/powerpc: Cache DWARF debug info Cache the DWARF debug info for DSO so we don't have to rebuild it for each address in the DSO (duh!).