linux-kernel - Re: [PATCH] perf list: Skip the invalid hybrid 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] [thread-next>] [day] [month] [year] [list]

Message-ID: <f96534e5-4a5c-f6da-4574-16f64f291834@linux.intel.com>
Date:   Tue, 6 Jul 2021 10:27:53 +0800
From:   "Jin, Yao" <yao.jin@...ux.intel.com>
To:     Andi Kleen <ak@...ux.intel.com>, acme@...nel.org, jolsa@...nel.org,
        peterz@...radead.org, mingo@...hat.com,
        alexander.shishkin@...ux.intel.com
Cc:     Linux-kernel@...r.kernel.org, kan.liang@...el.com,
        yao.jin@...el.com
Subject: Re: [PATCH] perf list: Skip the invalid hybrid pmu

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