lkml.org 
[lkml]   [2018]   [May]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[RFC 2/6] perf probe: Parse linerange for C++ functions
Date
From: Holger Hans Peter Freyther <holgar+kernel@google.com>

perf probe --funcs will demangle C++ symbols by default but these
functions can not be used for listing sourcecode. Modify the scanner
to start searching for a line number only after a single ':'.

./perf probe -x ./cxx-example -L \
"std::vector<int, std::allocator<int> >::at:1"

Signed-off-by: Holger Hans Peter Freyther <holgar+kernel@google.com>
---
tools/perf/util/probe-event.c | 57 ++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 56 insertions(+), 1 deletion(-)

diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c
index e1dbc98..39a2d47 100644
--- a/tools/perf/util/probe-event.c
+++ b/tools/perf/util/probe-event.c
@@ -1237,6 +1237,59 @@ static bool is_c_func_name(const char *name)
return true;
}

+/* Symbols in demangled CXX function names */
+static inline bool is_cxx_symbol(const char symbol)
+{
+ switch (symbol) {
+ case '_':
+ case ' ':
+ case '&':
+ case '*':
+ case '@':
+ case ',':
+ case ':':
+ case '<':
+ case '>':
+ case '(':
+ case ')':
+ return true;
+ default:
+ return false;
+ }
+}
+
+/* Is name a C++ name? */
+static bool is_cxx_func_name(const char *name)
+{
+ /* C name or a mangled name */
+ if (is_c_func_name(name))
+ return true;
+ while (*++name != '\0') {
+ if (!isalpha(*name) && !isdigit(*name) && !is_cxx_symbol(*name))
+ return false;
+ }
+ return true;
+}
+
+/*
+ * Find the first ':' that isn't part of a C++ namespace or class
+ * name.
+ */
+static char *first_non_cxx_ns(char *name)
+{
+ while (*name) {
+ char cur = *name, nxt = *(name + 1);
+
+ if (cur == ':' && nxt == ':')
+ name += 2;
+ else if (cur == ':')
+ return name;
+
+ name += 1;
+ }
+ return NULL;
+}
+
/*
* Stuff 'lr' according to the line range described by 'arg'.
* The line range syntax is described by:
@@ -1255,7 +1308,7 @@ int parse_line_range_desc(const char *arg, struct line_range *lr)
lr->start = 0;
lr->end = INT_MAX;

- range = strchr(name, ':');
+ range = first_non_cxx_ns(name);
if (range) {
*range++ = '\0';

@@ -1309,6 +1362,8 @@ int parse_line_range_desc(const char *arg, struct line_range *lr)
lr->file = name;
else if (is_c_func_name(name))/* We reuse it for checking funcname */
lr->function = name;
+ else if (is_cxx_func_name(name))
+ lr->function = name;
else { /* Invalid name */
semantic_error("'%s' is not a valid function name.\n", name);
err = -EINVAL;
--
2.7.4
\
 
 \ /
  Last update: 2018-05-14 06:28    [W:0.053 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site