Skip to content

Commit c7ff8ff

Browse files
zhang-ruispandruvada
authored andcommitted
tools/power/x86/intel-speed-select: Remove cpu mask display for non-cpu power domain
Non CPU power domains will not have any CPUs. So don't display any CPU count or enable mask. Signed-off-by: Zhang Rui <[email protected]> [[email protected]: subject and changelog edits] Signed-off-by: Srinivas Pandruvada <[email protected]>
1 parent c5a295c commit c7ff8ff

File tree

1 file changed

+23
-21
lines changed

1 file changed

+23
-21
lines changed

tools/power/x86/intel-speed-select/isst-display.c

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -356,31 +356,33 @@ void isst_ctdp_display_information(struct isst_id *id, FILE *outf, int tdp_level
356356
ctdp_level->level);
357357
format_and_print(outf, level + 1, header, NULL);
358358

359-
snprintf(header, sizeof(header), "cpu-count");
360-
j = get_cpu_count(id);
361-
snprintf(value, sizeof(value), "%d", j);
362-
format_and_print(outf, level + 2, header, value);
363-
364-
j = CPU_COUNT_S(ctdp_level->core_cpumask_size,
365-
ctdp_level->core_cpumask);
366-
if (j) {
367-
snprintf(header, sizeof(header), "enable-cpu-count");
359+
if (id->cpu >= 0) {
360+
snprintf(header, sizeof(header), "cpu-count");
361+
j = get_cpu_count(id);
368362
snprintf(value, sizeof(value), "%d", j);
369363
format_and_print(outf, level + 2, header, value);
370-
}
371364

372-
if (ctdp_level->core_cpumask_size) {
373-
snprintf(header, sizeof(header), "enable-cpu-mask");
374-
printcpumask(sizeof(value), value,
375-
ctdp_level->core_cpumask_size,
376-
ctdp_level->core_cpumask);
377-
format_and_print(outf, level + 2, header, value);
365+
j = CPU_COUNT_S(ctdp_level->core_cpumask_size,
366+
ctdp_level->core_cpumask);
367+
if (j) {
368+
snprintf(header, sizeof(header), "enable-cpu-count");
369+
snprintf(value, sizeof(value), "%d", j);
370+
format_and_print(outf, level + 2, header, value);
371+
}
378372

379-
snprintf(header, sizeof(header), "enable-cpu-list");
380-
printcpulist(sizeof(value), value,
381-
ctdp_level->core_cpumask_size,
382-
ctdp_level->core_cpumask);
383-
format_and_print(outf, level + 2, header, value);
373+
if (ctdp_level->core_cpumask_size) {
374+
snprintf(header, sizeof(header), "enable-cpu-mask");
375+
printcpumask(sizeof(value), value,
376+
ctdp_level->core_cpumask_size,
377+
ctdp_level->core_cpumask);
378+
format_and_print(outf, level + 2, header, value);
379+
380+
snprintf(header, sizeof(header), "enable-cpu-list");
381+
printcpulist(sizeof(value), value,
382+
ctdp_level->core_cpumask_size,
383+
ctdp_level->core_cpumask);
384+
format_and_print(outf, level + 2, header, value);
385+
}
384386
}
385387

386388
snprintf(header, sizeof(header), "thermal-design-power-ratio");

0 commit comments

Comments
 (0)