[tip:perf/core] perf top: Remove old kernel-only symbol filter

[tip-bot for Arnaldo Carvalho de Melo]

Commit-ID:  b6220212d48a9bfbc694d10b38dbdfbaab81f4a0
Gitweb: http://git.kernel.org/tip/b6220212d48a9bfbc694d10b38dbdfbaab81f4a0
Author: Arnaldo Carvalho de Melo 
AuthorDate: Thu, 1 Sep 2016 18:47:15 -0300
Committer:  Arnaldo Carvalho de Melo 
CommitDate: Mon, 5 Sep 2016 11:14:50 -0300

perf top: Remove old kernel-only symbol filter

Not needed, we already have code to prune aliases.

Cc: Adrian Hunter 
Cc: David Ahern 
Cc: Jiri Olsa 
Cc: Masami Hiramatsu 
Cc: Namhyung Kim 
Cc: Wang Nan 
Link: http://lkml.kernel.org/n/tip-1ysyce7qjgui93gi1efbj...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo 
---
 tools/perf/builtin-top.c | 27 ---
 1 file changed, 27 deletions(-)

diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c
index 6f48df1..4007857 100644
--- a/tools/perf/builtin-top.c
+++ b/tools/perf/builtin-top.c
@@ -657,31 +657,6 @@ repeat:
return NULL;
 }
 
-static int symbol_filter(struct map *map, struct symbol *sym)
-{
-   const char *name = sym->name;
-
-   if (!__map__is_kernel(map))
-   return 0;
-   /*
-* ppc64 uses function descriptors and appends a '.' to the
-* start of every instruction address. Remove it.
-*/
-   if (name[0] == '.')
-   name++;
-
-   if (!strcmp(name, "_text") ||
-   !strcmp(name, "_etext") ||
-   !strcmp(name, "_sinittext") ||
-   !strncmp("init_module", name, 11) ||
-   !strncmp("cleanup_module", name, 14) ||
-   strstr(name, "_text_start") ||
-   strstr(name, "_text_end"))
-   return 1;
-
-   return 0;
-}
-
 static int hist_iter__top_callback(struct hist_entry_iter *iter,
   struct addr_location *al, bool single,
   void *arg)
@@ -946,8 +921,6 @@ static int __cmd_top(struct perf_top *top)
if (top->session == NULL)
return -1;
 
-   machines__set_symbol_filter(>session->machines, symbol_filter);
-
if (!objdump_path) {
ret = perf_env__lookup_objdump(>session->header.env);
if (ret)

[tip:perf/core] perf top: Remove old kernel-only symbol filter

[tip-bot for Arnaldo Carvalho de Melo]

Commit-ID:  b6220212d48a9bfbc694d10b38dbdfbaab81f4a0
Gitweb: http://git.kernel.org/tip/b6220212d48a9bfbc694d10b38dbdfbaab81f4a0
Author: Arnaldo Carvalho de Melo 
AuthorDate: Thu, 1 Sep 2016 18:47:15 -0300
Committer:  Arnaldo Carvalho de Melo 
CommitDate: Mon, 5 Sep 2016 11:14:50 -0300

perf top: Remove old kernel-only symbol filter

Not needed, we already have code to prune aliases.

Cc: Adrian Hunter 
Cc: David Ahern 
Cc: Jiri Olsa 
Cc: Masami Hiramatsu 
Cc: Namhyung Kim 
Cc: Wang Nan 
Link: http://lkml.kernel.org/n/tip-1ysyce7qjgui93gi1efbj...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo 
---
 tools/perf/builtin-top.c | 27 ---
 1 file changed, 27 deletions(-)

diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c
index 6f48df1..4007857 100644
--- a/tools/perf/builtin-top.c
+++ b/tools/perf/builtin-top.c
@@ -657,31 +657,6 @@ repeat:
return NULL;
 }
 
-static int symbol_filter(struct map *map, struct symbol *sym)
-{
-   const char *name = sym->name;
-
-   if (!__map__is_kernel(map))
-   return 0;
-   /*
-* ppc64 uses function descriptors and appends a '.' to the
-* start of every instruction address. Remove it.
-*/
-   if (name[0] == '.')
-   name++;
-
-   if (!strcmp(name, "_text") ||
-   !strcmp(name, "_etext") ||
-   !strcmp(name, "_sinittext") ||
-   !strncmp("init_module", name, 11) ||
-   !strncmp("cleanup_module", name, 14) ||
-   strstr(name, "_text_start") ||
-   strstr(name, "_text_end"))
-   return 1;
-
-   return 0;
-}
-
 static int hist_iter__top_callback(struct hist_entry_iter *iter,
   struct addr_location *al, bool single,
   void *arg)
@@ -946,8 +921,6 @@ static int __cmd_top(struct perf_top *top)
if (top->session == NULL)
return -1;
 
-   machines__set_symbol_filter(>session->machines, symbol_filter);
-
if (!objdump_path) {
ret = perf_env__lookup_objdump(>session->header.env);
if (ret)