Messages in this thread | | | Date | Tue, 21 Aug 2012 11:59:20 +0200 | From | Ingo Molnar <> | Subject | Re: [PATCH] perf: create a helper structure perf_event_context |
| |
* Andrew Vagin <avagin@openvz.org> wrote:
> 18 files changed, 222 insertions(+), 255 deletions(-)
> -static int process_sample_event(struct perf_tool *tool, > - union perf_event *event, > - struct perf_sample *sample, > - struct perf_evsel *evsel, > - struct machine *machine) > +static int process_sample_event(struct perf_event_context *ectx) > { > - struct perf_annotate *ann = container_of(tool, struct perf_annotate, tool); > + struct perf_sample *sample = ectx->sample; > + struct perf_annotate *ann = container_of(ectx->tool, struct perf_annotate, tool); > struct addr_location al;
that looks like a nice cleanliness win.
Acked-by: Ingo Molnar <mingo@kernel.org>
Thanks,
Ingo
| |