Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 3 Sep 2020 15:49:07 -0300
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Jin Yao <yao.jin@linux.intel.com>
Cc:     jolsa@kernel.org, peterz@infradead.org, mingo@redhat.com,
        alexander.shishkin@linux.intel.com, Linux-kernel@vger.kernel.org,
        ak@linux.intel.com, kan.liang@intel.com, yao.jin@intel.com
Subject: Re: [PATCH] perf stat: Turn off summary for interval mode by default
Message-ID: <20200903184907.GD3495158@kernel.org>
References: <20200903010113.32232-1-yao.jin@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200903010113.32232-1-yao.jin@linux.intel.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Em Thu, Sep 03, 2020 at 09:01:13AM +0800, Jin Yao escreveu:
> There's a risk that outputting interval mode summaries by default
> breaks CSV consumers. It already broke pmu-tools/toplev.
> 
> So now we turn off the summary by default but we create a new
> option '--summary' to enable the summary. This is active even
> when not using CSV mode.
> 
> Before:

Thanks, tested and applied.

- Arnaldo