Re: [PATCH 4/4] perf, tools, stat: Implement CSV metrics output

2015-07-30 Thread Jiri Olsa
On Wed, Jul 29, 2015 at 05:21:40PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Now support CSV output for metrics. With the new output callbacks > this is relatively straight forward by creating new callbacks. > > The new line callback needs to know the number of fields to skip them > correc

Re: [PATCH 4/4] perf, tools, stat: Implement CSV metrics output

2015-07-30 Thread Jiri Olsa
On Wed, Jul 29, 2015 at 05:21:40PM -0700, Andi Kleen wrote: SNIP > + } > } > > static void print_aggr(char *prefix) > @@ -818,11 +883,7 @@ static void print_aggr(char *prefix) > continue; > } > uval = val * counter-

[PATCH 4/4] perf, tools, stat: Implement CSV metrics output

2015-07-29 Thread Andi Kleen
From: Andi Kleen Now support CSV output for metrics. With the new output callbacks this is relatively straight forward by creating new callbacks. The new line callback needs to know the number of fields to skip them correctly To avoid reordering the existing CSV fields, I had to move the noise