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

From: Borislav Petkov
Date: Sun Mar 20 2016 - 07:05:35 EST


On Sun, Mar 20, 2016 at 11:39:46AM +0100, Peter Zijlstra wrote:
> 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.

But but, amd_core_pmu_init() is called on F15h too. It only gets the
different F15h event constraints and MSRs...

--
Regards/Gruss,
Boris.

ECO tip #101: Trim your mails when you reply.