Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-16 Thread Jiri Olsa
On Fri, Dec 11, 2015 at 02:42:22PM +0200, Adrian Hunter wrote: SNIP > > I was wrong about there being any problem using evsel->threads. While the > patch "perf evlist: Factor perf_evlist__(enable|disable) function" changes > the number of threads (from perf_evlist__nr_threads() to thread_map__n

Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-11 Thread Adrian Hunter
On 09/12/15 15:44, Adrian Hunter wrote: > On 08/12/15 15:53, Arnaldo Carvalho de Melo wrote: >> Em Tue, Dec 08, 2015 at 09:29:51AM +0200, Adrian Hunter escreveu: >>> On 07/12/15 23:09, Arnaldo Carvalho de Melo wrote: Em Thu, Dec 03, 2015 at 10:06:39AM +0100, Jiri Olsa escreveu: > while tes

Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-09 Thread Adrian Hunter
On 08/12/15 15:53, Arnaldo Carvalho de Melo wrote: > Em Tue, Dec 08, 2015 at 09:29:51AM +0200, Adrian Hunter escreveu: >> On 07/12/15 23:09, Arnaldo Carvalho de Melo wrote: >>> Em Thu, Dec 03, 2015 at 10:06:39AM +0100, Jiri Olsa escreveu: while testing ftrace:function event I noticed we create

Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-08 Thread Arnaldo Carvalho de Melo
Em Tue, Dec 08, 2015 at 09:29:51AM +0200, Adrian Hunter escreveu: > On 07/12/15 23:09, Arnaldo Carvalho de Melo wrote: > > Em Thu, Dec 03, 2015 at 10:06:39AM +0100, Jiri Olsa escreveu: > >> while testing ftrace:function event I noticed we create > >> stat counters as enabled (except for enable_on_e

Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-07 Thread Adrian Hunter
On 07/12/15 23:09, Arnaldo Carvalho de Melo wrote: > Em Thu, Dec 03, 2015 at 10:06:39AM +0100, Jiri Olsa escreveu: >> hi, >> while testing ftrace:function event I noticed we create >> stat counters as enabled (except for enable_on_exec couters). >> >> This way we count also filter setup and other c

Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-07 Thread Arnaldo Carvalho de Melo
Em Thu, Dec 03, 2015 at 10:06:39AM +0100, Jiri Olsa escreveu: > hi, > while testing ftrace:function event I noticed we create > stat counters as enabled (except for enable_on_exec couters). > > This way we count also filter setup and other config code > which might be crucial for some events. > >

[PATCH 0/7] perf stat: Change event enable code

2015-12-03 Thread Jiri Olsa
hi, while testing ftrace:function event I noticed we create stat counters as enabled (except for enable_on_exec couters). This way we count also filter setup and other config code which might be crucial for some events. Posponing the events enable once everything is ready. The last patch is RFC

Re: [PATCH 0/7] perf stat: Change event enable code

2015-12-03 Thread Jiri Olsa
On Thu, Dec 03, 2015 at 10:06:39AM +0100, Jiri Olsa wrote: > hi, > while testing ftrace:function event I noticed we create > stat counters as enabled (except for enable_on_exec couters). > > This way we count also filter setup and other config code > which might be crucial for some events. > > Po