Skip to content

Commit

Permalink
tools/power turbostat: Remember global max_die_id
Browse files Browse the repository at this point in the history
This is necessary to gracefully handle sparse die_id's.

no functional change

Signed-off-by: Len Brown <[email protected]>
  • Loading branch information
lenb committed Apr 21, 2024
1 parent ae3326a commit cda2033
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tools/power/x86/turbostat/turbostat.c
Original file line number Diff line number Diff line change
Expand Up @@ -1415,6 +1415,7 @@ struct topo_params {
int allowed_cpus;
int allowed_cores;
int max_cpu_num;
int max_die_id;
int max_node_num;
int nodes_per_pkg;
int cores_per_node;
Expand Down Expand Up @@ -6980,7 +6981,6 @@ void topology_probe(bool startup)
int i;
int max_core_id = 0;
int max_package_id = 0;
int max_die_id = 0;
int max_siblings = 0;

/* Initialize num_cpus, max_cpu_num */
Expand Down Expand Up @@ -7097,8 +7097,8 @@ void topology_probe(bool startup)

/* get die information */
cpus[i].die_id = get_die_id(i);
if (cpus[i].die_id > max_die_id)
max_die_id = cpus[i].die_id;
if (cpus[i].die_id > topo.max_die_id)
topo.max_die_id = cpus[i].die_id;

/* get numa node information */
cpus[i].physical_node_id = get_physical_node_id(&cpus[i]);
Expand All @@ -7124,9 +7124,9 @@ void topology_probe(bool startup)
if (!summary_only && topo.cores_per_node > 1)
BIC_PRESENT(BIC_Core);

topo.num_die = max_die_id + 1;
topo.num_die = topo.max_die_id + 1;
if (debug > 1)
fprintf(outf, "max_die_id %d, sizing for %d die\n", max_die_id, topo.num_die);
fprintf(outf, "max_die_id %d, sizing for %d die\n", topo.max_die_id, topo.num_die);
if (!summary_only && topo.num_die > 1)
BIC_PRESENT(BIC_Die);

Expand Down

0 comments on commit cda2033

Please sign in to comment.