Re: [PATCH v3 2/2] perf test: Add CVS summary test

2021-03-24 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 24, 2021 at 10:12:43AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Mar 24, 2021 at 10:05:18AM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Fri, Mar 19, 2021 at 03:01:56PM +0800, Jin Yao escreveu: > > > The patch "perf stat: Align CSV output for summary mode" aligned > > > CV

Re: [PATCH v3 2/2] perf test: Add CVS summary test

2021-03-24 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 24, 2021 at 10:05:18AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Fri, Mar 19, 2021 at 03:01:56PM +0800, Jin Yao escreveu: > > The patch "perf stat: Align CSV output for summary mode" aligned > > CVS output and added "summary" to the first column of summary > > lines. > > > > Now w

Re: [PATCH v3 2/2] perf test: Add CVS summary test

2021-03-24 Thread Arnaldo Carvalho de Melo
Em Fri, Mar 19, 2021 at 03:01:56PM +0800, Jin Yao escreveu: > The patch "perf stat: Align CSV output for summary mode" aligned > CVS output and added "summary" to the first column of summary > lines. > > Now we check if the "summary" string is added to the CVS output. > > If we set '--no-cvs-summ

[PATCH v3 2/2] perf test: Add CVS summary test

2021-03-19 Thread Jin Yao
The patch "perf stat: Align CSV output for summary mode" aligned CVS output and added "summary" to the first column of summary lines. Now we check if the "summary" string is added to the CVS output. If we set '--no-cvs-summary' option, the "summary" string would not be added, also check with this