Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Stephane Eranian
On Fri, Sep 5, 2014 at 2:23 AM, Namhyung Kim wrote: > Hi Stephane, > > On Thu, 4 Sep 2014 16:37:51 +0200, Stephane Eranian wrote: >> On Tue, Aug 26, 2014 at 8:38 AM, Namhyung Kim wrote: >>> >>> There's a problem on finding correct kernel symbols when perf report >>> runs on a different kernel. A

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Namhyung Kim
On Fri, Sep 5, 2014 at 10:42 AM, Namhyung Kim wrote: > On Fri, Sep 5, 2014 at 10:18 AM, Arnaldo Carvalho de Melo > wrote: >> Em Fri, Sep 05, 2014 at 09:09:49AM +0900, Namhyung Kim escreveu: >>> The perf report rebuilds machine states from the event records only. In >>> this case, the kernel map

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Namhyung Kim
On Fri, Sep 5, 2014 at 10:18 AM, Arnaldo Carvalho de Melo wrote: > Em Fri, Sep 05, 2014 at 09:09:49AM +0900, Namhyung Kim escreveu: >> The perf report rebuilds machine states from the event records only. In >> this case, the kernel map was recorded in the name of [kernel.kallsyms] >> so it couldn

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Arnaldo Carvalho de Melo
Em Fri, Sep 05, 2014 at 09:09:49AM +0900, Namhyung Kim escreveu: > >> Before: > >> $ perf record -a usleep 1 > >> $ perf buildid-list > >> 00d5ff078efe1d30b8492854f259215fd877ce30 > >> /lib/modules/3.16.0-rc2+/build/vmlinux > >> 78186287bba77069a056a5ccbeb14b7fd2ca3a4b /usr/lib64/libc-2.1

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Arnaldo Carvalho de Melo
Em Fri, Sep 05, 2014 at 09:23:56AM +0900, Namhyung Kim escreveu: > Hmm.. now I'm curious that why the -H option is needed at all.. the perf > record already wrote build-ids that are actually hits.. Probably because when 'perf buildid-list' was introduced 'perf record' was inserting all the PERF_RE

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Namhyung Kim
Hi Stephane, On Thu, 4 Sep 2014 16:37:51 +0200, Stephane Eranian wrote: > On Tue, Aug 26, 2014 at 8:38 AM, Namhyung Kim wrote: >> >> There's a problem on finding correct kernel symbols when perf report >> runs on a different kernel. Although a part of the problem was solved >> by the prior commi

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Namhyung Kim
Hi Arnaldo, On Thu, 4 Sep 2014 11:18:45 -0300, Arnaldo Carvalho de Melo wrote: > Em Tue, Aug 26, 2014 at 03:38:39PM +0900, Namhyung Kim escreveu: >> There's a problem on finding correct kernel symbols when perf report >> runs on a different kernel. Although a part of the problem was solved >> by

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Stephane Eranian
On Tue, Aug 26, 2014 at 8:38 AM, Namhyung Kim wrote: > > There's a problem on finding correct kernel symbols when perf report > runs on a different kernel. Although a part of the problem was solved > by the prior commit 0a7e6d1b6844 ("perf tools: Check recorded kernel > version when finding vmlin

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-04 Thread Arnaldo Carvalho de Melo
Em Tue, Aug 26, 2014 at 03:38:39PM +0900, Namhyung Kim escreveu: > There's a problem on finding correct kernel symbols when perf report > runs on a different kernel. Although a part of the problem was solved > by the prior commit 0a7e6d1b6844 ("perf tools: Check recorded kernel > version when find

Re: [PATCH] perf tools: Fix build-id matching on vmlinux

2014-09-03 Thread Namhyung Kim
Ping! On Tue, 26 Aug 2014 15:38:39 +0900, Namhyung Kim wrote: > There's a problem on finding correct kernel symbols when perf report > runs on a different kernel. Although a part of the problem was solved > by the prior commit 0a7e6d1b6844 ("perf tools: Check recorded kernel > version when findi