Re: [PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-12-07 Thread Rob Herring
On Tue, Dec 6, 2022 at 10:28 AM Mark Rutland wrote: > > Peter, it looks like this series is blocked on the below now; what would you > prefer out of: > > (a) Take this as is, and look add adding additional validation on top. > > (b) Add some flag to indicate a PMU driver supports config3, and

Re: [PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-12-06 Thread Mark Rutland
Peter, it looks like this series is blocked on the below now; what would you prefer out of: (a) Take this as is, and look add adding additional validation on top. (b) Add some flag to indicate a PMU driver supports config3, and have the core code check that, but leave the existing fields

Re: [PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-11-29 Thread Alexander Shishkin
Rob Herring writes: > On Fri, Nov 18, 2022 at 10:49 AM Will Deacon wrote: >> >> On Fri, Nov 04, 2022 at 10:55:07AM -0500, Rob Herring wrote: >> > @@ -515,6 +516,8 @@ struct perf_event_attr { >> >* truncated accordingly on 32 bit architectures. >> >*/ >> > __u64 sig_data;

Re: [PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-11-28 Thread Rob Herring
On Mon, Nov 28, 2022 at 10:36 AM Alexander Shishkin wrote: > > Rob Herring writes: > > > On Fri, Nov 18, 2022 at 10:49 AM Will Deacon wrote: > >> > >> On Fri, Nov 04, 2022 at 10:55:07AM -0500, Rob Herring wrote: > >> > @@ -515,6 +516,8 @@ struct perf_event_attr { > >> >* truncated

Re: [PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-11-28 Thread Rob Herring
On Fri, Nov 18, 2022 at 10:49 AM Will Deacon wrote: > > On Fri, Nov 04, 2022 at 10:55:07AM -0500, Rob Herring wrote: > > Arm SPEv1.2 adds another 64-bits of event filtering control. As the > > existing perf_event_attr::configN fields are all used up for SPE PMU, an > > additional field is needed.

Re: [PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-11-18 Thread Will Deacon
On Fri, Nov 04, 2022 at 10:55:07AM -0500, Rob Herring wrote: > Arm SPEv1.2 adds another 64-bits of event filtering control. As the > existing perf_event_attr::configN fields are all used up for SPE PMU, an > additional field is needed. Add a new 'config3' field. > > Tested-by: James Clark >

[PATCH v3 7/8] perf: Add perf_event_attr::config3

2022-11-04 Thread Rob Herring
Arm SPEv1.2 adds another 64-bits of event filtering control. As the existing perf_event_attr::configN fields are all used up for SPE PMU, an additional field is needed. Add a new 'config3' field. Tested-by: James Clark Signed-off-by: Rob Herring --- v3: - No change v2: - Drop tools/ side