[PATCH] perf symbols: Fill in pgoff in mmap synthesized events

From: Anton Blanchard
Date: Sat Apr 03 2010 - 07:55:01 EST



When we synthesize mmap events we need to fill in the pgoff field.

Signed-off-by: Anton Blanchard <anton@xxxxxxxxx>
---

I wasn't able to test this completely since I couldn't find an executable
region with a non 0 offset. We will see it when we start doing data
profiling.

Index: linux.trees.git/tools/perf/util/event.c
===================================================================
--- linux.trees.git.orig/tools/perf/util/event.c 2010-04-01 07:55:13.000000000 +1100
+++ linux.trees.git/tools/perf/util/event.c 2010-04-03 22:45:02.000000000 +1100
@@ -130,6 +130,7 @@ static int event__synthesize_mmap_events
continue;
pbf += n + 3;
if (*pbf == 'x') { /* vm_exec */
+ u64 vm_pgoff;
char *execname = strchr(bf, '/');

/* Catch VDSO */
@@ -139,6 +140,14 @@ static int event__synthesize_mmap_events
if (execname == NULL)
continue;

+ pbf += 3;
+ n = hex2u64(pbf, &vm_pgoff);
+ /* pgoff is in bytes, not pages */
+ if (n >= 0)
+ ev.mmap.pgoff = vm_pgoff << getpagesize();
+ else
+ ev.mmap.pgoff = 0;
+
size = strlen(execname);
execname[size - 1] = '\0'; /* Remove \n */
memcpy(ev.mmap.filename, execname, size);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/