linux-kernel - Re: [PATCH v1] perf arm: Workaround ARM PMUs cpu maps having offline cpus

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: <cc1b0673-13b3-439f-afdf-c9cb450f8fed@arm.com>
Date: Wed, 12 Jun 2024 21:32:53 +0100
From: Leo Yan <leo.yan@....com>
To: Ian Rogers <irogers@...gle.com>
Cc: Suzuki K Poulose <suzuki.poulose@....com>,
 Mike Leach <mike.leach@...aro.org>, James Clark <james.clark@....com>,
 John Garry <john.g.garry@...cle.com>, Will Deacon <will@...nel.org>,
 Leo Yan <leo.yan@...ux.dev>, Peter Zijlstra <peterz@...radead.org>,
 Ingo Molnar <mingo@...hat.com>, Arnaldo Carvalho de Melo <acme@...nel.org>,
 Namhyung Kim <namhyung@...nel.org>, Mark Rutland <mark.rutland@....com>,
 Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
 Jiri Olsa <jolsa@...nel.org>, Adrian Hunter <adrian.hunter@...el.com>,
 Kan Liang <kan.liang@...ux.intel.com>, coresight@...ts.linaro.org,
 linux-arm-kernel@...ts.infradead.org, linux-perf-users@...r.kernel.org,
 linux-kernel@...r.kernel.org, Yicong Yang <yangyicong@...wei.com>
Subject: Re: [PATCH v1] perf arm: Workaround ARM PMUs cpu maps having offline
 cpus


Hi Ian,

On 6/10/24 18:33, Ian Rogers wrote:

[...]

>> Just a nitpick and I think it is not an issue caused by this patch.
>> After hotplug off one CPU and then if specify the CPU with option '-C',
>> the 'perf stat' command still continues to run. This is inconsistent
>> with the 'perf record' which exits with failures immediately.
>>
>> Maybe consider to send an extra patch to address this issue?
> 
> As you say, this doesn't relate to the problem fixed here. I don't
> have a problem with the command line behavior change but I think my
> getting shouted at budget is fairly well exhausted.

I understand you put much efforts on fixing issues, on Arm platforms and
other platforms. This is also why I want to contribute a bit for testing
the patches.

> How about we trade issues, if the following get fixed:
> 
> Renaming of the cycles event in arm_dsu_pmu.c - I'd say this is a top
> priority issue right now.

I cannot promise this. The main reason is that I still believe the
'cycles' event (or, generally speaking, all events) should be managed by
the tool rather than by the uncore PMU drivers. Additionally, the perf
tool currently has handled these symbolic events effectively.

> Fixing ARM hybrid default perf stat (it is crazy we can accept this
> being broken), opening events on both BIG.little PMUs as done here.

Yeah, James is working on this.

> I'll address this.

Anyway, I appreciate your work.

Thanks,
Leo