On Mon, Feb 22, 2016 at 11:06:56AM -0000, Thomas Gleixner wrote: > The Knights Landings support added the events and the detection case, but then > returns 0 without actually initializing the driver. > > Fixes: 3a2a7797326a4 "perf/x86/intel/rapl: Add support for Knights Landing > (KNL)" > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: Dasaratharaman Chandramouli <[email protected]> > --- > arch/x86/kernel/cpu/perf_event_intel_rapl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Index: b/arch/x86/kernel/cpu/perf_event_intel_rapl.c > =================================================================== > --- a/arch/x86/kernel/cpu/perf_event_intel_rapl.c > +++ b/arch/x86/kernel/cpu/perf_event_intel_rapl.c > @@ -731,7 +731,7 @@ static int __init rapl_pmu_init(void) > rapl_add_quirk(rapl_hsw_server_quirk); > rapl_cntr_mask = RAPL_IDX_KNL; > rapl_pmu_events_group.attrs = rapl_events_knl_attr; > - > + break; > default: > /* unsupported */ > return 0;
Cc: <[email protected]> # 4.3 -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply.
