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;
Subject: Re: [PATCH] perf list: Skip the invalid hybrid pmu
To:     Andi Kleen <ak@linux.intel.com>, acme@kernel.org, jolsa@kernel.org,
        peterz@infradead.org, mingo@redhat.com,
        alexander.shishkin@linux.intel.com
Cc:     Linux-kernel@vger.kernel.org, kan.liang@intel.com,
        yao.jin@intel.com
References: <20210610051646.4003-1-yao.jin@linux.intel.com>
 <77380298-b3d5-d165-c8d4-a24b22d15b0c@linux.intel.com>
From:   "Jin, Yao" <yao.jin@linux.intel.com>
Message-ID: <f96534e5-4a5c-f6da-4574-16f64f291834@linux.intel.com>
Date:   Tue, 6 Jul 2021 10:27:53 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: <77380298-b3d5-d165-c8d4-a24b22d15b0c@linux.intel.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Arnaldo, hi Jiri,

On 6/17/2021 12:37 AM, Andi Kleen wrote:
> 
> On 6/9/2021 10:16 PM, Jin Yao wrote:
>> On hybrid platform, such as Alderlake, if atom CPUs are offlined,
>> the kernel still exports the sysfs path '/sys/devices/cpu_atom/' for
>> 'cpu_atom' pmu but the file '/sys/devices/cpu_atom/cpus' is empty,
>> which indicates this is an invalid pmu.
> 
> 
> The patch looks good, but it would probably make sense to fix the kernel to avoid this too.
> 
> Of course the tools patch would still be needed for older kernels.
> 
> -Andi
> 
> 

Any comments for this patch?

Thanks
Jin Yao