Skip to content

Commit 21816bb

Browse files
kaushlengregkh
authored andcommitted
platform/x86: intel_telemetry: Fix swapped arrays in PSS output
commit 25e9e32 upstream. 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: 87bee29 ("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> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent b5a0229 commit 21816bb

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • drivers/platform/x86/intel/telemetry

drivers/platform/x86/intel/telemetry/debugfs.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -449,7 +449,7 @@ static int telem_pss_states_show(struct seq_file *s, void *unused)
449449
for (index = 0; index < debugfs_conf->pss_ltr_evts; index++) {
450450
seq_printf(s, "%-32s\t%u\n",
451451
debugfs_conf->pss_ltr_data[index].name,
452-
pss_s0ix_wakeup[index]);
452+
pss_ltr_blkd[index]);
453453
}
454454

455455
seq_puts(s, "\n--------------------------------------\n");
@@ -459,7 +459,7 @@ static int telem_pss_states_show(struct seq_file *s, void *unused)
459459
for (index = 0; index < debugfs_conf->pss_wakeup_evts; index++) {
460460
seq_printf(s, "%-32s\t%u\n",
461461
debugfs_conf->pss_wakeup[index].name,
462-
pss_ltr_blkd[index]);
462+
pss_s0ix_wakeup[index]);
463463
}
464464

465465
return 0;

0 commit comments

Comments
 (0)