Messages in this thread Patch in this message | | | Date | Thu, 9 Dec 2010 23:37:27 GMT | From | tip-bot for Thomas Gleixner <> | Subject | [tip:perf/core] perf session: Dont queue events w/o timestamps |
| |
Commit-ID: 79a14c1f458d598642bf11f09512c83d33a114e6 Gitweb: http://git.kernel.org/tip/79a14c1f458d598642bf11f09512c83d33a114e6 Author: Thomas Gleixner <tglx@linutronix.de> AuthorDate: Tue, 7 Dec 2010 12:48:44 +0000 Committer: Arnaldo Carvalho de Melo <acme@redhat.com> CommitDate: Thu, 9 Dec 2010 11:15:38 -0200
perf session: Dont queue events w/o timestamps
If the event has no timestamp assigned then the parse code sets it to ~0ULL which causes the ordering code to enqueue it at the end.
Process it right away.
Reported-by: Ian Munsie <imunsie@au1.ibm.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Ian Munsie <imunsie@au1.ibm.com> Cc: Ingo Molnar <mingo@elte.hu> Cc: Peter Zijlstra <peterz@infradead.org> LKML-Reference: <20101207124550.528788441@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 | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/perf/util/session.c b/tools/perf/util/session.c index b3b145a..e5d0ed9 100644 --- a/tools/perf/util/session.c +++ b/tools/perf/util/session.c @@ -603,7 +603,7 @@ static int perf_session_queue_event(struct perf_session *s, event_t *event, u64 timestamp = data->time; struct sample_queue *new; - if (!timestamp) + if (!timestamp || timestamp == ~0ULL) return -ETIME; if (timestamp < s->ordered_samples.last_flush) {
| |