Skip to contentSkip to conversation

perf: add --symfs option to perf annotate

PATCH In review Stephane Eranian 2 participants 3 messages Jul 29, 2011 – Aug 18, 2011 perf events

lore
Share & tools
lore

Stephane Eranian

author
#0 lore raw

PATCH 1/1 perf: add --symfs option to perf annotate

If you have --symfs in perf report, then you also need it for perf annotate. This allows off-box assembly level analysis of perf.data samples.

This patch complements:

commit ec5761eab318e50e69fcf8e63e9edaef5949c067

Author: David Ahern <[email protected]>
Date:   Thu Dec 9 13:27:07 2010 -0700

perf symbols: Add symfs option for off-box analysis using specified tree

Signed-off-by Stephane Eranian <[email protected]>

---

tools/perf/Documentation/perf-annotate.txt +3 -0
index 85c5f02..15919dd 100644
@@ -72,6 +72,9 @@ OPTIONS
 	CPUs are specified with -: 0-2. Default is to report samples on all
 	CPUs.
 
+--symfs=<directory>::
+        Look for files with symbols relative to this directory.
+
 SEE ALSO
 --------
 linkperf:perf-record[1], linkperf:perf-report[1]
tools/perf/builtin-annotate.c +2 -0
index 555aefd..1605fbe 100644
@@ -267,6 +267,8 @@ static const struct option options[] = {
 	OPT_BOOLEAN('P', "full-paths", &full_paths,
 		    "Don't shorten the displayed pathnames"),
 	OPT_STRING('c', "cpu", &cpu_list, "cpu", "list of cpus to profile"),
+	OPT_STRING(0, "symfs", &symbol_conf.symfs, "directory",
+		    "Look for files with symbols relative to this directory"),
 	OPT_END()
 };
Acked-by on v1 1/1
#1 lore raw
Quoted 35 lines · Stephane Eranian

If you have --symfs in perf report, then you also need it for perf annotate. This allows off-box assembly level analysis of perf.data samples.

This patch complements:

commit ec5761eab318e50e69fcf8e63e9edaef5949c067

Author: David Ahern <[email protected]>
Date:   Thu Dec 9 13:27:07 2010 -0700

perf symbols: Add symfs option for off-box analysis using specified tree

Signed-off-by Stephane Eranian <[email protected]>

---

tools/perf/Documentation/perf-annotate.txt +3 -0
index 85c5f02..15919dd 100644
@@ -72,6 +72,9 @@ OPTIONS
 	CPUs are specified with -: 0-2. Default is to report samples on all
 	CPUs.
 
+--symfs=<directory>::
+        Look for files with symbols relative to this directory.
+
 SEE ALSO
 --------
 linkperf:perf-record[1], linkperf:perf-report[1]
tools/perf/builtin-annotate.c +0 -0
index 555aefd..1605fbe 100644

tools/perf/builtin-annotate.c static const struct option options[] = { · line 274

Quoted 8 lines
tools/perf/builtin-annotate.c +2 -0
@@ -267,6 +267,8 @@ static const struct option options[] = {
 	OPT_BOOLEAN('P', "full-paths", &full_paths,
 		    "Don't shorten the displayed pathnames"),
 	OPT_STRING('c', "cpu", &cpu_list, "cpu", "list of cpus to profile"),
+	OPT_STRING(0, "symfs", &symbol_conf.symfs, "directory",
+		    "Look for files with symbols relative to this directory"),
 	OPT_END()
 };
Acked-by David Ahern <[email protected]>

Nice to see others making use of that option.

David

author
#2 lore raw

[tip:perf/core] perf annotate: Add --symfs option

In reply to Stephane Eranian
Commit-ID:  e71a059832753a8834a5a5080366879954ccdc4d
Gitweb:     http://git.kernel.org/tip/e71a059832753a8834a5a5080366879954ccdc4d
Author:     Stephane Eranian <[email protected]>
AuthorDate: Sat, 30 Jul 2011 01:20:40 +0200
Committer:  Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Thu, 18 Aug 2011 07:40:34 -0300

perf annotate: Add --symfs option

If you have --symfs in perf report, then you also need it for perf annotate. This allows off-box assembly level analysis of perf.data samples.

This patch complements:

commit ec5761eab318e50e69fcf8e63e9edaef5949c067

Author: David Ahern <[email protected]>
Date:   Thu Dec 9 13:27:07 2010 -0700

perf symbols: Add symfs option for off-box analysis using specified tree

Acked-by David Ahern <[email protected]>
Cc Ingo Molnar <[email protected]>
Cc Peter Zijlstra <[email protected]>
Cc David Ahern <[email protected]>
Link http://lkml.kernel.org/r/20110729232040.GA21838@quad open here
Signed-off-by Stephane Eranian <[email protected]>
Signed-off-by Arnaldo Carvalho de Melo <[email protected]>
---
 tools/perf/Documentation/perf-annotate.txt |    3 +++
 tools/perf/builtin-annotate.c              |    2 ++
 2 files changed, 5 insertions(+), 0 deletions(-)
tools/perf/Documentation/perf-annotate.txt +3 -0
index 5bc0600..98a31e3 100644
@@ -80,6 +80,9 @@ OPTIONS
 	Interleave source code with assembly code. Enabled by default,
 	disable with --no-source.
 
+--symfs=<directory>::
+        Look for files with symbols relative to this directory.
+
 SEE ALSO
 --------
 linkperf:perf-record[1], linkperf:perf-report[1]
tools/perf/builtin-annotate.c +2 -0
index 5015e04..c5be288 100644
@@ -267,6 +267,8 @@ static const struct option options[] = {
 	OPT_BOOLEAN('P', "full-paths", &full_paths,
 		    "Don't shorten the displayed pathnames"),
 	OPT_STRING('c', "cpu", &cpu_list, "cpu", "list of cpus to profile"),
+	OPT_STRING(0, "symfs", &symbol_conf.symfs, "directory",
+		   "Look for files with symbols relative to this directory"),
 	OPT_BOOLEAN('0', "source", &symbol_conf.annotate_src,
 		    "Interleave source code with assembly code (default)"),
 	OPT_BOOLEAN('0', "asm-raw", &symbol_conf.annotate_asm_raw,