On Mon, Jul 02, 2018 at 04:33:28PM -0600, Mathieu Poirier wrote: > It can be advantagous to have access to all the information conveyed by > a perf_event when setting up the AUX buffer, as it is the case when > dealing with PMU specific driver configuration communicated to the kernel > using an ioctl() call. > > As such simply replace the cpu information by the complete perf_event > structure and change all affected customers. > > Signed-off-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx> > --- > arch/s390/kernel/perf_cpum_sf.c | 4 ++-- > arch/x86/events/intel/bts.c | 4 +++- > arch/x86/events/intel/pt.c | 5 +++-- > drivers/hwtracing/coresight/coresight-etm-perf.c | 6 +++--- > drivers/perf/arm_spe_pmu.c | 6 +++--- > include/linux/perf_event.h | 2 +- > kernel/events/ring_buffer.c | 2 +- > 7 files changed, 16 insertions(+), 13 deletions(-) > > diff --git a/arch/s390/kernel/perf_cpum_sf.c b/arch/s390/kernel/perf_cpum_sf.c > index 0292d68e7dde..e06daaa08894 100644 > --- a/arch/s390/kernel/perf_cpum_sf.c > +++ b/arch/s390/kernel/perf_cpum_sf.c > @@ -1601,8 +1601,8 @@ static void aux_buffer_free(void *data) > * > * Return the private AUX buffer structure if success or NULL if fails. > */ > -static void *aux_buffer_setup(int cpu, void **pages, int nr_pages, > - bool snapshot) > +static void *aux_buffer_setup(struct perf_event *event, void **pages, > + int nr_pages, bool snapshot); Please remove the trailing semi-colon (;) in the function definition causing the kbuild error. Also, it would be great if you also could update the function comment and replace the @cpu by the @event. Many thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
