perf session: Use appropriate pointer type instead of silly typecasting
authorThomas Gleixner <tglx@linutronix.de>
Tue, 30 Nov 2010 17:49:35 +0000 (17:49 +0000)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 30 Nov 2010 21:55:26 +0000 (19:55 -0200)
There is no reason to use a struct sample_event pointer in struct sample_queue
and type cast it when flushing the queue.

Cc: Ingo Molnar <mingo@elte.hu>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
LKML-Reference: <20101130163819.969462809@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/session.c

index daca557b9d2896077ba88e2b9b5c950c6f6064d4..525bcf6adc211000dee3f0205b10dc07c86bf31a 100644 (file)
@@ -386,7 +386,7 @@ static event__swap_op event__swap_ops[] = {
 
 struct sample_queue {
        u64                     timestamp;
-       struct sample_event     *event;
+       event_t                 *event;
        struct list_head        list;
 };
 
@@ -406,7 +406,7 @@ static void flush_sample_queue(struct perf_session *s,
                if (iter->timestamp > limit)
                        break;
 
-               ops->sample((event_t *)iter->event, s);
+               ops->sample(iter->event, s);
 
                os->last_flush = iter->timestamp;
                list_del(&iter->list);