Messages in this thread | | | From | Rob Herring <> | Date | Fri, 14 Oct 2022 17:05:36 -0500 | Subject | Re: [PATCH v4 2/3] perf tools: Sync perf_event_attr::config3 addition |
| |
On Fri, Oct 14, 2022 at 4:51 PM Arnaldo Carvalho de Melo <acme@kernel.org> wrote: > > Em Tue, Oct 04, 2022 at 02:12:36PM -0500, Rob Herring escreveu: > > 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. > > > > Signed-off-by: Rob Herring <robh@kernel.org> > > --- > > This patch is dependent on the kernel side landing first. > > --- > > include/uapi/linux/perf_event.h | 3 +++ > > tools/include/uapi/linux/perf_event.h | 3 +++ > > 2 files changed, 6 insertions(+) > > > > diff --git a/include/uapi/linux/perf_event.h b/include/uapi/linux/perf_event.h > > index 03b370062741..b53f9b958235 100644 > > --- a/include/uapi/linux/perf_event.h > > +++ b/include/uapi/linux/perf_event.h > > @@ -333,6 +333,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: > > @@ -474,6 +475,8 @@ struct perf_event_attr { > > * truncated accordingly on 32 bit architectures. > > */ > > __u64 sig_data; > > + > > + __u64 config3; /* extension of config2 */ > > }; > > > > /* > > > I'm removing the above part, as this is for the kernel, not for tooling.
Oops, yes.
> > Was this part submitted upstream? I couldn't find, so far, where it was > submitted, can you please clarify?
I have to respin it after rc1. I'm waiting on patch 1 to be accepted first.
You should only take patch 1 (for 6.1/stable) as I've noted in the patches. I sent the whole thing because Leo asked to see the whole thing...
Rob
| |