From: "Liang, Kan" <kan.liang@intel.com>
To: Stephane Eranian <eranian@google.com>
CC: Peter Zijlstra <a.p.zijlstra@chello.nl>,
        "mingo@redhat.com" <mingo@redhat.com>,
        Arnaldo Carvalho de Melo <acme@kernel.org>,
        "ak@linux.intel.com" <ak@linux.intel.com>,
        LKML <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH 1/1] perf/x86: Add Intel power cstate PMUs support
Thread-Topic: [PATCH 1/1] perf/x86: Add Intel power cstate PMUs support
Thread-Index: AQHQxkqP3Pe++2jjf02AQx0+fSwp7J3qkLUAgACIR2A=
Date: Fri, 24 Jul 2015 20:57:43 +0000
Message-ID: <37D7C6CF3E00A74B8858931C1DB2F077018CC567@SHSMSX103.ccr.corp.intel.com>
References: <1437741564-8717-1-git-send-email-kan.liang@intel.com>
 <CABPqkBTSRYMWYXVQCjvCuR328QFAWo9B2SXVftaN3MCF_GGC_A@mail.gmail.com>
In-Reply-To: <CABPqkBTSRYMWYXVQCjvCuR328QFAWo9B2SXVftaN3MCF_GGC_A@mail.gmail.com>
Accept-Language: zh-CN, en-US
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 2077
Lines: 41

> > + *     MSR_PKG_C6_RESIDENCY:  Package C6 Residency Counter.
> > + *                            perf code: 0x06
> > + *                            Available model: NHM,WSM,SNB,IVB,HSW,BDW
> > + *                            Scope: Package (physical package)
> > + *     MSR_PKG_C7_RESIDENCY:  Package C7 Residency Counter.
> > + *                            perf code: 0x07
> > + *                            Available model: NHM,WSM,SNB,IVB,HSW,BDW
> > + *                            Scope: Package (physical package)
> > + *     MSR_PKG_C8_RESIDENCY:  Package C8 Residency Counter.
> > + *                            perf code: 0x08
> > + *                            Available model: HSW ULT only
> > + *                            Scope: Package (physical package)
> > + *     MSR_PKG_C9_RESIDENCY:  Package C9 Residency Counter.
> > + *                            perf code: 0x09
> > + *                            Available model: HSW ULT only
> > + *                            Scope: Package (physical package)
> > + *     MSR_PKG_C10_RESIDENCY: Package C10 Residency Counter.
> > + *                            perf code: 0x0a
> > + *                            Available model: HSW ULT only
> > + *                            Scope: Package (physical package)
> > + *     MSR_SLM_PKG_C6_RESIDENCY: Package C6 Residency Counter for
> SLM.
> > + *                            perf code: 0x0b
> > + *                            Available model: SLM,AMT
> > + *                            Scope: Package (physical package)
> > + *
> > + */
> > +
> Why would the user (and the tools) have to change the event code to
> measure,  let's say C6 residency, between SLM (0xb) and and BDW (0x6)?
> Are they not measuring the same thing?
> 

Yes, they are measuring the same thing, but with different MSR.
I will make their event code consistent and special handle PKG_C6
in event_init in next version.


Thanks,
Kan
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?