Re: [PATCH v2 1/3] libperf: Add perf_cpu_map__default_new()

[Jin, Yao]

Hi Arnaldo,

Any comments for v2 series of cpulist hybrid support? :)

Thanks
Jin Yao

On 6/21/2021 10:33 PM, Jin Yao wrote:
> The libperf already has a static function 'cpu_map__default_new()'.
> Add a new API perf_cpu_map__default_new() to export the function.
>
> Signed-off-by: Jin Yao <yao.jin@xxxxxxxxxxxxxxx>
> ---
>   tools/lib/perf/cpumap.c              | 5 +++++
>   tools/lib/perf/include/perf/cpumap.h | 1 +
>   2 files changed, 6 insertions(+)
>
> diff --git a/tools/lib/perf/cpumap.c b/tools/lib/perf/cpumap.c
> index ca0215047c32..51b6553912e0 100644
> --- a/tools/lib/perf/cpumap.c
> +++ b/tools/lib/perf/cpumap.c
> @@ -68,6 +68,11 @@ static struct perf_cpu_map *cpu_map__default_new(void)
>   	return cpus;
>   }
>   
> +struct perf_cpu_map *perf_cpu_map__default_new(void)
> +{
> +	return cpu_map__default_new();
> +}
> +
>   static int cmp_int(const void *a, const void *b)
>   {
>   	return *(const int *)a - *(const int*)b;
> diff --git a/tools/lib/perf/include/perf/cpumap.h b/tools/lib/perf/include/perf/cpumap.h
> index 6a17ad730cbc..7c27766ea0bf 100644
> --- a/tools/lib/perf/include/perf/cpumap.h
> +++ b/tools/lib/perf/include/perf/cpumap.h
> @@ -9,6 +9,7 @@
>   struct perf_cpu_map;
>   
>   LIBPERF_API struct perf_cpu_map *perf_cpu_map__dummy_new(void);
> +LIBPERF_API struct perf_cpu_map *perf_cpu_map__default_new(void);
>   LIBPERF_API struct perf_cpu_map *perf_cpu_map__new(const char *cpu_list);
>   LIBPERF_API struct perf_cpu_map *perf_cpu_map__read(FILE *file);
>   LIBPERF_API struct perf_cpu_map *perf_cpu_map__get(struct perf_cpu_map *map);