Re: [RFC PATCH 0/5] x86/cpufeature: Cleanups and improvements

From: Brian Gerst
Date: Thu Jan 21 2016 - 16:53:21 EST


On Thu, Jan 21, 2016 at 1:34 PM, Borislav Petkov <bp@xxxxxxxxx> wrote:
> From: Borislav Petkov <bp@xxxxxxx>
>
> So here's the first version of what we talked about. I've ran randbuilds
> on it for a night because of patch 1 with all the header reorg and we
> all know how that can be painful. So this version passes all randbuild
> configs I had generated.
>
> In talking about patch 1, it is a bit big but if you look at the
> diffstat, it basically is fixing a bunch of includes and the meat of it
> being the carving out of the X86_FEATURE_* things to a separate file.
>
> Rest of the patches should be pretty self-explanatory, check the
> respective commit messages.
>
> I haven't tested it functionally yet (booting in kvm doesn't really
> count) - thus the RFC tag. I thought I should send it out first so that
> people can doublecheck me on the direction this is going.
>
> Thanks!
>
> Borislav Petkov (4):
> x86/cpufeature: Carve out X86_FEATURE_*
> x86/cpufeature: Remove static_cpu_has()
> x86/cpufeature: Get rid of the non-asm goto variant
> x86/alternatives: Add an auxilary section
>
> Brian Gerst (1):
> x86/alternatives: Discard dynamic check after init

Tested it and it works. Looks good to me. Only thing was it didn't
apply to tip/master due to a new cpuid flag added.

--
Brian Gerst