Messages in this thread | | | Date | Sat, 4 Feb 2023 11:36:43 +0100 | From | Peter Zijlstra <> | Subject | Re: [PATCH v4 7/8] perf: Add perf_event_attr::config3 |
| |
On Mon, Jan 09, 2023 at 01:26:23PM -0600, 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 <james.clark@arm.com> > Signed-off-by: Rob Herring <robh@kernel.org> > --- > There's still an unresolved discussion about validating 'config3' with > the options laid out here[1]. > > v4: > - Rebase on v6.2-rc1 > v3: > - No change > v2: > - Drop tools/ side update > > [1] https://lore.kernel.org/all/Y49ttrv6W5k3ZNYw@FVFF77S0Q05N.cambridge.arm.com/ > --- > include/uapi/linux/perf_event.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/include/uapi/linux/perf_event.h b/include/uapi/linux/perf_event.h > index ccb7f5dad59b..37675437b768 100644 > --- a/include/uapi/linux/perf_event.h > +++ b/include/uapi/linux/perf_event.h > @@ -374,6 +374,7 @@ enum perf_event_read_format { > #define PERF_ATTR_SIZE_VER5 112 /* add: aux_watermark */ > #define PERF_ATTR_SIZE_VER6 120 /* add: aux_sample_size */ > #define PERF_ATTR_SIZE_VER7 128 /* add: sig_data */ > +#define PERF_ATTR_SIZE_VER8 136 /* add: config3 */ > > /* > * Hardware event_id to monitor via a performance monitoring event: > @@ -515,6 +516,8 @@ struct perf_event_attr { > * truncated accordingly on 32 bit architectures. > */ > __u64 sig_data; > + > + __u64 config3; /* extension of config2 */ > };
Yeah, that was bound to happen I suppose..
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
| |