|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [PATCH 02/30] x86/CPUID: enable APX sub-leaf
The logic is modeled as closely as possible after that of leaf 7 and AVX10
sub-leaf handling.
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
--- a/tools/libs/light/libxl_cpuid.c
+++ b/tools/libs/light/libxl_cpuid.c
@@ -344,6 +344,7 @@ int libxl_cpuid_parse_config(libxl_cpuid
MSR_ENTRY(0x10a, CPUID_REG_EDX),
CPUID_ENTRY(0x80000021, NA, CPUID_REG_ECX),
CPUID_ENTRY(0x00000024, 1, CPUID_REG_ECX),
+ CPUID_ENTRY(0x00000029, 0, CPUID_REG_EBX),
#undef MSR_ENTRY
#undef CPUID_ENTRY
};
--- a/tools/misc/xen-cpuid.c
+++ b/tools/misc/xen-cpuid.c
@@ -39,6 +39,7 @@ static const struct {
{ "MSR_ARCH_CAPS.hi", "m10Ah" },
{ "CPUID 0x80000021.ecx", "e21c" },
{ "CPUID 0x00000024:1.ecx", "24c1" },
+ { "CPUID 0x00000029:0.ebx", "29b0" },
};
#define COL_ALIGN "24"
--- a/xen/arch/x86/cpu/common.c
+++ b/xen/arch/x86/cpu/common.c
@@ -605,6 +605,11 @@ void identify_cpu(struct cpuinfo_x86 *c)
&tmp);
}
+ if (cpu_has(c, X86_FEATURE_APX_F) && c->cpuid_level >= 0x29)
+ cpuid_count(0x29, 0, &tmp,
+ &c->x86_capability[FEATURESET_29b0],
+ &tmp, &tmp);
+
if (test_bit(X86_FEATURE_ARCH_CAPS, c->x86_capability)) {
val = rdmsr(MSR_ARCH_CAPABILITIES);
c->x86_capability[FEATURESET_m10Al] = val;
--- a/xen/arch/x86/cpu-policy.c
+++ b/xen/arch/x86/cpu-policy.c
@@ -287,6 +287,13 @@ static void recalculate_misc(struct cpu_
memset(p->avx10.raw, 0, sizeof(p->avx10.raw));
}
+ zero_leaves(p->basic.raw, 0x25, 0x28);
+
+ p->apx.raw[0].c = 0;
+ p->apx.raw[0].d = 0;
+ if ( !p->feat.apx_f )
+ memset(p->apx.raw, 0, sizeof(p->apx.raw));
+
p->extd.e1d &= ~CPUID_COMMON_1D_FEATURES;
/* Most of Power/RAS hidden from guests. */
@@ -423,6 +430,7 @@ static void __init guest_common_max_leav
p->basic.max_leaf = ARRAY_SIZE(p->basic.raw) - 1;
p->feat.max_subleaf = ARRAY_SIZE(p->feat.raw) - 1;
p->avx10.max_subleaf = ARRAY_SIZE(p->avx10.raw) - 1;
+ p->apx.max_subleaf = ARRAY_SIZE(p->apx.raw) - 1;
p->extd.max_leaf = 0x80000000U + ARRAY_SIZE(p->extd.raw) - 1;
}
@@ -432,6 +440,7 @@ static void __init guest_common_default_
p->basic.max_leaf = host_cpu_policy.basic.max_leaf;
p->feat.max_subleaf = host_cpu_policy.feat.max_subleaf;
p->avx10.max_subleaf = host_cpu_policy.avx10.max_subleaf;
+ p->apx.max_subleaf = host_cpu_policy.apx.max_subleaf;
p->extd.max_leaf = host_cpu_policy.extd.max_leaf;
}
@@ -1043,6 +1052,7 @@ void recalculate_cpuid_policy(struct dom
p->basic.max_leaf = min(p->basic.max_leaf, max->basic.max_leaf);
p->feat.max_subleaf = min(p->feat.max_subleaf, max->feat.max_subleaf);
p->avx10.max_subleaf = min(p->avx10.max_subleaf, max->avx10.max_subleaf);
+ p->apx.max_subleaf = min(p->apx.max_subleaf, max->apx.max_subleaf);
p->extd.max_leaf = 0x80000000U | min(p->extd.max_leaf & 0xffff,
((p->x86_vendor & (X86_VENDOR_AMD |
X86_VENDOR_HYGON))
@@ -1091,6 +1101,8 @@ void recalculate_cpuid_policy(struct dom
__clear_bit(X86_FEATURE_XSAVE, fs);
if ( p->basic.max_leaf < 0x24 )
__clear_bit(X86_FEATURE_AVX10, fs);
+ if ( p->basic.max_leaf < 0x29 )
+ __clear_bit(X86_FEATURE_APX_F, fs);
sanitise_featureset(fs);
@@ -1188,6 +1200,8 @@ void __init init_dom0_cpuid_policy(struc
*/
if ( p->feat.avx10 )
p->avx10 = max->avx10;
+ if ( p->feat.apx_f )
+ p->apx = max->apx;
}
/*
@@ -1222,6 +1236,8 @@ static void __init __maybe_unused build_
sizeof(raw_cpu_policy.xstate.raw));
BUILD_BUG_ON(sizeof(raw_cpu_policy.avx10) !=
sizeof(raw_cpu_policy.avx10.raw));
+ BUILD_BUG_ON(sizeof(raw_cpu_policy.apx) !=
+ sizeof(raw_cpu_policy.apx.raw));
BUILD_BUG_ON(sizeof(raw_cpu_policy.extd) !=
sizeof(raw_cpu_policy.extd.raw));
}
--- a/xen/arch/x86/cpuid.c
+++ b/xen/arch/x86/cpuid.c
@@ -235,6 +235,15 @@ void guest_cpuid(const struct vcpu *v, u
*res = array_access_nospec(p->avx10.raw, subleaf);
break;
+ case 0x29:
+ ASSERT(p->apx.max_subleaf < ARRAY_SIZE(p->apx.raw));
+ if ( subleaf > min_t(uint32_t, p->apx.max_subleaf,
+ ARRAY_SIZE(p->apx.raw) - 1) )
+ return;
+
+ *res = array_access_nospec(p->apx.raw, subleaf);
+ break;
+
default:
*res = array_access_nospec(p->basic.raw, leaf);
break;
--- a/xen/arch/x86/lib/cpu-policy/copy-to-buffer.c
+++ b/xen/arch/x86/lib/cpu-policy/copy-to-buffer.c
@@ -130,6 +130,13 @@ int x86_cpuid_copy_to_buffer(const struc
COPY_LEAF(leaf, subleaf, &p->avx10.raw[subleaf]);
break;
+ case 0x29:
+ for ( subleaf = 0;
+ subleaf <= MIN(p->apx.max_subleaf,
+ ARRAY_SIZE(p->apx.raw) - 1); ++subleaf )
+ COPY_LEAF(leaf, subleaf, &p->apx.raw[subleaf]);
+ break;
+
default:
COPY_LEAF(leaf, XEN_CPUID_NO_SUBLEAF, &p->basic.raw[leaf]);
break;
--- a/xen/arch/x86/lib/cpu-policy/copy-from-buffer.c
+++ b/xen/arch/x86/lib/cpu-policy/copy-from-buffer.c
@@ -115,6 +115,13 @@ int x86_cpuid_copy_from_buffer(struct cp
array_access_nospec(p->avx10.raw, data.subleaf) = l;
break;
+ case 0x29:
+ if ( data.subleaf >= ARRAY_SIZE(p->apx.raw) )
+ goto out_of_range;
+
+ array_access_nospec(p->apx.raw, data.subleaf) = l;
+ break;
+
default:
if ( data.subleaf != XEN_CPUID_NO_SUBLEAF )
goto out_of_range;
--- a/xen/arch/x86/lib/cpu-policy/cpuid.c
+++ b/xen/arch/x86/lib/cpu-policy/cpuid.c
@@ -83,6 +83,7 @@ void x86_cpu_policy_to_featureset(
fs[FEATURESET_m10Ah] = p->arch_caps.hi;
fs[FEATURESET_e21c] = p->extd.e21c;
fs[FEATURESET_24c1] = p->avx10._24c1;
+ fs[FEATURESET_29b0] = p->apx._29b0;
}
void x86_cpu_featureset_to_policy(
@@ -108,6 +109,7 @@ void x86_cpu_featureset_to_policy(
p->arch_caps.hi = fs[FEATURESET_m10Ah];
p->extd.e21c = fs[FEATURESET_e21c];
p->avx10._24c1 = fs[FEATURESET_24c1];
+ p->apx._29b0 = fs[FEATURESET_29b0];
}
void x86_cpu_policy_recalc_synth(struct cpu_policy *p)
@@ -127,7 +129,7 @@ void x86_cpu_policy_fill_native(struct c
switch ( i )
{
case 0x4: case 0x7: case 0xb: case 0xd:
- case 0x24:
+ case 0x24: case 0x29:
/* Multi-invocation leaves. Deferred. */
continue;
}
@@ -230,6 +232,15 @@ void x86_cpu_policy_fill_native(struct c
cpuid_count_leaf(0x24, i, &p->avx10.raw[i]);
}
+ if ( p->basic.max_leaf >= 0x29 )
+ {
+ cpuid_count_leaf(0x29, 0, &p->apx.raw[0]);
+
+ for ( i = 1; i <= MIN(p->apx.max_subleaf,
+ ARRAY_SIZE(p->apx.raw) - 1); ++i )
+ cpuid_count_leaf(0x24, i, &p->apx.raw[i]);
+ }
+
/* Extended leaves. */
cpuid_leaf(0x80000000U, &p->extd.raw[0]);
for ( i = 1; i <= MIN(p->extd.max_leaf & 0xffffU,
@@ -302,6 +313,9 @@ void x86_cpu_policy_clear_out_of_range_l
if ( p->basic.max_leaf < 0x24 )
memset(p->avx10.raw, 0, sizeof(p->avx10.raw));
+ if ( p->basic.max_leaf < 0x29 )
+ memset(p->apx.raw, 0, sizeof(p->apx.raw));
+
zero_leaves(p->extd.raw,
((p->extd.max_leaf >> 16) == 0x8000
? (p->extd.max_leaf & 0xffff) + 1 : 0),
@@ -316,6 +330,8 @@ void __init x86_cpu_policy_bound_max_lea
min_t(uint32_t, p->feat.max_subleaf, ARRAY_SIZE(p->feat.raw) - 1);
p->avx10.max_subleaf =
min_t(uint32_t, p->avx10.max_subleaf, ARRAY_SIZE(p->avx10.raw) - 1);
+ p->apx.max_subleaf =
+ min_t(uint32_t, p->apx.max_subleaf, ARRAY_SIZE(p->apx.raw) - 1);
p->extd.max_leaf = 0x80000000U | min_t(uint32_t, p->extd.max_leaf & 0xffff,
ARRAY_SIZE(p->extd.raw) - 1);
}
@@ -345,6 +361,8 @@ void x86_cpu_policy_shrink_max_leaves(st
p->basic.raw[0x24] = p->avx10.raw[0];
+ p->basic.raw[0x29] = p->apx.raw[0];
+
for ( i = p->basic.max_leaf; i; --i )
if ( p->basic.raw[i].a | p->basic.raw[i].b |
p->basic.raw[i].c | p->basic.raw[i].d )
--- a/xen/arch/x86/lib/cpu-policy/policy.c
+++ b/xen/arch/x86/lib/cpu-policy/policy.c
@@ -28,6 +28,9 @@ int x86_cpu_policies_are_compatible(cons
guest->avx10.max_subleaf > host->avx10.max_subleaf )
FAIL_CPUID(0x24, 0);
+ if ( guest->apx.max_subleaf > host->apx.max_subleaf )
+ FAIL_CPUID(0x29, 0);
+
if ( guest->extd.max_leaf > host->extd.max_leaf )
FAIL_CPUID(0x80000000U, NA);
--- a/xen/include/public/arch-x86/cpufeatureset.h
+++ b/xen/include/public/arch-x86/cpufeatureset.h
@@ -416,6 +416,9 @@ XEN_CPUFEATURE(AVX10_BMM, 19*32
XEN_CPUFEATURE(AVX10_V1_AUX, 19*32+ 2) /*a AVX10 V1 Auxiliary
Instructions */
XEN_CPUFEATURE(AVX10_V2_AUX, 19*32+ 3) /*a AVX10 V2 Auxiliary
Instructions */
+/* Intel-defined CPU features, CPUID level 0x00000029:0.ebx, word 20 */
+XEN_CPUFEATURE(APX_NCI_NDD_NF, 20*32+ 0) /*a APX NCI/NDD/NF instruction
forms */
+
#endif /* XEN_CPUFEATURE */
/* Clean up from a default include. Close the enum (for C). */
--- a/xen/include/xen/lib/x86/cpu-policy.h
+++ b/xen/include/xen/lib/x86/cpu-policy.h
@@ -24,6 +24,7 @@
#define FEATURESET_m10Ah 17 /* 0x0000010a.edx */
#define FEATURESET_e21c 18 /* 0x80000021.ecx */
#define FEATURESET_24c1 19 /* 0x00000024:1.ecx */
+#define FEATURESET_29b0 20 /* 0x00000029:0.ebx */
struct cpuid_leaf
{
@@ -60,12 +61,13 @@ unsigned int x86_cpuid_lookup_vendor(uin
*/
const char *x86_cpuid_vendor_to_str(unsigned int vendor);
-#define CPUID_GUEST_NR_BASIC (0x24u + 1)
+#define CPUID_GUEST_NR_BASIC (0x29u + 1)
#define CPUID_GUEST_NR_CACHE (5u + 1)
#define CPUID_GUEST_NR_FEAT (2u + 1)
#define CPUID_GUEST_NR_TOPO (1u + 1)
#define CPUID_GUEST_NR_XSTATE (62u + 1)
#define CPUID_GUEST_NR_AVX10 (1u + 1)
+#define CPUID_GUEST_NR_APX (0u + 1)
#define CPUID_GUEST_NR_EXTD_INTEL (0x8u + 1)
#define CPUID_GUEST_NR_EXTD_AMD (0x21u + 1)
#define CPUID_GUEST_NR_EXTD MAX(CPUID_GUEST_NR_EXTD_INTEL, \
@@ -287,6 +289,20 @@ struct cpu_policy
};
} avx10;
+ /* Structured APX information leaf: 0x000000029[xx] */
+ union {
+ struct cpuid_leaf raw[CPUID_GUEST_NR_APX];
+ struct {
+ /* Subleaf 0. */
+ uint32_t max_subleaf;
+ union {
+ uint32_t _29b0;
+ struct { DECL_BITFIELD(29b0); };
+ };
+ uint32_t /* c */:32, /* d */:32;
+ };
+ } apx;
+
/* Extended leaves: 0x800000xx */
union {
struct cpuid_leaf raw[CPUID_GUEST_NR_EXTD];
--- a/xen/tools/gen-cpuid.py
+++ b/xen/tools/gen-cpuid.py
@@ -355,6 +355,9 @@ def crunch_numbers(state):
# computational instructions. All further AMX features are built on
top
# of AMX-TILE.
AMX_TILE: [AMX_BF16, AMX_INT8, AMX_FP16, AMX_COMPLEX],
+
+ # APX discrete features.
+ APX_F: [APX_NCI_NDD_NF],
}
deep_features = tuple(sorted(deps.keys()))
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |