Re: [PATCH 2/3] x86/topology: Fix AMD core count

From: Peter Zijlstra
Date: Sun Mar 20 2016 - 06:43:51 EST


On Sat, Mar 19, 2016 at 10:24:59AM +0100, Thomas Gleixner wrote:
> Unfortunately that will break stuff in event/amd/core.c, ras/mce_amd_inj.c
> which rely on the AMD interpretation of c->x86_max_cores.

So the AMD NB stuff in events/amd/core.c is only for Fam10, Fam15 got
its own uncore driver. (Fam10 had the uncore events through the same
counters as the core PMU with with 'fun' constraints).

And since Fam10 isn't affected by this change in x86_max_cores, this
_should_ work out, IF that NB code knows to switch off properly when not
required.