The purpose of this patch series is to replace MAX_NR_CPUS with a dynamic value throughout perf wherever possible using nr_cpus_online, the number of CPUs online during a record session, and cpu__max_cpu, the possible number of CPUs as defined in the sysfs. MAX_NR_CPUS is still used by DECLARE_BITMAP at compile time, however, it's replaced elsewhere.
This patch series was tested using "perf record -a -g" on both an eight socket (288 CPU) system and a single socket (36 CPU) system. Each system was then rebooted single socket and eight socket before "perf report" was used to read the perf.data out file. "perf report --header" was used to confirm that each perf.data file had information on the correct number of CPUs. Change since v1: Broke PATCH 2/2 into multiple patches. Kyle Meyer (6): perf: Refactor svg_build_topology_map perf/util/svghelper: Replace MAX_NR_CPUS with env->nr_cpus_online perf/util/stat: Replace MAX_NR_CPUS with cpu__max_cpu perf/util/session: Replace MAX_NR_CPUS with nr_cpus_online perf/util/machine: Replace MAX_NR_CPUS with nr_cpus_online perf/util/header: Replace MAX_NR_CPUS with cpu__max_cpu tools/perf/builtin-timechart.c | 5 +---- tools/perf/util/header.c | 7 +++--- tools/perf/util/machine.c | 12 +++++----- tools/perf/util/session.c | 6 ++--- tools/perf/util/stat.c | 4 ++-- tools/perf/util/svghelper.c | 50 ++++++++++++++++++++---------------------- tools/perf/util/svghelper.h | 4 +++- 7 files changed, 44 insertions(+), 44 deletions(-) Cc: Peter Zijlstra <[email protected]> Cc: Ingo Molnar <[email protected]> Cc: Arnaldo Carvalho de Melo <[email protected]> Cc: Alexander Shishkin <[email protected]> Cc: Jiri Olsa <[email protected]> Cc: Namhyung Kim <[email protected]> Cc: [email protected] Cc: Russ Anderson <[email protected]> Signed-off-by: Kyle Meyer <[email protected]> -- 2.12.3
