Re: [PATCH v1 0/6] perf sched stats: Fixes and improvements

[Shrikanth Hegde <sshegde@linux.ibm.com>]

On 1/28/26 12:19 AM, Swapnil Sapkal wrote:
> These patches contain fixes and improvements related to perf sched stats
> code. The series includes:
> 
> * Bug fixes
>    - Add NULL check for cd_map in perf sched stats to prevent potential
>      NULL pointer dereference
>    - Fix NULL check in cpumask_to_cpulist() in perf util
> 
> * Code quality improvements:
>    - Replace hardcoded max cpus value with MAX_NR_CPUS constant in perf
>      header
>    - Define SEP_LEN macro to replace magic number in perf sched stats
>    - Correct spelling of function name in perf sched stats
> 
> * Documentation:
>    - Fix various issues in the perf sched stats man page
> 
> Swapnil Sapkal (6):
>    perf header: Replace hardcoded max cpus by MAX_NR_CPUS
>    perf util: Fix NULL check in cpumask_to_cpulist()
>    perf sched stats: Add NULL check for cd_map
>    perf sched stats: correct spelling of function name
>    perf sched stats: Define macro for SEP_LEN
>    perf sched stats: Fixes in man page
> 
>   tools/perf/Documentation/perf-sched.txt |  8 +++----
>   tools/perf/builtin-sched.c              | 30 +++++++++++++++----------
>   tools/perf/util/header.c                |  5 +++--
>   tools/perf/util/util.c                  |  2 +-
>   4 files changed, 26 insertions(+), 19 deletions(-)
> 

Other than the comment in Patch 1/6, rest looks good to me.

For the series:
Reviewed-by: Shrikanth Hegde <sshegde@linux.ibm.com>