[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20091022050044.GX4808@kryten>
Date: Thu, 22 Oct 2009 16:00:44 +1100
From: Anton Blanchard <anton@...ba.org>
To: Ingo Molnar <mingo@...e.hu>
Cc: paulus@...ba.org, a.p.zijlstra@...llo.nl,
linux-kernel@...r.kernel.org, fweisbec@...il.com, acme@...hat.com
Subject: Re: [PATCH] perf record: Enable PERF_SAMPLE_ID when sampling
multiple events
Hi Ingo,
> > If we are sampling multiple events we need the id in each sample so we
> > can differentiate between them in a perf data file.
>
> Wondering, what are you (or will you be) using this for?
I put together a simple python library for parsing perf.data files:
http://ozlabs.org/~anton/junkcode/perf_event.py
An example of using it is here:
http://ozlabs.org/~anton/junkcode/perf_event_example.py
Only tested on powerpc so far, but it should work on x86. It's still
missing bits but it has been useful for finding some corner cases in
perf_event. It should also make it easy to post process complex profiles with
multiple events in them.
One problem this has just found though, is with PERF_EVENT_SAMPLE:
# FIXME: If sampling multiple events we have an issue
# here. Since the SAMPLE_ID is not the first optional field
# it might be impossible to differentiate between
# events since the SAMPLE_ID field would be at different
# offsets. For now we assume all events use the same
# set of optional fields.
eventnr = 0
self.event = sample_event(eventbuf,
self.header.attrs[eventnr].sample_type)
It seems like the API allows us to specify different sample options for
different events, but since the ID isnt the first option it could end up
in different places in different events, making it difficult (if not
impossible in some cases) to tag events correctly.
Anton
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists