Re: [PATCH 1/3] perf/x86/intel/bts: Fix confused ordering of PMU callbacks

From: Alexander Shishkin
Date: Wed Aug 24 2016 - 08:29:39 EST


Alexander Shishkin <alexander.shishkin@xxxxxxxxxxxxxxx> writes:

> The intel_bts driver is using a cpu-local 'started' variable to order
> callbacks and PMIs and make sure that AUX transactions don't get messed
> up. However, the ordering rules in regard to this variable is a complete
> mess, which recently resulted in perf_fuzzer-triggered warnings and
> panics.
>
> The general ordering rule that is patch is enforcing is that this
> cpu-local variable be set only when the cpu-local AUX transaction is
> active; consequently, this variable is to be checked before the AUX
> related bits can be touched.
>
> Signed-off-by: Alexander Shishkin <alexander.shishkin@xxxxxxxxxxxxxxx>

Ok, this one is broken, please disregard.

Regards,
--
Alex