Commit 7e447b5f authored by Arnaldo Carvalho de Melo's avatar Arnaldo Carvalho de Melo Committed by Kamal Mostafa

perf symbols: Store if there is a filter in place

commit 0bc2f2f7 upstream.

When setting yup the symbols library we setup several filter lists,
for dsos, comms, symbols, etc, and there is code that, if there are
filters, do certain operations, like recalculate the number of non
filtered histogram entries in the top/report TUI.

But they were considering just the "Zoom" filters, when they need to
take into account as well the above mentioned filters (perf top --comms,
--dsos, etc).

So store in symbol_conf.has_filter true if any of those filters is in
place.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Borislav Petkov <bp@suse.de>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-f5edfmhq69vfvs1kmikq1wep@git.kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
[ kamal: 3.19-stable prereq for
  9c0fa8dd perf hists browser: Take the --comm, --dsos, etc filters into account ]
Signed-off-by: default avatarKamal Mostafa <kamal@canonical.com>
parent 843d28ea
...@@ -1856,6 +1856,8 @@ int setup_list(struct strlist **list, const char *list_str, ...@@ -1856,6 +1856,8 @@ int setup_list(struct strlist **list, const char *list_str,
pr_err("problems parsing %s list\n", list_name); pr_err("problems parsing %s list\n", list_name);
return -1; return -1;
} }
symbol_conf.has_filter = true;
return 0; return 0;
} }
......
...@@ -103,7 +103,8 @@ struct symbol_conf { ...@@ -103,7 +103,8 @@ struct symbol_conf {
demangle_kernel, demangle_kernel,
filter_relative, filter_relative,
show_hist_headers, show_hist_headers,
branch_callstack; branch_callstack,
has_filter;
const char *vmlinux_name, const char *vmlinux_name,
*kallsyms_name, *kallsyms_name,
*source_prefix, *source_prefix,
......
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