> > On Thu, Aug 6, 2015 at 11:52 AM, Liang, Kan <kan.li...@intel.com> wrote: > > > > > >> >> +static cpumask_t power_cstate_core_cpu_mask; > >> > > >> > That one typically does not need a cpumask. > >> > > >> You need to pick one CPU out of the multi-core. But it is for client > >> parts thus there is only one socket. At least this is my understanding. > >> > > > > CORE_C*_RESIDENCY are available for physical processor core. > > So logical processor in same physical processor core share the same > > counter. > > I think we need the cpumask to identify the default logical processor > > which do counting. > > > Did you restrict these events to system-wide mode only? > Yes
- Re: [PATCH V2 1/1] perf/x86: Add Intel power cstate PMUs ... Peter Zijlstra
- Re: [PATCH V2 1/1] perf/x86: Add Intel power cstate ... Stephane Eranian
- RE: [PATCH V2 1/1] perf/x86: Add Intel power cst... Liang, Kan
- Re: [PATCH V2 1/1] perf/x86: Add Intel power... Stephane Eranian
- RE: [PATCH V2 1/1] perf/x86: Add Intel p... Liang, Kan
- Re: [PATCH V2 1/1] perf/x86: Add In... Stephane Eranian
- RE: [PATCH V2 1/1] perf/x86: Ad... Liang, Kan
- Re: [PATCH V2 1/1] perf/x86... Stephane Eranian
- RE: [PATCH V2 1/1] perf/x86... Liang, Kan
- Re: [PATCH V2 1/1] perf/x86... Stephane Eranian
- RE: [PATCH V2 1/1] perf/x86... Liang, Kan
- Re: [PATCH V2 1/1] perf/x86: Add Intel power cst... Peter Zijlstra