Message ID | 20210617165105.574178-3-aneesh.kumar@linux.ibm.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Add support for FORM2 associativity | expand |
On Thu, Jun 17, 2021 at 10:21:00PM +0530, Aneesh Kumar K.V wrote: > No functional change in this patch I've been convinced of your other rename, but I'm not yet convinced this one actually clarifies anything. > > Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com> > --- > arch/powerpc/mm/numa.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c > index 8365b298ec48..132813dd1a6c 100644 > --- a/arch/powerpc/mm/numa.c > +++ b/arch/powerpc/mm/numa.c > @@ -56,7 +56,7 @@ static int n_mem_addr_cells, n_mem_size_cells; > static int form1_affinity; > > #define MAX_DISTANCE_REF_POINTS 4 > -static int distance_ref_points_depth; > +static int max_associativity_domain_index; > static const __be32 *distance_ref_points; > static int distance_lookup_table[MAX_NUMNODES][MAX_DISTANCE_REF_POINTS]; > > @@ -169,7 +169,7 @@ int cpu_distance(__be32 *cpu1_assoc, __be32 *cpu2_assoc) > > int i, index; > > - for (i = 0; i < distance_ref_points_depth; i++) { > + for (i = 0; i < max_associativity_domain_index; i++) { > index = be32_to_cpu(distance_ref_points[i]); > if (cpu1_assoc[index] == cpu2_assoc[index]) > break; > @@ -193,7 +193,7 @@ int __node_distance(int a, int b) > if (!form1_affinity) > return ((a == b) ? LOCAL_DISTANCE : REMOTE_DISTANCE); > > - for (i = 0; i < distance_ref_points_depth; i++) { > + for (i = 0; i < max_associativity_domain_index; i++) { > if (distance_lookup_table[a][i] == distance_lookup_table[b][i]) > break; > > @@ -213,7 +213,7 @@ static void initialize_distance_lookup_table(int nid, > if (!form1_affinity) > return; > > - for (i = 0; i < distance_ref_points_depth; i++) { > + for (i = 0; i < max_associativity_domain_index; i++) { > const __be32 *entry; > > entry = &associativity[be32_to_cpu(distance_ref_points[i]) - 1]; > @@ -240,7 +240,7 @@ static int associativity_to_nid(const __be32 *associativity) > nid = NUMA_NO_NODE; > > if (nid > 0 && > - of_read_number(associativity, 1) >= distance_ref_points_depth) { > + of_read_number(associativity, 1) >= max_associativity_domain_index) { > /* > * Skip the length field and send start of associativity array > */ > @@ -310,14 +310,14 @@ static int __init find_primary_domain_index(void) > */ > distance_ref_points = of_get_property(root, > "ibm,associativity-reference-points", > - &distance_ref_points_depth); > + &max_associativity_domain_index); > > if (!distance_ref_points) { > dbg("NUMA: ibm,associativity-reference-points not found.\n"); > goto err; > } > > - distance_ref_points_depth /= sizeof(int); > + max_associativity_domain_index /= sizeof(int); > > if (firmware_has_feature(FW_FEATURE_OPAL) || > firmware_has_feature(FW_FEATURE_TYPE1_AFFINITY)) { > @@ -328,7 +328,7 @@ static int __init find_primary_domain_index(void) > if (form1_affinity) { > index = of_read_number(distance_ref_points, 1); > } else { > - if (distance_ref_points_depth < 2) { > + if (max_associativity_domain_index < 2) { > printk(KERN_WARNING "NUMA: " > "short ibm,associativity-reference-points\n"); > goto err; > @@ -341,10 +341,10 @@ static int __init find_primary_domain_index(void) > * Warn and cap if the hardware supports more than > * MAX_DISTANCE_REF_POINTS domains. > */ > - if (distance_ref_points_depth > MAX_DISTANCE_REF_POINTS) { > + if (max_associativity_domain_index > MAX_DISTANCE_REF_POINTS) { > printk(KERN_WARNING "NUMA: distance array capped at " > "%d entries\n", MAX_DISTANCE_REF_POINTS); > - distance_ref_points_depth = MAX_DISTANCE_REF_POINTS; > + max_associativity_domain_index = MAX_DISTANCE_REF_POINTS; > } > > of_node_put(root);
diff --git a/arch/powerpc/mm/numa.c b/arch/powerpc/mm/numa.c index 8365b298ec48..132813dd1a6c 100644 --- a/arch/powerpc/mm/numa.c +++ b/arch/powerpc/mm/numa.c @@ -56,7 +56,7 @@ static int n_mem_addr_cells, n_mem_size_cells; static int form1_affinity; #define MAX_DISTANCE_REF_POINTS 4 -static int distance_ref_points_depth; +static int max_associativity_domain_index; static const __be32 *distance_ref_points; static int distance_lookup_table[MAX_NUMNODES][MAX_DISTANCE_REF_POINTS]; @@ -169,7 +169,7 @@ int cpu_distance(__be32 *cpu1_assoc, __be32 *cpu2_assoc) int i, index; - for (i = 0; i < distance_ref_points_depth; i++) { + for (i = 0; i < max_associativity_domain_index; i++) { index = be32_to_cpu(distance_ref_points[i]); if (cpu1_assoc[index] == cpu2_assoc[index]) break; @@ -193,7 +193,7 @@ int __node_distance(int a, int b) if (!form1_affinity) return ((a == b) ? LOCAL_DISTANCE : REMOTE_DISTANCE); - for (i = 0; i < distance_ref_points_depth; i++) { + for (i = 0; i < max_associativity_domain_index; i++) { if (distance_lookup_table[a][i] == distance_lookup_table[b][i]) break; @@ -213,7 +213,7 @@ static void initialize_distance_lookup_table(int nid, if (!form1_affinity) return; - for (i = 0; i < distance_ref_points_depth; i++) { + for (i = 0; i < max_associativity_domain_index; i++) { const __be32 *entry; entry = &associativity[be32_to_cpu(distance_ref_points[i]) - 1]; @@ -240,7 +240,7 @@ static int associativity_to_nid(const __be32 *associativity) nid = NUMA_NO_NODE; if (nid > 0 && - of_read_number(associativity, 1) >= distance_ref_points_depth) { + of_read_number(associativity, 1) >= max_associativity_domain_index) { /* * Skip the length field and send start of associativity array */ @@ -310,14 +310,14 @@ static int __init find_primary_domain_index(void) */ distance_ref_points = of_get_property(root, "ibm,associativity-reference-points", - &distance_ref_points_depth); + &max_associativity_domain_index); if (!distance_ref_points) { dbg("NUMA: ibm,associativity-reference-points not found.\n"); goto err; } - distance_ref_points_depth /= sizeof(int); + max_associativity_domain_index /= sizeof(int); if (firmware_has_feature(FW_FEATURE_OPAL) || firmware_has_feature(FW_FEATURE_TYPE1_AFFINITY)) { @@ -328,7 +328,7 @@ static int __init find_primary_domain_index(void) if (form1_affinity) { index = of_read_number(distance_ref_points, 1); } else { - if (distance_ref_points_depth < 2) { + if (max_associativity_domain_index < 2) { printk(KERN_WARNING "NUMA: " "short ibm,associativity-reference-points\n"); goto err; @@ -341,10 +341,10 @@ static int __init find_primary_domain_index(void) * Warn and cap if the hardware supports more than * MAX_DISTANCE_REF_POINTS domains. */ - if (distance_ref_points_depth > MAX_DISTANCE_REF_POINTS) { + if (max_associativity_domain_index > MAX_DISTANCE_REF_POINTS) { printk(KERN_WARNING "NUMA: distance array capped at " "%d entries\n", MAX_DISTANCE_REF_POINTS); - distance_ref_points_depth = MAX_DISTANCE_REF_POINTS; + max_associativity_domain_index = MAX_DISTANCE_REF_POINTS; } of_node_put(root);
No functional change in this patch Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@linux.ibm.com> --- arch/powerpc/mm/numa.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-)