Commit 3c42258c authored by Masami Hiramatsu's avatar Masami Hiramatsu Committed by Arnaldo Carvalho de Melo

perf probe: Add filters support for available functions

Add filters support for available function list.

Default filter is "!_*" for filtering out local-purpose symbols.

e.g.:
 # perf probe --filter="add*" -F
add_disk
add_disk_randomness
add_input_randomness
add_interrupt_randomness
add_memory
add_page_to_unevictable_list
add_page_wait_queue
...

Cc: 2nddept-manager@sdl.hitachi.co.jp
Cc: Chase Douglas <chase.douglas@canonical.com>
Cc: Franck Bui-Huu <fbuihuu@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
LKML-Reference: <20110120141545.25915.85930.stgit@ltc236.sdl.hitachi.co.jp>
Signed-off-by: default avatarMasami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent bd09d7b5
...@@ -78,10 +78,11 @@ OPTIONS ...@@ -78,10 +78,11 @@ OPTIONS
Show available functions in given module or kernel. Show available functions in given module or kernel.
--filter=FILTER:: --filter=FILTER::
(Only for --vars) Set filter for variables. FILTER is a combination of (Only for --vars and --funcs) Set filter. FILTER is a combination of glob
glob pattern, see FILTER PATTERN for details. pattern, see FILTER PATTERN for detail.
Default FILTER is "!__k???tab_* & !__crc_*". Default FILTER is "!__k???tab_* & !__crc_*" for --vars, and "!_*"
If several filters are specified, only the last filter is valid. for --funcs.
If several filters are specified, only the last filter is used.
-f:: -f::
--force:: --force::
......
...@@ -45,6 +45,7 @@ ...@@ -45,6 +45,7 @@
#include "util/probe-event.h" #include "util/probe-event.h"
#define DEFAULT_VAR_FILTER "!__k???tab_* & !__crc_*" #define DEFAULT_VAR_FILTER "!__k???tab_* & !__crc_*"
#define DEFAULT_FUNC_FILTER "!_*"
#define MAX_PATH_LEN 256 #define MAX_PATH_LEN 256
/* Session management structure */ /* Session management structure */
...@@ -159,6 +160,7 @@ static int opt_show_vars(const struct option *opt __used, ...@@ -159,6 +160,7 @@ static int opt_show_vars(const struct option *opt __used,
return ret; return ret;
} }
#endif
static int opt_set_filter(const struct option *opt __used, static int opt_set_filter(const struct option *opt __used,
const char *str, int unset __used) const char *str, int unset __used)
...@@ -180,7 +182,6 @@ static int opt_set_filter(const struct option *opt __used, ...@@ -180,7 +182,6 @@ static int opt_set_filter(const struct option *opt __used,
return 0; return 0;
} }
#endif
static const char * const probe_usage[] = { static const char * const probe_usage[] = {
"perf probe [<options>] 'PROBEDEF' ['PROBEDEF' ...]", "perf probe [<options>] 'PROBEDEF' ['PROBEDEF' ...]",
...@@ -236,10 +237,6 @@ static const struct option options[] = { ...@@ -236,10 +237,6 @@ static const struct option options[] = {
"Show accessible variables on PROBEDEF", opt_show_vars), "Show accessible variables on PROBEDEF", opt_show_vars),
OPT_BOOLEAN('\0', "externs", &params.show_ext_vars, OPT_BOOLEAN('\0', "externs", &params.show_ext_vars,
"Show external variables too (with --vars only)"), "Show external variables too (with --vars only)"),
OPT_CALLBACK('\0', "filter", NULL,
"[!]FILTER", "Set a variable filter (with --vars only)\n"
"\t\t\t(default: \"" DEFAULT_VAR_FILTER "\")",
opt_set_filter),
OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name, OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name,
"file", "vmlinux pathname"), "file", "vmlinux pathname"),
OPT_STRING('s', "source", &symbol_conf.source_prefix, OPT_STRING('s', "source", &symbol_conf.source_prefix,
...@@ -252,6 +249,11 @@ static const struct option options[] = { ...@@ -252,6 +249,11 @@ static const struct option options[] = {
"Set how many probe points can be found for a probe."), "Set how many probe points can be found for a probe."),
OPT_BOOLEAN('F', "funcs", &params.show_funcs, OPT_BOOLEAN('F', "funcs", &params.show_funcs,
"Show potential probe-able functions."), "Show potential probe-able functions."),
OPT_CALLBACK('\0', "filter", NULL,
"[!]FILTER", "Set a filter (with --vars/funcs only)\n"
"\t\t\t(default: \"" DEFAULT_VAR_FILTER "\" for --vars,\n"
"\t\t\t \"" DEFAULT_FUNC_FILTER "\" for --funcs)",
opt_set_filter),
OPT_END() OPT_END()
}; };
...@@ -322,7 +324,12 @@ int cmd_probe(int argc, const char **argv, const char *prefix __used) ...@@ -322,7 +324,12 @@ int cmd_probe(int argc, const char **argv, const char *prefix __used)
pr_err(" Error: Don't use --funcs with --vars.\n"); pr_err(" Error: Don't use --funcs with --vars.\n");
usage_with_options(probe_usage, options); usage_with_options(probe_usage, options);
} }
ret = show_available_funcs(params.target_module); if (!params.filter)
params.filter = strfilter__new(DEFAULT_FUNC_FILTER,
NULL);
ret = show_available_funcs(params.target_module,
params.filter);
strfilter__delete(params.filter);
if (ret < 0) if (ret < 0)
pr_err(" Error: Failed to show functions." pr_err(" Error: Failed to show functions."
" (%d)\n", ret); " (%d)\n", ret);
......
...@@ -1951,21 +1951,23 @@ int del_perf_probe_events(struct strlist *dellist) ...@@ -1951,21 +1951,23 @@ int del_perf_probe_events(struct strlist *dellist)
return ret; return ret;
} }
/* TODO: don't use a global variable for filter ... */
static struct strfilter *available_func_filter;
/* /*
* If a symbol corresponds to a function with global binding return 0. * If a symbol corresponds to a function with global binding and
* For all others return 1. * matches filter return 0. For all others return 1.
*/ */
static int filter_non_global_functions(struct map *map __unused, static int filter_available_functions(struct map *map __unused,
struct symbol *sym) struct symbol *sym)
{ {
if (sym->binding != STB_GLOBAL) if (sym->binding == STB_GLOBAL &&
return 1; strfilter__compare(available_func_filter, sym->name))
return 0;
return 0; return 1;
} }
int show_available_funcs(const char *module) int show_available_funcs(const char *module, struct strfilter *_filter)
{ {
struct map *map; struct map *map;
int ret; int ret;
...@@ -1981,7 +1983,8 @@ int show_available_funcs(const char *module) ...@@ -1981,7 +1983,8 @@ int show_available_funcs(const char *module)
pr_err("Failed to find %s map.\n", (module) ? : "kernel"); pr_err("Failed to find %s map.\n", (module) ? : "kernel");
return -EINVAL; return -EINVAL;
} }
if (map__load(map, filter_non_global_functions)) { available_func_filter = _filter;
if (map__load(map, filter_available_functions)) {
pr_err("Failed to load map.\n"); pr_err("Failed to load map.\n");
return -EINVAL; return -EINVAL;
} }
......
...@@ -128,7 +128,7 @@ extern int show_line_range(struct line_range *lr, const char *module); ...@@ -128,7 +128,7 @@ extern int show_line_range(struct line_range *lr, const char *module);
extern int show_available_vars(struct perf_probe_event *pevs, int npevs, extern int show_available_vars(struct perf_probe_event *pevs, int npevs,
int max_probe_points, const char *module, int max_probe_points, const char *module,
struct strfilter *filter, bool externs); struct strfilter *filter, bool externs);
extern int show_available_funcs(const char *module); extern int show_available_funcs(const char *module, struct strfilter *filter);
/* Maximum index number of event-name postfix */ /* Maximum index number of event-name postfix */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment