Messages in this thread | | | Date | Thu, 18 Mar 2021 10:15:56 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH v2] perf stat: Align CSV output for summary mode |
| |
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 that handle > both). It's really hard to parse the previous mess. It's simply not > valid CSV. > > That's why I'm arguing that keeping compatibility is not useful here. > > We would be stuck with the broken mess as default forever.
Fair enough, lets fix the default then. Jin, can you please consider adding a 'perf test' shell entry to parse the CSV mode with/without that summary? This way we'll notice when the new normal gets broken.
- Arnaldo
| |