Messages in this thread | | | Date | Mon, 13 Aug 2012 11:17:43 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH 2/3] perf: teach perf inject to merge sched_stat_* and sched_switch events |
| |
* Arnaldo Carvalho de Melo <acme@ghostprotocols.net> wrote:
> static int process_sample_event(struct perf_tool *tool __used, > union perf_event *event, > struct perf_sample *sample, > struct perf_evsel *evsel, > struct machine *machine)
Just saw this 5-parameter function signature fly by: as a separate clean-up it would be really neat to stick most of these into an intuitively named helper structure or so.
struct event_context ectx?
That would make extension of the context easier as well in the future. Or so.
Thanks,
Ingo
| |