]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
KVM: arm64: Convert TCR2_EL2 to config-driven sanitisation
authorMarc Zyngier <maz@kernel.org>
Mon, 14 Jul 2025 11:55:00 +0000 (12:55 +0100)
committerOliver Upton <oliver.upton@linux.dev>
Wed, 16 Jul 2025 03:39:42 +0000 (20:39 -0700)
As for other registers, convert the determination of the RES0 bits
affecting TCR2_EL2 to be driven by a table extracted from the 2025-06
JSON drop.

Signed-off-by: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/20250714115503.3334242-3-maz@kernel.org
Signed-off-by: Oliver Upton <oliver.upton@linux.dev>
arch/arm64/kvm/config.c
arch/arm64/kvm/nested.c

index 54911a93b0018e8e644496f8c4fd6b61e00ea66f..8265b722d442b5ae9a6dc9cfcc6b311285a1f4ab 100644 (file)
@@ -131,6 +131,9 @@ struct reg_bits_to_feat_map {
 #define FEAT_SPMU              ID_AA64DFR1_EL1, SPMU, IMP
 #define FEAT_SPE_nVM           ID_AA64DFR2_EL1, SPE_nVM, IMP
 #define FEAT_STEP2             ID_AA64DFR2_EL1, STEP, IMP
+#define FEAT_ASID2             ID_AA64MMFR4_EL1, ASID2, IMP
+#define FEAT_MEC               ID_AA64MMFR3_EL1, MEC, IMP
+#define FEAT_HAFT              ID_AA64MMFR1_EL1, HAFDBS, HAFT
 
 static bool not_feat_aa64el3(struct kvm *kvm)
 {
@@ -218,6 +221,21 @@ static bool feat_trbe_mpam(struct kvm *kvm)
                (read_sysreg_s(SYS_TRBIDR_EL1) & TRBIDR_EL1_MPAM));
 }
 
+static bool feat_asid2_e2h1(struct kvm *kvm)
+{
+       return kvm_has_feat(kvm, FEAT_ASID2) && !kvm_has_feat(kvm, FEAT_E2H0);
+}
+
+static bool feat_d128_e2h1(struct kvm *kvm)
+{
+       return kvm_has_feat(kvm, FEAT_D128) && !kvm_has_feat(kvm, FEAT_E2H0);
+}
+
+static bool feat_mec_e2h1(struct kvm *kvm)
+{
+       return kvm_has_feat(kvm, FEAT_MEC) && !kvm_has_feat(kvm, FEAT_E2H0);
+}
+
 static bool feat_ebep_pmuv3_ss(struct kvm *kvm)
 {
        return kvm_has_feat(kvm, FEAT_EBEP) || kvm_has_feat(kvm, FEAT_PMUv3_SS);
@@ -832,6 +850,28 @@ static const struct reg_bits_to_feat_map hcr_feat_map[] = {
        NEEDS_FEAT_FIXED(HCR_EL2_E2H, compute_hcr_e2h),
 };
 
+static const struct reg_bits_to_feat_map tcr2_el2_feat_map[] = {
+       NEEDS_FEAT(TCR2_EL2_FNG1        |
+                  TCR2_EL2_FNG0        |
+                  TCR2_EL2_A2,
+                  feat_asid2_e2h1),
+       NEEDS_FEAT(TCR2_EL2_DisCH1      |
+                  TCR2_EL2_DisCH0      |
+                  TCR2_EL2_D128,
+                  feat_d128_e2h1),
+       NEEDS_FEAT(TCR2_EL2_AMEC1, feat_mec_e2h1),
+       NEEDS_FEAT(TCR2_EL2_AMEC0, FEAT_MEC),
+       NEEDS_FEAT(TCR2_EL2_HAFT, FEAT_HAFT),
+       NEEDS_FEAT(TCR2_EL2_PTTWI       |
+                  TCR2_EL2_PnCH,
+                  FEAT_THE),
+       NEEDS_FEAT(TCR2_EL2_AIE, FEAT_AIE),
+       NEEDS_FEAT(TCR2_EL2_POE         |
+                  TCR2_EL2_E0POE,
+                  FEAT_S1POE),
+       NEEDS_FEAT(TCR2_EL2_PIE, FEAT_S1PIE),
+};
+
 static void __init check_feat_map(const struct reg_bits_to_feat_map *map,
                                  int map_size, u64 res0, const char *str)
 {
@@ -863,6 +903,8 @@ void __init check_feature_map(void)
                       __HCRX_EL2_RES0, "HCRX_EL2");
        check_feat_map(hcr_feat_map, ARRAY_SIZE(hcr_feat_map),
                       HCR_EL2_RES0, "HCR_EL2");
+       check_feat_map(tcr2_el2_feat_map, ARRAY_SIZE(tcr2_el2_feat_map),
+                      TCR2_EL2_RES0, "TCR2_EL2");
 }
 
 static bool idreg_feat_match(struct kvm *kvm, const struct reg_bits_to_feat_map *map)
