Re: [PATCH v4 0/2] perf header: Support HYBRID_TOPOLOGY and HYBRID_CPU_PMU_CAPS

From: Jin, Yao
Date: Mon May 17 2021 - 21:40:49 EST

Hi Jiri,

On 5/16/2021 5:18 AM, Jiri Olsa wrote:
On Fri, May 14, 2021 at 08:29:46PM +0800, Jin Yao wrote:
AlderLake uses a hybrid architecture utilizing Golden Cove cores
(core cpu) and Gracemont cores (atom cpu). It would be useful to let user
know the hybrid topology, the HYBRID_TOPOLOGY feature in header indicates
which cpus are core cpus, and which cpus are atom cpus.

On hybrid platform, it may have several cpu pmus, such as, "cpu_core" and
"cpu_atom". The HYBRID_CPU_PMU_CAPS feature in perf header is created to
support multiple cpu pmus.

- Only minor update in '[PATCH v4 2/2] perf header: Support HYBRID_CPU_PMU_CAPS feature'.
1. Directly return process_per_cpu_pmu_caps() in process_cpu_pmu_caps.
Remove the variable 'ret'.
2. Set 'ret = -1' if (!n->pmu_name) in process_hybrid_cpu_pmu_caps().
Acked-by: Jiri Olsa<jolsa@xxxxxxxxxx>


Thanks so much!

I have some followup patches which depend on this patchset. I will post them after this patchset is applied.

Jin Yao