Re: [PATCH RESEND 4/5] x86/MCE: Make number of MCA banks per_cpu

From: Borislav Petkov
Date: Mon Apr 08 2019 - 16:50:29 EST


On Mon, Apr 08, 2019 at 08:42:36PM +0000, Luck, Tony wrote:
> > Actually, it should not be exported at all. A function returning the num
> > banks is better instead.
>
> Are all the places it is used in non-pre-emptible sections of code? Looping
> in the CMCI and #MC handlers should be fine. But do we need get_cpu()/put_cpu()
> in any places?

That export is needed only in the mce injector. Actually, it would be
much cleaner if the injector would find out the count straight from the
MSR as it does now, but be changed to do rdmsr_on_cpu() now, since can
have different num_banks on a CPU.

--
Regards/Gruss,
Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.