[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180305155557.2iidi6346zcwz6r2@lakrids.cambridge.arm.com>
Date: Mon, 5 Mar 2018 15:55:57 +0000
From: Mark Rutland <mark.rutland@....com>
To: Timur Tabi <timur@...eaurora.org>
Cc: Shanker Donthineni <shankerd@...eaurora.org>,
Thomas Speier <tspeier@...eaurora.org>,
Vikram Sethi <vikrams@...eaurora.org>,
Sean Campbell <scampbel@...eaurora.org>,
Marc Zyngier <marc.zyngier@....com>,
Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will.deacon@....com>,
linux-kernel <linux-kernel@...r.kernel.org>,
Christoffer Dall <christoffer.dall@...aro.org>,
kvmarm <kvmarm@...ts.cs.columbia.edu>,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH] arm64: KVM: Use SMCCC_ARCH_WORKAROUND_1 for Falkor BP
hardening
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@...eaurora.org> 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.
Powered by blists - more mailing lists