Commit 644c2ae1 authored by James Morse's avatar James Morse Committed by Catalin Marinas

arm64: cpufeature: Test 'matches' pointer to find the end of the list

CPU feature code uses the desc field as a test to find the end of the list,
this means every entry must have a description. This generates noise for
entries in the list that aren't really features, but combinations of them.
e.g.
> CPU features: detected feature: Privileged Access Never
> CPU features: detected feature: PAN and not UAO

These combination features are needed for corner cases with alternatives,
where cpu features interact.

Change all walkers of the arm64_features[] and arm64_hwcaps[] lists to test
'matches' not 'desc', and only print 'desc' if it is non-NULL.
Signed-off-by: default avatarJames Morse <james.morse@arm.com>
Reviewed-by : Suzuki K Poulose <suzuki.poulose@arm.com>
Signed-off-by: default avatarCatalin Marinas <catalin.marinas@arm.com>
parent 57f4959b
...@@ -771,7 +771,7 @@ static void __init setup_cpu_hwcaps(void) ...@@ -771,7 +771,7 @@ static void __init setup_cpu_hwcaps(void)
int i; int i;
const struct arm64_cpu_capabilities *hwcaps = arm64_hwcaps; const struct arm64_cpu_capabilities *hwcaps = arm64_hwcaps;
for (i = 0; hwcaps[i].desc; i++) for (i = 0; hwcaps[i].matches; i++)
if (hwcaps[i].matches(&hwcaps[i])) if (hwcaps[i].matches(&hwcaps[i]))
cap_set_hwcap(&hwcaps[i]); cap_set_hwcap(&hwcaps[i]);
} }
...@@ -781,11 +781,11 @@ void update_cpu_capabilities(const struct arm64_cpu_capabilities *caps, ...@@ -781,11 +781,11 @@ void update_cpu_capabilities(const struct arm64_cpu_capabilities *caps,
{ {
int i; int i;
for (i = 0; caps[i].desc; i++) { for (i = 0; caps[i].matches; i++) {
if (!caps[i].matches(&caps[i])) if (!caps[i].matches(&caps[i]))
continue; continue;
if (!cpus_have_cap(caps[i].capability)) if (!cpus_have_cap(caps[i].capability) && caps[i].desc)
pr_info("%s %s\n", info, caps[i].desc); pr_info("%s %s\n", info, caps[i].desc);
cpus_set_cap(caps[i].capability); cpus_set_cap(caps[i].capability);
} }
...@@ -800,7 +800,7 @@ enable_cpu_capabilities(const struct arm64_cpu_capabilities *caps) ...@@ -800,7 +800,7 @@ enable_cpu_capabilities(const struct arm64_cpu_capabilities *caps)
{ {
int i; int i;
for (i = 0; caps[i].desc; i++) for (i = 0; caps[i].matches; i++)
if (caps[i].enable && cpus_have_cap(caps[i].capability)) if (caps[i].enable && cpus_have_cap(caps[i].capability))
on_each_cpu(caps[i].enable, NULL, true); on_each_cpu(caps[i].enable, NULL, true);
} }
...@@ -907,7 +907,7 @@ void verify_local_cpu_capabilities(void) ...@@ -907,7 +907,7 @@ void verify_local_cpu_capabilities(void)
return; return;
caps = arm64_features; caps = arm64_features;
for (i = 0; caps[i].desc; i++) { for (i = 0; caps[i].matches; i++) {
if (!cpus_have_cap(caps[i].capability) || !caps[i].sys_reg) if (!cpus_have_cap(caps[i].capability) || !caps[i].sys_reg)
continue; continue;
/* /*
...@@ -920,7 +920,7 @@ void verify_local_cpu_capabilities(void) ...@@ -920,7 +920,7 @@ void verify_local_cpu_capabilities(void)
caps[i].enable(NULL); caps[i].enable(NULL);
} }
for (i = 0, caps = arm64_hwcaps; caps[i].desc; i++) { for (i = 0, caps = arm64_hwcaps; caps[i].matches; i++) {
if (!cpus_have_hwcap(&caps[i])) if (!cpus_have_hwcap(&caps[i]))
continue; continue;
if (!feature_matches(__raw_read_system_reg(caps[i].sys_reg), &caps[i])) if (!feature_matches(__raw_read_system_reg(caps[i].sys_reg), &caps[i]))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment