Re: [PATCH v2] perf stat: Align CSV output for summary mode

2021-03-18 Thread Jin, Yao
Hi Arnaldo, On 3/18/2021 9:15 PM, Arnaldo Carvalho de Melo wrote: Em Wed, Mar 17, 2021 at 02:51:42PM -0700, Andi Kleen escreveu: If you care about not breaking existing scripts, then the output they get with what they use as command line options must continue to produce the same output. It's

Re: [PATCH v2] perf stat: Align CSV output for summary mode

2021-03-18 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 17, 2021 at 02:51:42PM -0700, Andi Kleen escreveu: > > If you care about not breaking existing scripts, then the output they > > get with what they use as command line options must continue to produce > > the same output. > > It's not clear there are any useful ones (except for tools

Re: [PATCH v2] perf stat: Align CSV output for summary mode

2021-03-17 Thread Andi Kleen
> If you care about not breaking existing scripts, then the output they > get with what they use as command line options must continue to produce > the same output. It's not clear there are any useful ones (except for tools that handle both). It's really hard to parse the previous mess. It's

Re: [PATCH v2] perf stat: Align CSV output for summary mode

2021-03-17 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 17, 2021 at 03:02:05PM +0800, Jin Yao escreveu: > perf-stat has supported the summary mode. But the summary > lines break the CSV output so it's hard for scripts to parse > the result. > > Before: > > # perf stat -x, -I1000 --interval-count 1 --summary >

[PATCH v2] perf stat: Align CSV output for summary mode

2021-03-17 Thread Jin Yao
perf-stat has supported the summary mode. But the summary lines break the CSV output so it's hard for scripts to parse the result. Before: # perf stat -x, -I1000 --interval-count 1 --summary 1.001323097,8013.48,msec,cpu-clock,8013483384,100.00,8.013,CPUs utilized