Re: [PATCH v2] perf machine: Fix load kernel symbol with '-k' option

2018-03-11 Thread Jiri Olsa
On Sat, Mar 10, 2018 at 05:50:56PM +0800, Leo Yan wrote: SNIP > > > diff --git a/tools/perf/util/machine.c b/tools/perf/util/machine.c > > > index 12b7427..3125871 100644 > > > --- a/tools/perf/util/machine.c > > > +++ b/tools/perf/util/machine.c > > > @@ -1299,9 +1299,18 @@ static int > > > mac

Re: [PATCH v2] perf machine: Fix load kernel symbol with '-k' option

2018-03-10 Thread Leo Yan
Hi Jiri, On Fri, Mar 09, 2018 at 09:56:00PM +0100, Jiri Olsa wrote: > On Fri, Mar 09, 2018 at 02:05:23PM +0800, Leo Yan wrote: > > On Hikey arm64 octa A53 platform, when use command './perf report -v > > -k vmlinux --stdio' it outputs below error info, and it skips to load > > kernel symbol and do

Re: [PATCH v2] perf machine: Fix load kernel symbol with '-k' option

2018-03-09 Thread Jiri Olsa
On Fri, Mar 09, 2018 at 02:05:23PM +0800, Leo Yan wrote: > On Hikey arm64 octa A53 platform, when use command './perf report -v > -k vmlinux --stdio' it outputs below error info, and it skips to load > kernel symbol and doesn't print symbol for event: > Failed to open [kernel.kallsyms]_text, contin

Re: [PATCH v2] perf machine: Fix load kernel symbol with '-k' option

2018-03-09 Thread Arnaldo Carvalho de Melo
Em Fri, Mar 09, 2018 at 02:05:23PM +0800, Leo Yan escreveu: > On Hikey arm64 octa A53 platform, when use command './perf report -v > -k vmlinux --stdio' it outputs below error info, and it skips to load > kernel symbol and doesn't print symbol for event: > Failed to open [kernel.kallsyms]_text, con

Re: [PATCH v2] perf machine: Fix load kernel symbol with '-k' option

2018-03-09 Thread Mathieu Poirier
On 8 March 2018 at 23:05, Leo Yan wrote: > On Hikey arm64 octa A53 platform, when use command './perf report -v > -k vmlinux --stdio' it outputs below error info, and it skips to load > kernel symbol and doesn't print symbol for event: > Failed to open [kernel.kallsyms]_text, continuing without sy

[PATCH v2] perf machine: Fix load kernel symbol with '-k' option

2018-03-08 Thread Leo Yan
On Hikey arm64 octa A53 platform, when use command './perf report -v -k vmlinux --stdio' it outputs below error info, and it skips to load kernel symbol and doesn't print symbol for event: Failed to open [kernel.kallsyms]_text, continuing without symbols. The regression is introduced by commit ("8