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

Swapnil Sapkal posted 6 patches 1 week, 3 days ago
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(-)
[PATCH v1 0/6] perf sched stats: Fixes and improvements
Posted by Swapnil Sapkal 1 week, 3 days ago
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(-)

-- 
2.43.0
Re: [PATCH v1 0/6] perf sched stats: Fixes and improvements
Posted by Shrikanth Hegde 1 week, 3 days ago

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>