Unverified Commit 25e9e322 authored by Kaushlendra Kumar's avatar Kaushlendra Kumar Committed by Ilpo Järvinen
Browse files

platform/x86: intel_telemetry: Fix swapped arrays in PSS output 



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: 87bee290 ("platform:x86: Add Intel Telemetry Debugfs interfaces")
Cc: stable@vger.kernel.org
Signed-off-by: default avatarKaushlendra Kumar <kaushlendra.kumar@intel.com>
Link: https://patch.msgid.link/20251224032053.3915900-1-kaushlendra.kumar@intel.com


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
parent 98bdc485

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

+2 −2
Original line number Diff line number Diff line
@@ -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;