Re: [PATCH] perf tools mem: Fix -t store option for record command

2016-08-11 Thread Jiri Olsa
On Thu, Aug 11, 2016 at 11:46:46AM -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Aug 11, 2016 at 02:00:32PM +0200, Jiri Olsa escreveu: > > On Thu, Aug 11, 2016 at 08:51:38AM -0300, Arnaldo Carvalho de Melo wrote: > > > Em Thu, Aug 11, 2016 at 10:50:57AM +0200, Jiri Olsa escreveu: > > > > Michael

Re: [PATCH] perf tools mem: Fix -t store option for record command

2016-08-11 Thread Arnaldo Carvalho de Melo
Em Thu, Aug 11, 2016 at 02:00:32PM +0200, Jiri Olsa escreveu: > On Thu, Aug 11, 2016 at 08:51:38AM -0300, Arnaldo Carvalho de Melo wrote: > > Em Thu, Aug 11, 2016 at 10:50:57AM +0200, Jiri Olsa escreveu: > > > Michael reported 'perf mem -t store record' being broken. > > > The reason is latest rewo

Re: [PATCH] perf tools mem: Fix -t store option for record command

2016-08-11 Thread Jiri Olsa
On Thu, Aug 11, 2016 at 08:51:38AM -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Aug 11, 2016 at 10:50:57AM +0200, Jiri Olsa escreveu: > > Michael reported 'perf mem -t store record' being broken. > > The reason is latest rework of this area: > > commit acbe613e0c03 ("perf tools: Add monitored

Re: [PATCH] perf tools mem: Fix -t store option for record command

2016-08-11 Thread Arnaldo Carvalho de Melo
Em Thu, Aug 11, 2016 at 10:50:57AM +0200, Jiri Olsa escreveu: > Michael reported 'perf mem -t store record' being broken. > The reason is latest rework of this area: > commit acbe613e0c03 ("perf tools: Add monitored events array") > > We don't mark perf_mem_events store record when -t store > op

[PATCH] perf tools mem: Fix -t store option for record command

2016-08-11 Thread Jiri Olsa
Michael reported 'perf mem -t store record' being broken. The reason is latest rework of this area: commit acbe613e0c03 ("perf tools: Add monitored events array") We don't mark perf_mem_events store record when -t store option is specified. Fixes: commit acbe613e0c03 ("perf tools: Add monitored