Re: [PATCH 03/18] perf tools: Keep group information

2012-12-02 Thread Namhyung Kim
Hi Arnaldo, On Thu, 29 Nov 2012 16:09:22 -0300, Arnaldo Carvalho de Melo wrote: > Em Fri, Nov 30, 2012 at 12:09:11AM +0900, Namhyung Kim escreveu: >> 2012-11-29 (목), 16:02 +0100, Jiri Olsa: >> > > You mean adding 'evsel->nr_members > 1' to this function? For some >> > > reason, I'd like to treat

Re: [PATCH 03/18] perf tools: Keep group information

2012-11-30 Thread Arnaldo Carvalho de Melo
Em Fri, Nov 30, 2012 at 12:09:11AM +0900, Namhyung Kim escreveu: > 2012-11-29 (목), 16:02 +0100, Jiri Olsa: > > > You mean adding 'evsel->nr_members > 1' to this function? For some > > > reason, I'd like to treat non-group events as group leaders so I dropped > > > that check from the function. > >

Re: [PATCH 03/18] perf tools: Keep group information

2012-11-29 Thread Namhyung Kim
2012-11-29 (목), 16:02 +0100, Jiri Olsa: > On Thu, Nov 29, 2012 at 11:58:54PM +0900, Namhyung Kim wrote: > > 2012-11-29 (목), 15:33 +0100, Jiri Olsa: > > > On Thu, Nov 29, 2012 at 03:38:31PM +0900, Namhyung Kim wrote: > > > > From: Namhyung Kim > > > > > > SNIP > > > > > > > --- a/tools/perf/util/

Re: [PATCH 03/18] perf tools: Keep group information

2012-11-29 Thread Jiri Olsa
On Thu, Nov 29, 2012 at 11:58:54PM +0900, Namhyung Kim wrote: > 2012-11-29 (목), 15:33 +0100, Jiri Olsa: > > On Thu, Nov 29, 2012 at 03:38:31PM +0900, Namhyung Kim wrote: > > > From: Namhyung Kim > > > > SNIP > > > > > --- a/tools/perf/util/evlist.h > > > +++ b/tools/perf/util/evlist.h > > > @@ -

Re: [PATCH 03/18] perf tools: Keep group information

2012-11-29 Thread Namhyung Kim
2012-11-29 (목), 15:33 +0100, Jiri Olsa: > On Thu, Nov 29, 2012 at 03:38:31PM +0900, Namhyung Kim wrote: > > From: Namhyung Kim > > SNIP > > > --- a/tools/perf/util/evlist.h > > +++ b/tools/perf/util/evlist.h > > @@ -21,6 +21,7 @@ struct perf_evlist { > > struct list_head entries; > > str

Re: [PATCH 03/18] perf tools: Keep group information

2012-11-29 Thread Jiri Olsa
On Thu, Nov 29, 2012 at 03:38:31PM +0900, Namhyung Kim wrote: > From: Namhyung Kim SNIP > --- a/tools/perf/util/evlist.h > +++ b/tools/perf/util/evlist.h > @@ -21,6 +21,7 @@ struct perf_evlist { > struct list_head entries; > struct hlist_head heads[PERF_EVLIST__HLIST_SIZE]; > i

[PATCH 03/18] perf tools: Keep group information

2012-11-28 Thread Namhyung Kim
From: Namhyung Kim Add a few of group-related field in struct perf_{evlist,evsel} so that the group information in a evlist can be known easily. It only counts groups which have more than 1 members since leader-only groups are treated as non-group events. Cc: Jiri Olsa Cc: Stephane Eranian Si