2009-06-30 23:18:48

by Anton Blanchard

Subject: [patch 6/8] perf report: Add hypervisor dso

Add a dso for hypervisor samples. We don't get any symbol information on
the ppc64 hypervisor but this at least gives us a high level summary of the
time spent in there.

Signed-off-by: Anton Blanchard <[email protected]>
---

Index: linux-2.6-tip/tools/perf/builtin-report.c
===================================================================
--- linux-2.6-tip.orig/tools/perf/builtin-report.c 2009-07-01 08:21:39.000000000 +1000
+++ linux-2.6-tip/tools/perf/builtin-report.c 2009-07-01 08:21:43.000000000 +1000
@@ -118,6 +118,7 @@
static LIST_HEAD(dsos);
static struct dso *kernel_dso;
static struct dso *vdso;
+static struct dso *hypervisor_dso;

static void dsos__add(struct dso *dso)
{
@@ -199,6 +200,11 @@

dsos__add(vdso);

+ hypervisor_dso = dso__new("[hypervisor]", 0);
+ if (!hypervisor_dso)
+ return -1;
+ dsos__add(hypervisor_dso);
+
return err;
}

@@ -637,7 +643,8 @@

if (self->sym) {
ret += fprintf(fp, "[%c] %s",
- self->dso == kernel_dso ? 'k' : '.', self->sym->name);
+ self->dso == kernel_dso ? 'k' :
+ self->dso == hypervisor_dso ? 'h' : '.', self->sym->name);
} else {
ret += fprintf(fp, "%#016llx", (u64)self->ip);
}
@@ -960,6 +967,9 @@
}

switch (context) {
+ case PERF_CONTEXT_HV:
+ dso = hypervisor_dso;
+ break;
case PERF_CONTEXT_KERNEL:
dso = kernel_dso;
break;
@@ -1269,6 +1279,9 @@
} else {
show = SHOW_HV;
level = 'H';
+
+ dso = hypervisor_dso;
+
dprintf(" ...... dso: [hypervisor]\n");
}


--

2009-06-30 23:29:50

by Anton Blanchard

Subject: [tip:perfcounters/urgent] perf report: Add hypervisor dso

Commit-ID: fb9c818873a788c5c01c9868cc6050df96e2c7df
Gitweb: http://git.kernel.org/tip/fb9c818873a788c5c01c9868cc6050df96e2c7df
Author: Anton Blanchard <[email protected]>
AuthorDate: Wed, 1 Jul 2009 09:00:49 +1000
Committer: Ingo Molnar <[email protected]>
CommitDate: Wed, 1 Jul 2009 01:25:20 +0200

perf report: Add hypervisor dso

Add a dso for hypervisor samples. We don't get any symbol
information on the ppc64 hypervisor but this at least gives
us a high level summary of the time spent in there.

Signed-off-by: Anton Blanchard <[email protected]>
Cc: [email protected]
Cc: [email protected]
LKML-Reference: <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>


---
tools/perf/builtin-report.c | 15 ++++++++++++++-
1 files changed, 14 insertions(+), 1 deletions(-)

diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 88e88c5..3f5d8ea 100644
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c
@@ -121,6 +121,7 @@ typedef union event_union {
static LIST_HEAD(dsos);
static struct dso *kernel_dso;
static struct dso *vdso;
+static struct dso *hypervisor_dso;

static void dsos__add(struct dso *dso)
{
@@ -202,6 +203,11 @@ static int load_kernel(void)

dsos__add(vdso);

+ hypervisor_dso = dso__new("[hypervisor]", 0);
+ if (!hypervisor_dso)
+ return -1;
+ dsos__add(hypervisor_dso);
+
return err;
}

@@ -640,7 +646,8 @@ sort__sym_print(FILE *fp, struct hist_entry *self)

if (self->sym) {
ret += fprintf(fp, "[%c] %s",
- self->dso == kernel_dso ? 'k' : '.', self->sym->name);
+ self->dso == kernel_dso ? 'k' :
+ self->dso == hypervisor_dso ? 'h' : '.', self->sym->name);
} else {
ret += fprintf(fp, "%#016llx", (u64)self->ip);
}
@@ -963,6 +970,9 @@ hist_entry__add(struct thread *thread, struct map *map, struct dso *dso,
}

switch (context) {
+ case PERF_CONTEXT_HV:
+ dso = hypervisor_dso;
+ break;
case PERF_CONTEXT_KERNEL:
dso = kernel_dso;
break;
@@ -1275,6 +1285,9 @@ process_sample_event(event_t *event, unsigned long offset, unsigned long head)
} else {
show = SHOW_HV;
level = 'H';
+
+ dso = hypervisor_dso;
+
dprintf(" ...... dso: [hypervisor]\n");
}