lkml.org 
[lkml]   [2017]   [Nov]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v1 0/9] perf stat: Enable '--per-thread' on all threads
On Mon, Nov 20, 2017 at 08:15:56PM +0800, Jin, Yao wrote:
>
>
> On 11/20/2017 5:26 PM, Jiri Olsa wrote:
> > On Mon, Nov 20, 2017 at 10:43:35PM +0800, Jin Yao wrote:
> > > perf stat --per-thread is useful to break down data per thread.
> > > But it currently requires specifying --pid/--tid to limit it to a process.
> > >
> > > For analysis it would be useful to do it globally for the whole system.
> >
> > I can't compile this:
> >
> > builtin-script.c: In function ‘perf_sample__fprint_metric’:
> > builtin-script.c:1549:2: error: too few arguments to function ‘perf_stat__update_shadow_stats’
> > perf_stat__update_shadow_stats(evsel,
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > In file included from builtin-script.c:24:0:
> > util/stat.h:134:6: note: declared here
> > void perf_stat__update_shadow_stats(struct perf_evsel *counter, u64 count,
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > builtin-script.c:1555:4: error: too few arguments to function ‘perf_stat__print_shadow_stats’
> > perf_stat__print_shadow_stats(ev2,
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > In file included from builtin-script.c:24:0:
> > util/stat.h:143:6: note: declared here
> > void perf_stat__print_shadow_stats(struct perf_evsel *evsel,
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > mv: cannot stat './.builtin-script.o.tmp': No such file or directory
> >
> > thanks,
> > jirka
> >
>
> Hi Jiri,
>
> This patch set is based on the latest perf/core branch.
>
> I pull the branch right now and try the build again, the building is OK.
>
> Could you tell me which branch you are testing on?

ugh.. I was over Andi's changes.. I'll recheck

sry for noise

jirka

\
 
 \ /
  Last update: 2017-11-20 13:30    [W:0.109 / U:2.652 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site