tools/power/x86/intel-speed-select: Remove cpu mask display for non-cpu power domain
authorZhang Rui <rui.zhang@intel.com>
Thu, 18 Aug 2022 13:26:23 +0000 (21:26 +0800)
committerSrinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Wed, 22 Mar 2023 20:36:55 +0000 (13:36 -0700)
Non CPU power domains will not have any CPUs. So don't display any CPU
count or enable mask.

Signed-off-by: Zhang Rui <rui.zhang@intel.com>
[srinivas.pandruvada@linux.intel.com: subject and changelog edits]
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
tools/power/x86/intel-speed-select/isst-display.c

index d8b789b..e357389 100644 (file)
@@ -356,31 +356,33 @@ void isst_ctdp_display_information(struct isst_id *id, FILE *outf, int tdp_level
                         ctdp_level->level);
                format_and_print(outf, level + 1, header, NULL);
 
-               snprintf(header, sizeof(header), "cpu-count");
-               j = get_cpu_count(id);
-               snprintf(value, sizeof(value), "%d", j);
-               format_and_print(outf, level + 2, header, value);
-
-               j = CPU_COUNT_S(ctdp_level->core_cpumask_size,
-                               ctdp_level->core_cpumask);
-               if (j) {
-                       snprintf(header, sizeof(header), "enable-cpu-count");
+               if (id->cpu >= 0) {
+                       snprintf(header, sizeof(header), "cpu-count");
+                       j = get_cpu_count(id);
                        snprintf(value, sizeof(value), "%d", j);
                        format_and_print(outf, level + 2, header, value);
-               }
 
-               if (ctdp_level->core_cpumask_size) {
-                       snprintf(header, sizeof(header), "enable-cpu-mask");
-                       printcpumask(sizeof(value), value,
-                                    ctdp_level->core_cpumask_size,
-                                    ctdp_level->core_cpumask);
-                       format_and_print(outf, level + 2, header, value);
+                       j = CPU_COUNT_S(ctdp_level->core_cpumask_size,
+                                       ctdp_level->core_cpumask);
+                       if (j) {
+                               snprintf(header, sizeof(header), "enable-cpu-count");
+                               snprintf(value, sizeof(value), "%d", j);
+                               format_and_print(outf, level + 2, header, value);
+                       }
 
-                       snprintf(header, sizeof(header), "enable-cpu-list");
-                       printcpulist(sizeof(value), value,
-                                    ctdp_level->core_cpumask_size,
-                                    ctdp_level->core_cpumask);
-                       format_and_print(outf, level + 2, header, value);
+                       if (ctdp_level->core_cpumask_size) {
+                               snprintf(header, sizeof(header), "enable-cpu-mask");
+                               printcpumask(sizeof(value), value,
+                                            ctdp_level->core_cpumask_size,
+                                            ctdp_level->core_cpumask);
+                               format_and_print(outf, level + 2, header, value);
+
+                               snprintf(header, sizeof(header), "enable-cpu-list");
+                               printcpulist(sizeof(value), value,
+                                            ctdp_level->core_cpumask_size,
+                                            ctdp_level->core_cpumask);
+                               format_and_print(outf, level + 2, header, value);
+                       }
                }
 
                snprintf(header, sizeof(header), "thermal-design-power-ratio");