Re: [PATCH v2] tools/cpupower: Fix incorrect size in cpuidle_state_disable()

[Shuah Khan <skhan@linuxfoundation.org>]

On 9/16/25 23:08, Kaushlendra Kumar wrote:
> Fix incorrect size parameter passed to cpuidle_state_write_file() in
> cpuidle_state_disable().
> 
> The function was incorrectly using sizeof(disable) which returns the
> size of the unsigned int variable (4 bytes) instead of the actual
> length of the string stored in the 'value' buffer.
> 
> Since 'value' is populated with snprintf() to contain the string
> representation of the disable value, we should use the length
> returned by snprintf() to get the correct string length for
> writing to the sysfs file.
> 
> This ensures the correct number of bytes is written to the cpuidle
> state disable file in sysfs.
> 
> Signed-off-by: Kaushlendra Kumar <kaushlendra.kumar@intel.com>
> ---
> Changes in v2:
> - Use snprintf() return value instead of strlen() for better efficiency
> 

Thank you. Applied to git.kernel.org/pub/scm/linux/kernel/git/shuah/linux.git/log/?h=cpupower
for next PR to PM maintainer.

thanks,
-- Shuah