Re: [PATCH V7 00/15] perf/x86: Add perf text poke events

[Arnaldo Carvalho de Melo <acme@kernel.org>]

Em Wed, Jul 08, 2020 at 09:34:23AM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Wed, Jul 08, 2020 at 02:55:36PM +0300, Adrian Hunter escreveu:
> > On 27/05/20 6:54 pm, Peter Zijlstra wrote:
> > > On Wed, May 27, 2020 at 12:47:16PM -0300, Arnaldo Carvalho de Melo wrote:
> > >> Em Wed, May 20, 2020 at 05:56:34PM +0200, Peter Zijlstra escreveu:
> > >>> On Tue, May 19, 2020 at 10:40:01PM -0300, Arnaldo Carvalho de Melo wrote:
> > >>>> PeterZ, from what we discussed for the next merge Window, perhaps we
> > >>>> should route the kernel bits via the tip tree while I will push the
> > >>>> tooling bits on my 5.8 merge request to Linus, Ok?
> > >>>
> > >>> Sure, I can take the kernel bits. Thanks!
> > >>
> > >> So, I'm going thru the userpace bits and stopped at the patch copying
> > >> include/uapi/linux/perf_events.h to
> > >> tools/include/uapi/linux/perf_events.h to get the text_poke
> > >> perf_event_attr bit, looked at tip/ and this hasn't landed there yet,
> > >> any issues with the kernel bits?
> > > 
> > > Nah, I was waiting on the 0day robot, but its broken *again* :-(
> > > 
> > > Let me go move things along manually.
> > > 
> > 
> > Hi
> > 
> > The kernel patches are in tip and the remaining tools patches still apply.
> > Remaining patches 11-15 are:
> > 
> >       perf tools: Add support for PERF_RECORD_TEXT_POKE
> >       perf tools: Add support for PERF_RECORD_KSYMBOL_TYPE_OOL
> >       perf intel-pt: Add support for text poke events
> >       perf script: Add option --show-text-poke-events
> >       perf script: Show text poke address symbol
> 
> Sure, I'll merge those now that the kernel support is in tip, thanks for
> the reminder,

Done, now I need to build a kernel from tip/perf/core and test with the
tooling from acme/perf/core, will do later today.

So far what I have is in my repo at the tmp.perf/core branch, if anyone
wants to test before I do test it with the kernel with these features
and the usual container build tests.

- Arnaldo