Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1693833

Re: [PATCH 0/5] perf script python: Provide perf_sample dict to all handlers

From Arnaldo Carvalho de Melo <acme@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH 0/5] perf script python: Provide perf_sample dict to all handlers
Date 2017-07-21 18:30 +0200
Message-ID <u5IYp-788-5@gated-at.bofh.it> (permalink)
References <u5vya-7aa-5@gated-at.bofh.it> <u5ARc-1Up-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Em Fri, Jul 21, 2017 at 09:46:39AM +0200, Jiri Olsa escreveu:
> On Thu, Jul 20, 2017 at 07:01:13PM -0700, Arun Kalyanasundaram wrote:
> > The process_event python hook receives a dict with most perf_sample
> > entries.
> 
> hi,
> was there any change to the rfc post?

I was wondering that, please stick a v1, v2, etc to the subject, and
state the changes across versions, to make things clear.

- Arnaldo
 
> thanks,
> jirka
> 
> > 
> > Other handlers (e.g. trace_unhandled, python_process_tracepoint) predate
> > the introduction of this dict and do not receive it. This patch series
> > adds the dict to all handlers, aiming to unify the information passed to
> > them.
> > 
> > This change adds an additional argument to the affected handlers. To
> > keep backwards compatibility (and avoid unnecessary work), do not pass
> > the aforementioned dict if the number of arguments signals that handler
> > version predates this change.
> > 
> > In addition, provide time_enabled, time_running and counter value in the
> > perf_sample dict.
> > 
> > Initial Discussion: https://lkml.org/lkml/2017/7/1/108
> > 
> > Arun Kalyanasundaram (5):
> >   perf script python: Allocate memory only if handler exists
> >   perf script python: Refactor creation of perf sample dict
> >   perf script python: Add sample_read to dict
> >   perf script python: Add perf_sample dict to tracepoint handlers
> >   perf script python: Generate hooks with additional argument
> > 
> >  .../util/scripting-engines/trace-event-python.c    | 246 +++++++++++++++------
> >  1 file changed, 184 insertions(+), 62 deletions(-)
> > 
> > -- 
> > 2.14.0.rc0.284.gd933b75aa4-goog
> > 

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 0/5] perf script python: Provide perf_sample dict to all handlers Arun Kalyanasundaram <arunkaly@google.com> - 2017-07-21 04:10 +0200
  [PATCH 5/5] perf script python: Generate hooks with additional argument Arun Kalyanasundaram <arunkaly@google.com> - 2017-07-21 04:10 +0200
  [PATCH 3/5] perf script python: Add sample_read to dict Arun Kalyanasundaram <arunkaly@google.com> - 2017-07-21 04:10 +0200
  [PATCH 4/5] perf script python: Add perf_sample dict to tracepoint handlers Arun Kalyanasundaram <arunkaly@google.com> - 2017-07-21 04:10 +0200
  Re: [PATCH 0/5] perf script python: Provide perf_sample dict to all  handlers Jiri Olsa <jolsa@redhat.com> - 2017-07-21 09:50 +0200
    Re: [PATCH 0/5] perf script python: Provide perf_sample dict to all  handlers Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-07-21 18:30 +0200
      Re: [PATCH 0/5] perf script python: Provide perf_sample dict to all handlers Arun Kalyanasundaram <arunkaly@google.com> - 2017-07-21 19:00 +0200
        Re: [PATCH 0/5] perf script python: Provide perf_sample dict to all  handlers Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-07-21 19:20 +0200

csiph-web