On Tue, Aug 12, 2014 at 9:09 AM, Peter Zijlstra <[email protected]> wrote: > On Tue, Aug 12, 2014 at 08:00:31AM +0200, Stephane Eranian wrote: >> arch/x86/kernel/cpu/perf_event_intel_uncore.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/x86/kernel/cpu/perf_event_intel_uncore.c >> b/arch/x86/kernel/cpu/perf_event_intel_uncore.c >> index cfc6f9d..800e087 100644 >> --- a/arch/x86/kernel/cpu/perf_event_intel_uncore.c >> +++ b/arch/x86/kernel/cpu/perf_event_intel_uncore.c >> @@ -1391,6 +1391,7 @@ static struct intel_uncore_type ivt_uncore_imc = { >> .fixed_ctr_bits = 48, >> .fixed_ctr = SNBEP_MC_CHy_PCI_PMON_FIXED_CTR, >> .fixed_ctl = SNBEP_MC_CHy_PCI_PMON_FIXED_CTL, >> + .event_descs = snbep_uncore_imc_events, >> IVT_UNCORE_PCI_COMMON_INIT(), >> }; > > I changed that to the below; seeing how I have the uncore splitup > patches. > Looks good to me.
> Also, I'm very tempted to do a s/ivt/ivbep/ on that whole thing. We > really should not mix all these stupid names. > That's okay with me. Thanks. > --- > Subject: perf/x86/uncore: export basic memory events for IVT IMC PMU > From: Stephane Eranian <[email protected]> > Date: Tue, 12 Aug 2014 08:00:31 +0200 > > This patch exposes two basic events for Ivytown IMC uncore PMU: > > - cas_count_read: number of full-cache line reads to memory controller > - cas_count_write: number of full-cache line writes to memory controller > > Those events use the same encoding as for SNB-EP, so reuse the same > event table. See specification in: > > http://www.intel.com/content/dam/www/public/us/en/documents/manuals/xeon-e5-2600-v2-uncore-manual.pdf > > By aggregating all the read and write events from all the memory controllers > of each processor socket, one can determine the total memory bandwidth > utilization. > > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Stephane Eranian <[email protected]> > Signed-off-by: Peter Zijlstra <[email protected]> > Link: http://lkml.kernel.org/r/20140812060031.GA25239@quad > --- > arch/x86/kernel/cpu/perf_event_intel_uncore_snbep.c | 1 + > 1 file changed, 1 insertion(+) > > --- a/arch/x86/kernel/cpu/perf_event_intel_uncore_snbep.c > +++ b/arch/x86/kernel/cpu/perf_event_intel_uncore_snbep.c > @@ -1422,6 +1422,7 @@ static struct intel_uncore_type ivt_unco > .fixed_ctr_bits = 48, > .fixed_ctr = SNBEP_MC_CHy_PCI_PMON_FIXED_CTR, > .fixed_ctl = SNBEP_MC_CHy_PCI_PMON_FIXED_CTL, > + .event_descs = snbep_uncore_imc_events, > IVT_UNCORE_PCI_COMMON_INIT(), > }; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
