Re: [PATCH 0/2] perf stat: add per-core count aggregation

2013-02-12 Thread Stephane Eranian
On Tue, Feb 12, 2013 at 6:29 PM, Andi Kleen wrote: >> >> > The idea itself is useful. >> > >> Yes, it is. > > BTW it would be even more useful if it could print some of the > statistics turbostat does (in particular frequency and C0 residency) > Often you only care about cycles not idle, and the f

Re: [PATCH 0/2] perf stat: add per-core count aggregation

2013-02-12 Thread Andi Kleen
> > > The idea itself is useful. > > > Yes, it is. BTW it would be even more useful if it could print some of the statistics turbostat does (in particular frequency and C0 residency) Often you only care about cycles not idle, and the frequency tells you how fast the cycles happen. I think Cx cou

Re: [PATCH 0/2] perf stat: add per-core count aggregation

2013-02-12 Thread Stephane Eranian
On Tue, Feb 12, 2013 at 6:23 PM, Andi Kleen wrote: > On Tue, Feb 12, 2013 at 03:09:26PM +0100, Stephane Eranian wrote: >> This patch series contains improvement to the aggregation support >> in perf stat. >> >> First, the aggregation code is refactored and a aggr_mode enum >> is defined. There is

Re: [PATCH 0/2] perf stat: add per-core count aggregation

2013-02-12 Thread Andi Kleen
On Tue, Feb 12, 2013 at 03:09:26PM +0100, Stephane Eranian wrote: > This patch series contains improvement to the aggregation support > in perf stat. > > First, the aggregation code is refactored and a aggr_mode enum > is defined. There is also an important bug fix for the existing > per-socket ag

[PATCH 0/2] perf stat: add per-core count aggregation

2013-02-12 Thread Stephane Eranian
This patch series contains improvement to the aggregation support in perf stat. First, the aggregation code is refactored and a aggr_mode enum is defined. There is also an important bug fix for the existing per-socket aggregation. Second, the patch adds a new --aggr-core option to perf stat. It a