@@ -1077,6 +1119,12 @@ void get_reg_fixed_bits(struct kvm *kvm, enum vcpu_sysreg reg, u64 *res0, u64 *r
                *res0 |= HCR_EL2_RES0 | (mask & ~fixed);
                *res1 = HCR_EL2_RES1 | (mask & fixed);
                break;
+       case TCR2_EL2:
+               *res0 = compute_res0_bits(kvm, tcr2_el2_feat_map,
+                                         ARRAY_SIZE(tcr2_el2_feat_map), 0, 0);
+               *res0 |= TCR2_EL2_RES0;
+               *res1 = TCR2_EL2_RES1;
+               break;
        default:
                WARN_ON_ONCE(1);
                *res0 = *res1 = 0;
index 5b191f4dc5668c89682082d27409327044ff512a..efb1f2caca6268b8fe374ab939fb6aa2731697bf 100644 (file)
@@ -1663,25 +1663,7 @@ int kvm_init_nv_sysregs(struct kvm_vcpu *vcpu)
        set_sysreg_masks(kvm, HFGITR2_EL2, res0, res1);
 
        /* TCR2_EL2 */
-       res0 = TCR2_EL2_RES0;
-       res1 = TCR2_EL2_RES1;
-       if (!kvm_has_feat(kvm, ID_AA64MMFR3_EL1, D128, IMP))
-               res0 |= (TCR2_EL2_DisCH0 | TCR2_EL2_DisCH1 | TCR2_EL2_D128);
-       if (!kvm_has_feat(kvm, ID_AA64MMFR3_EL1, MEC, IMP))
-               res0 |= TCR2_EL2_AMEC1 | TCR2_EL2_AMEC0;
-       if (!kvm_has_feat(kvm, ID_AA64MMFR1_EL1, HAFDBS, HAFT))
-               res0 |= TCR2_EL2_HAFT;
-       if (!kvm_has_feat(kvm, ID_AA64PFR1_EL1, THE, IMP))
-               res0 |= TCR2_EL2_PTTWI | TCR2_EL2_PnCH;
-       if (!kvm_has_feat(kvm, ID_AA64MMFR3_EL1, AIE, IMP))
-               res0 |= TCR2_EL2_AIE;
-       if (!kvm_has_s1poe(kvm))
-               res0 |= TCR2_EL2_POE | TCR2_EL2_E0POE;
-       if (!kvm_has_s1pie(kvm))
-               res0 |= TCR2_EL2_PIE;
-       if (!kvm_has_feat(kvm, ID_AA64MMFR1_EL1, VH, IMP))
-               res0 |= (TCR2_EL2_E0POE | TCR2_EL2_D128 |
-                        TCR2_EL2_AMEC1 | TCR2_EL2_DisCH0 | TCR2_EL2_DisCH1);
+       get_reg_fixed_bits(kvm, TCR2_EL2, &res0, &res1);
        set_sysreg_masks(kvm, TCR2_EL2, res0, res1);
 
        /* SCTLR_EL1 */