]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
platform/x86: intel_telemetry: Fix swapped arrays in PSS output
authorKaushlendra Kumar <kaushlendra.kumar@intel.com>
Wed, 24 Dec 2025 03:20:53 +0000 (08:50 +0530)
committerIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Mon, 26 Jan 2026 14:42:25 +0000 (16:42 +0200)
The LTR blocking statistics and wakeup event counters are incorrectly
cross-referenced during debugfs output rendering. The code populates
pss_ltr_blkd[] with LTR blocking data and pss_s0ix_wakeup[] with wakeup
data, but the display loops reference the wrong arrays.

This causes the "LTR Blocking Status" section to print wakeup events
and the "Wakes Status" section to print LTR blockers, misleading power
management analysis and S0ix residency debugging.

Fix by aligning array usage with the intended output section labels.

Fixes: 87bee290998d ("platform:x86: Add Intel Telemetry Debugfs interfaces")
Cc: stable@vger.kernel.org
Signed-off-by: Kaushlendra Kumar <kaushlendra.kumar@intel.com>
Link: https://patch.msgid.link/20251224032053.3915900-1-kaushlendra.kumar@intel.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/platform/x86/intel/telemetry/debugfs.c

index 70e5736c44c71ac32a7dd3afa37668b4554fa4c0..189c61ff7ff0c06b9c5e3533b877885e816ce324 100644 (file)
@@ -449,7 +449,7 @@ static int telem_pss_states_show(struct seq_file *s, void *unused)
        for (index = 0; index < debugfs_conf->pss_ltr_evts; index++) {
                seq_printf(s, "%-32s\t%u\n",
                           debugfs_conf->pss_ltr_data[index].name,
-                          pss_s0ix_wakeup[index]);
+                          pss_ltr_blkd[index]);
        }
 
        seq_puts(s, "\n--------------------------------------\n");
@@ -459,7 +459,7 @@ static int telem_pss_states_show(struct seq_file *s, void *unused)
        for (index = 0; index < debugfs_conf->pss_wakeup_evts; index++) {
                seq_printf(s, "%-32s\t%u\n",
                           debugfs_conf->pss_wakeup[index].name,
-                          pss_ltr_blkd[index]);
+                          pss_s0ix_wakeup[index]);
        }
 
        return 0;