linux-kernel - Re: [PATCH V2 0/2] perf parse-events: Do not break up AUX event group

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: <13640a9e-a186-2480-2649-ee19869228bb@intel.com>
Date:   Wed, 10 May 2023 20:38:55 +0300
From:   Adrian Hunter <adrian.hunter@...el.com>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     Jiri Olsa <jolsa@...nel.org>, Namhyung Kim <namhyung@...nel.org>,
        Ian Rogers <irogers@...gle.com>, linux-kernel@...r.kernel.org,
        linux-perf-users@...r.kernel.org
Subject: Re: [PATCH V2 0/2] perf parse-events: Do not break up AUX event group

On 10/05/23 20:13, Arnaldo Carvalho de Melo wrote:
> Em Mon, May 08, 2023 at 12:39:50PM +0300, Adrian Hunter escreveu:
>> Hi
>>
>> Here is a fix needed in v6.4, but dependent on Ian's patch
>> "perf evsel: Modify group pmu name for software events", refer:
>>
>> https://lore.kernel.org/lkml/20230502223851.2234828-20-irogers@google.com/
>>
>> There is also a test amendment to test the AUX event group fix for
>> Intel PT.
> 
> Ok, so I think I can add your Tested-by for that "perf evsel: Modify
> group pmu name for software events" patch, ok?

Yes, thank you!

> 
> - Arnaldo
>  
>>
>> Changes in V2:
>>
>>    Add an Intel PT test amendment for the AUX event group fix
>>    Change AUX event group fix as suggested by Ian
>>
>>
>> Adrian Hunter (2):
>>       perf test: test_intel_pt.sh: Test sample mode with named PMU
>>       perf parse-events: Do not break up AUX event group
>>
>>  tools/perf/tests/shell/test_intel_pt.sh | 7 +++++++
>>  tools/perf/util/evsel.c                 | 9 +++------
>>  2 files changed, 10 insertions(+), 6 deletions(-)