Re: [PATCH v4 1/2] perf/x86/intel/rapl: support Skylake RAPL domains

[Srinivas Pandruvada]

On Wed, 2016-04-20 at 17:43 +0200, Peter Zijlstra wrote:
> On Sun, Apr 17, 2016 at 03:03:00PM -0700, Srinivas Pandruvada wrote:
> > 
> > @@ -729,6 +771,11 @@ static int __init rapl_pmu_init(void)
> > Â		rapl_cntr_mask = RAPL_IDX_SRV;
> > Â		rapl_pmu_events_group.attrs =
> > rapl_events_srv_attr;
> > Â		break;
> > +	case 78: /* Skylake client */
> > +	case 94: /* Skylake H/S */
> > +		rapl_cntr_mask = RAPL_IDX_SKL_CLN;
> > +		rapl_pmu_events_group.attrs =
> > rapl_events_skl_attr;
> > +		break;
> > Â	case 87: /* Knights Landing */
> > Â		apply_quirk = true;
> > Â		rapl_cntr_mask = RAPL_IDX_KNL;
> Please post this against tip/perf/core, all this code got munged in
> patch:
> 
> Â 4b6e2571bf00 ("x86/perf/intel/rapl: Make the Intel RAPL PMU driver
> modular")
I will repost.

> 
> Also; what about 85 (skl-server)?
I didn't get chance to test RAPL on SKL server. It should follow BDW
server, but want to test before.

Thanks,
Srinivas

> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm"
> in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info atÂÂhttp://vger.kernel.org/majordomo-info.html