Re: [PATCH v3 05/46] perf/x86/intel/cmt: add per-package locks

From: Thomas Gleixner
Date: Fri Nov 11 2016 - 04:43:56 EST


On Fri, 11 Nov 2016, Peter Zijlstra wrote:
> Well, its very hard to suggest alternatives, because there simply isn't
> anything of content here. This patch just adds locks, and the next few
> patches don't describe much either.
>
> Why can't this be RCU?

AFAICT from looking at later patches the context switch path can do RMID
borrowing/stealing whatever things which need protection of the package
data. Other pathes (setup/rotation/ ...) take all the package locks to
prevent concurrent RMID operations.

I still have not figured out why all of this is necessary and unfortunately
there is no real coherent epxlanation of the overall design. The cover
letter is not really helpful either.

Thanks,

tglx