Messages in this thread |  | | | Date | Fri, 18 May 2012 10:55:27 +0800 | | From | Feng Tang <> | | Subject | Re: [PATCH 3/3] perf script/python: Pass thread/dso name and symbol info to event handler in python |
| |
On Thu, 17 May 2012 12:47:26 -0300 Arnaldo Carvalho de Melo <acme@redhat.com> wrote:
> Em Wed, May 16, 2012 at 08:59:15PM +0800, Feng Tang escreveu: > > Signed-off-by: Feng Tang <feng.tang@intel.com> > > --- > > .../util/scripting-engines/trace-event-python.c | 11 ++++++++++- > > 1 files changed, 10 insertions(+), 1 deletions(-) > > > > diff --git a/tools/perf/util/scripting-engines/trace-event-python.c > > b/tools/perf/util/scripting-engines/trace-event-python.c index > > a202881..c96f5e2 100644 --- > > a/tools/perf/util/scripting-engines/trace-event-python.c +++ > > b/tools/perf/util/scripting-engines/trace-event-python.c @@ -336,6 +336,7 > > @@ static void python_process_general_event(union perf_event *pevent > > __unused, static char handler_name[64]; unsigned n = 0; > > void *data = sample->raw_data; > > + struct thread *thread = al->thread; > > > > t = PyTuple_New(MAX_FIELDS); > > if (!t) > > @@ -348,13 +349,21 @@ static void python_process_general_event(union > > perf_event *pevent __unused, goto exit; > > } > > > > - /* Pass 3 parameters: event_attr, perf_sample, raw data, thread > > name */ > > + /* Pass parameters: attr, perf_sample, raw data, thread and dso > > name */ PyTuple_SetItem(t, n++, PyString_FromStringAndSize( > > (const char *)&evsel->attr, sizeof(evsel->attr))); > > PyTuple_SetItem(t, n++, PyString_FromStringAndSize( > > (const char *)sample, sizeof(*sample))); > > PyTuple_SetItem(t, n++, PyString_FromStringAndSize( > > data, sample->raw_size)); > > + PyTuple_SetItem(t, n++, PyString_FromString(thread->comm)); > > + PyTuple_SetItem(t, n++, PyString_FromString(al->map->dso->name)); > > + > > + /* Pass the resolved symbol if there is, othersize pass "Unkown" */ > > + if (al->sym) > > + PyTuple_SetItem(t, n++, > > PyString_FromString(al->sym->name)); > > + else > > + PyTuple_SetItem(t, n++, PyString_FromString("Unknown")); > > Isn't this getting a little bit convoluted? > > I.e. python has dictionaries, perhaps we could pass a dict instead of a > tuple, in that case we would simply not add the "symbol" key.
IIRC, the PyObject_CallObjects() only accept "tuple" arguments, or did you mean make the "symbol" a dict and an item of the "t" tuple?
I agree the symbol code is ugly, is following a little better? PyTuple_SetItem(t, n++, PyString_FromString((al->sym) ? al->sym->name : "Unknown"));
Thanks, Feng
> > > if (_PyTuple_Resize(&t, n) == -1) > > Py_FatalError("error resizing Python tuple"); > > -- > > 1.7.1
|  |