Re: [PATCH] arm64: KVM: Use SMCCC_ARCH_WORKAROUND_1 for Falkor BP hardening

[Mark Rutland]

On Mon, Mar 05, 2018 at 09:52:19AM -0600, Timur Tabi wrote:
> On Fri, Mar 2, 2018 at 3:50 PM, Shanker Donthineni
> <shankerd@xxxxxxxxxxxxxx> wrote:
> > diff --git a/arch/arm64/include/asm/cpucaps.h b/arch/arm64/include/asm/cpucaps.h
> > index bb26382..6ecc249 100644
> > --- a/arch/arm64/include/asm/cpucaps.h
> > +++ b/arch/arm64/include/asm/cpucaps.h
> > @@ -43,7 +43,7 @@
> >  #define ARM64_SVE                              22
> >  #define ARM64_UNMAP_KERNEL_AT_EL0              23
> >  #define ARM64_HARDEN_BRANCH_PREDICTOR          24
> > -#define ARM64_HARDEN_BP_POST_GUEST_EXIT                25
> > +/* #define ARM64_UNALLOCATED_ENTRY                     25 */
> 
> Why not just delete the entry?  

Deleting the entry, and renumbering subsequent entries is the right
thing to do.

> Is ARM64_NCAPS never supposed to get smaller?

It's internal to the kernel, and it makes sense to keep it as small as
possible. There are datastructures allocated for each entry, like the
static key array.

Thanks,
Mark.