Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: NcMMjWyFBa3WGzXhLn4grdqtVJ8y/SBKqsZRfXUTd4kU9bA0JRDCGFnjCjb8uUFNByHr3ExTQ5
 N3rbpyw2rfDg==
IronPort-SDR: kjwZxmv0D1N0fy9kwt36P/CB7RZKzYFuBWM/a16aaZc/vkQ3CGOG1LoKM5gQZULMUH+27ki473
 ITz0ObCTJiyg==
Subject: Re: [PATCH v4 0/2] perf header: Support HYBRID_TOPOLOGY and
 HYBRID_CPU_PMU_CAPS
To:     Jiri Olsa <jolsa@redhat.com>
Cc:     acme@kernel.org, jolsa@kernel.org, peterz@infradead.org,
        mingo@redhat.com, alexander.shishkin@linux.intel.com,
        Linux-kernel@vger.kernel.org, ak@linux.intel.com,
        kan.liang@intel.com, yao.jin@intel.com
References: <20210514122948.9472-1-yao.jin@linux.intel.com>
 <YKA6oqfP0niinZkD@krava>
From:   "Jin, Yao" <yao.jin@linux.intel.com>
Message-ID: <228cc228-464d-c4dd-039c-535f050704d5@linux.intel.com>
Date:   Tue, 18 May 2021 09:40:41 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.10.1
MIME-Version: 1.0
In-Reply-To: <YKA6oqfP0niinZkD@krava>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

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@redhat.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