linux-kernel - Re: [PATCH v4 0/2] perf header: Support HYBRID_TOPOLOGY and HYBRID_CPU_PMU

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <228cc228-464d-c4dd-039c-535f050704d5@linux.intel.com>
Date:   Tue, 18 May 2021 09:40:41 +0800
From:   "Jin, Yao" <yao.jin@...ux.intel.com>
To:     Jiri Olsa <jolsa@...hat.com>
Cc:     acme@...nel.org, jolsa@...nel.org, peterz@...radead.org,
        mingo@...hat.com, alexander.shishkin@...ux.intel.com,
        Linux-kernel@...r.kernel.org, ak@...ux.intel.com,
        kan.liang@...el.com, yao.jin@...el.com
Subject: Re: [PATCH v4 0/2] perf header: Support HYBRID_TOPOLOGY and
 HYBRID_CPU_PMU_CAPS

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.
>>
>> v4:
>> ---
>> - 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@...hat.com>
> 
> thanks,
> jirka
> 

Thanks so much!

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

Thanks
Jin Yao