[RFC PATCH v7 18/28] KVM: arm64: config: Use functions from spe.c to test FEAT_SPE_{FnE,FDS}
Lorenzo Stoakes (ARM)
ljs at kernel.org
Thu Oct 29 10:47:52 PDT 2026
On Thu, Sep 03, 2026 at 05:06:13PM +0100, Alexandru Elisei wrote:
> KVM's FGU mechanism will trap the registers introduced by FEAT_SPE_FnE and
> FEAT_SPE_FDS if the feature is not present for the VM. Move the functions
> that check for the presence of these features out of config.c and into
> spe.c, since that's where the bulk of SPE virtualization lies.
>
> Signed-off-by: Alexandru Elisei <alexandru.elisei at arm.com>
Aside from naming nit below, LGTM and:
Reviewed-by: Lorenzo Stoakes (ARM) <ljs at kernel.org>
> ---
> arch/arm64/include/asm/kvm_spe.h | 11 +++++++++++
> arch/arm64/kvm/config.c | 30 +++++-------------------------
> arch/arm64/kvm/spe.c | 22 ++++++++++++++++++++++
> 3 files changed, 38 insertions(+), 25 deletions(-)
>
> diff --git a/arch/arm64/include/asm/kvm_spe.h b/arch/arm64/include/asm/kvm_spe.h
> index 3d8e9720daca..618051dcf59f 100644
> --- a/arch/arm64/include/asm/kvm_spe.h
> +++ b/arch/arm64/include/asm/kvm_spe.h
> @@ -43,6 +43,9 @@ u8 kvm_spe_get_pmsver(struct kvm *kvm);
>
> bool kvm_spe_write_sysreg(struct kvm_vcpu *vcpu, int reg, u64 val);
> u64 kvm_spe_read_sysreg(struct kvm_vcpu *vcpu, int reg);
> +
> +bool kvm_spe_has_feat_spe_fne(struct kvm *kvm);
> +bool kvm_spe_has_feat_spe_fds(struct kvm *kvm);
These both say spe twice, can probably cut the 2nd one in each case I think
:) Could you drop the 'feat' too, kvm_spe_has_fne() and kvm_spe_has_fds()
seem better!
> #else
> struct kvm_spe {
> };
> @@ -99,6 +102,14 @@ static inline u64 kvm_spe_read_sysreg(struct kvm_vcpu *vcpu, int reg)
> {
> return 0;
> }
> +static inline bool kvm_spe_has_feat_spe_fne(struct kvm *kvm)
> +{
> + return false;
> +}
> +static inline bool kvm_spe_has_feat_spe_fds(struct kvm *kvm)
> +{
> + return false;
> +}
> #endif /* CONFIG_KVM_ARM_SPE */
>
> #endif /* __ARM64_KVM_SPE_H__ */
> diff --git a/arch/arm64/kvm/config.c b/arch/arm64/kvm/config.c
> index 1053676551af..78e414ed736d 100644
> --- a/arch/arm64/kvm/config.c
> +++ b/arch/arm64/kvm/config.c
> @@ -7,6 +7,7 @@
> #include <linux/kvm_host.h>
> #include <asm/kvm_emulate.h>
> #include <asm/kvm_nested.h>
> +#include <asm/kvm_spe.h>
> #include <asm/sysreg.h>
>
> /*
> @@ -159,7 +160,6 @@ struct reg_feat_map_desc {
> #define FEAT_GICv3 ID_AA64PFR0_EL1, GIC, IMP
> #define FEAT_LOR ID_AA64MMFR1_EL1, LO, IMP
> #define FEAT_SPEv1p2 ID_AA64DFR0_EL1, PMSVer, V1P2
> -#define FEAT_SPEv1p4 ID_AA64DFR0_EL1, PMSVer, V1P4
> #define FEAT_SPEv1p5 ID_AA64DFR0_EL1, PMSVer, V1P5
> #define FEAT_ATS1A ID_AA64ISAR2_EL1, ATS1A, IMP
> #define FEAT_SPECRES2 ID_AA64ISAR1_EL1, SPECRES, COSP_RCTX
> @@ -292,26 +292,6 @@ static bool feat_sme_smps(struct kvm *kvm)
> (read_sysreg_s(SYS_SMIDR_EL1) & SMIDR_EL1_SMPS));
> }
>
> -static bool feat_spe_fds(struct kvm *kvm)
> -{
> - /*
> - * Revisit this if KVM ever supports SPE -- this really should
> - * look at the guest's view of PMSIDR_EL1.
> - */
> - return (kvm_has_feat(kvm, FEAT_SPEv1p4) &&
> - (read_sysreg_s(SYS_PMSIDR_EL1) & PMSIDR_EL1_FDS));
> -}
> -
> -static bool feat_spe_fne(struct kvm *kvm)
> -{
> - /*
> - * Revisit this if KVM ever supports SPE -- this really should
> - * look at the guest's view of PMSIDR_EL1.
> - */
> - return (kvm_has_feat(kvm, FEAT_SPEv1p2) &&
> - (read_sysreg_s(SYS_PMSIDR_EL1) & PMSIDR_EL1_FnE));
> -}
> -
> static bool feat_trbe_mpam(struct kvm *kvm)
> {
> /*
> @@ -547,7 +527,7 @@ static const struct reg_bits_to_feat_map hdfgrtr_feat_map[] = {
> HDFGRTR_EL2_PMBPTR_EL1 |
> HDFGRTR_EL2_PMBLIMITR_EL1,
> FEAT_SPE),
> - NEEDS_FEAT(HDFGRTR_EL2_nPMSNEVFR_EL1, feat_spe_fne),
> + NEEDS_FEAT(HDFGRTR_EL2_nPMSNEVFR_EL1, kvm_spe_has_feat_spe_fne),
> NEEDS_FEAT(HDFGRTR_EL2_nBRBDATA |
> HDFGRTR_EL2_nBRBCTL |
> HDFGRTR_EL2_nBRBIDR,
> @@ -615,7 +595,7 @@ static const struct reg_bits_to_feat_map hdfgwtr_feat_map[] = {
> HDFGWTR_EL2_PMBPTR_EL1 |
> HDFGWTR_EL2_PMBLIMITR_EL1,
> FEAT_SPE),
> - NEEDS_FEAT(HDFGWTR_EL2_nPMSNEVFR_EL1, feat_spe_fne),
> + NEEDS_FEAT(HDFGWTR_EL2_nPMSNEVFR_EL1, kvm_spe_has_feat_spe_fne),
> NEEDS_FEAT(HDFGWTR_EL2_nBRBDATA |
> HDFGWTR_EL2_nBRBCTL,
> FEAT_BRBE),
> @@ -851,7 +831,7 @@ static const struct reg_bits_to_feat_map hdfgrtr2_feat_map[] = {
> HDFGRTR2_EL2_nPMSSDATA,
> FEAT_PMUv3_SS),
> NEEDS_FEAT(HDFGRTR2_EL2_nPMIAR_EL1, FEAT_SEBEP),
> - NEEDS_FEAT(HDFGRTR2_EL2_nPMSDSFR_EL1, feat_spe_fds),
> + NEEDS_FEAT(HDFGRTR2_EL2_nPMSDSFR_EL1, kvm_spe_has_feat_spe_fds),
> NEEDS_FEAT(HDFGRTR2_EL2_nPMBMAR_EL1, FEAT_SPE_nVM),
> NEEDS_FEAT(HDFGRTR2_EL2_nSPMACCESSR_EL1 |
> HDFGRTR2_EL2_nSPMCNTEN |
> @@ -884,7 +864,7 @@ static const struct reg_bits_to_feat_map hdfgwtr2_feat_map[] = {
> feat_pmuv3p9),
> NEEDS_FEAT(HDFGWTR2_EL2_nPMSSCR_EL1, FEAT_PMUv3_SS),
> NEEDS_FEAT(HDFGWTR2_EL2_nPMIAR_EL1, FEAT_SEBEP),
> - NEEDS_FEAT(HDFGWTR2_EL2_nPMSDSFR_EL1, feat_spe_fds),
> + NEEDS_FEAT(HDFGWTR2_EL2_nPMSDSFR_EL1, kvm_spe_has_feat_spe_fds),
> NEEDS_FEAT(HDFGWTR2_EL2_nPMBMAR_EL1, FEAT_SPE_nVM),
> NEEDS_FEAT(HDFGWTR2_EL2_nSPMACCESSR_EL1 |
> HDFGWTR2_EL2_nSPMCNTEN |
> diff --git a/arch/arm64/kvm/spe.c b/arch/arm64/kvm/spe.c
> index 8c173a1f48bd..3b285b45332b 100644
> --- a/arch/arm64/kvm/spe.c
> +++ b/arch/arm64/kvm/spe.c
> @@ -113,6 +113,28 @@ void kvm_spe_destroy_vm(struct kvm *kvm)
> module_put(spe_pmu->pmu.module);
> }
>
> +bool kvm_spe_has_feat_spe_fds(struct kvm *kvm)
> +{
> + struct arm_spe_pmu *spe_pmu = kvm->arch.kvm_spe.spe_pmu;
> +
> + if (!spe_pmu)
> + return false;
> +
> + return kvm_has_feat(kvm, ID_AA64DFR0_EL1, PMSVer, V1P4) &&
> + FIELD_GET(PMSIDR_EL1_FDS, spe_pmu->pmsidr_el1);
> +}
> +
> +bool kvm_spe_has_feat_spe_fne(struct kvm *kvm)
> +{
> + struct arm_spe_pmu *spe_pmu = kvm->arch.kvm_spe.spe_pmu;
> +
> + if (!spe_pmu)
> + return false;
> +
> + return kvm_has_feat(kvm, ID_AA64DFR0_EL1, PMSVer, V1P2) &&
> + FIELD_GET(PMSIDR_EL1_FnE, spe_pmu->pmsidr_el1);
> +}
> +
> bool kvm_spe_write_sysreg(struct kvm_vcpu *vcpu, int reg, u64 val)
> {
> struct kvm_spe *kvm_spe = &vcpu->kvm->arch.kvm_spe;
> --
> 2.43.0
>
>
--
Cheers, Lorenzo
More information about the linux-arm-kernel
mailing list