Re: [PATCH 02/10] perf, tools, stat: Collapse identically named events

Jiri Olsa
2017-03-05T17:55:57Z

On Tue, Feb 28, 2017 at 10:49:16PM -0800, Andi Kleen wrote:

SNIP

>
> struct aggr_data {
> @@ -1208,6 +1231,16 @@ static void aggr_cb(struct perf_evsel *counter, void *data, bool first)
> if (first)
> ad->nr++;
> counts = perf_counts(counter->counts, cpu, 0);
> + /*
> + * When any result is bad, make them all to give
> + * consistent output in interval mode.
> + */
> + if (counts->ena == 0 || counts->run == 0 ||
> + counter->counts->scaled == -1) {
> + ad->ena = 0;
> + ad->run = 0;
> + break;
> + }

I think this needs to be in separate patch, showing
the interval output before and after the change..

thanks,
jirka