Commit 6b401a5b authored by Kaushlendra Kumar's avatar Kaushlendra Kumar Committed by Shuah Khan
Browse files

cpupower: idle_monitor: fix incorrect value logged after stop 

The cpuidle sysfs monitor printed the previous sample’s counter
value in cpuidle_stop() instead of the freshly read one. The dprint
line used previous_count[cpu][state] while current_count[cpu][state]
had just been populated. This caused misleading debug output.

Switch the logging to current_count so the post-interval snapshot
matches the displayed value.

Link: https://lore.kernel.org/r/20251202044012.3844790-1-kaushlendra.kumar@intel.com


Signed-off-by: default avatarKaushlendra Kumar <kaushlendra.kumar@intel.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent 8f0b4cce

tools/power/cpupower/utils/idle_monitor/cpuidle_sysfs.c

+1 −1
Original line number Diff line number Diff line
@@ -70,7 +70,7 @@ static int cpuidle_stop(void) 
			current_count[cpu][state] =

				cpuidle_state_time(cpu, state);

			dprint("CPU %d - State: %d - Val: %llu\n",

			       cpu, state, previous_count[cpu][state]);

			       cpu, state, current_count[cpu][state]);

		}

	}

	return 0;