Commit 18f278d2 authored by Jiri Olsa's avatar Jiri Olsa Committed by Arnaldo Carvalho de Melo

perf c2c report: Add --no-source option

Add a possibility to disable source line column with new --no-source
option. It source line data could take lot of time to retrieve, so it
could be a performance burden for big data.
Signed-off-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Don Zickus <dzickus@redhat.com>
Cc: Joe Mario <jmario@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/n/tip-8p6s2727fq8nbsm3it5gix3p@git.kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 465f27a3
...@@ -94,6 +94,9 @@ REPORT OPTIONS ...@@ -94,6 +94,9 @@ REPORT OPTIONS
--full-symbols:: --full-symbols::
Display full length of symbols. Display full length of symbols.
--no-source::
Do not display Source:Line column.
C2C RECORD C2C RECORD
---------- ----------
The perf c2c record command setup options related to HITM cacheline analysis The perf c2c record command setup options related to HITM cacheline analysis
......
...@@ -2404,7 +2404,7 @@ static int setup_display(const char *str) ...@@ -2404,7 +2404,7 @@ static int setup_display(const char *str)
for (__tok = strtok_r(__buf, __sep, &__tmp); __tok; \ for (__tok = strtok_r(__buf, __sep, &__tmp); __tok; \
__tok = strtok_r(NULL, __sep, &__tmp)) __tok = strtok_r(NULL, __sep, &__tmp))
static int build_cl_output(char *cl_sort) static int build_cl_output(char *cl_sort, bool no_source)
{ {
char *tok, *tmp, *buf = strdup(cl_sort); char *tok, *tmp, *buf = strdup(cl_sort);
bool add_pid = false; bool add_pid = false;
...@@ -2426,7 +2426,7 @@ static int build_cl_output(char *cl_sort) ...@@ -2426,7 +2426,7 @@ static int build_cl_output(char *cl_sort)
add_iaddr = true; add_iaddr = true;
add_sym = true; add_sym = true;
add_dso = true; add_dso = true;
add_src = true; add_src = no_source ? false : true;
} else if (!strcmp(tok, "dso")) { } else if (!strcmp(tok, "dso")) {
add_dso = true; add_dso = true;
} else if (strcmp(tok, "offset")) { } else if (strcmp(tok, "offset")) {
...@@ -2462,14 +2462,14 @@ static int build_cl_output(char *cl_sort) ...@@ -2462,14 +2462,14 @@ static int build_cl_output(char *cl_sort)
return 0; return 0;
} }
static int setup_coalesce(const char *coalesce) static int setup_coalesce(const char *coalesce, bool no_source)
{ {
const char *c = coalesce ?: coalesce_default; const char *c = coalesce ?: coalesce_default;
if (asprintf(&c2c.cl_sort, "offset,%s", c) < 0) if (asprintf(&c2c.cl_sort, "offset,%s", c) < 0)
return -ENOMEM; return -ENOMEM;
if (build_cl_output(c2c.cl_sort)) if (build_cl_output(c2c.cl_sort, no_source))
return -1; return -1;
if (asprintf(&c2c.cl_resort, "offset,%s", if (asprintf(&c2c.cl_resort, "offset,%s",
...@@ -2494,6 +2494,7 @@ static int perf_c2c__report(int argc, const char **argv) ...@@ -2494,6 +2494,7 @@ static int perf_c2c__report(int argc, const char **argv)
char callchain_default_opt[] = CALLCHAIN_DEFAULT_OPT; char callchain_default_opt[] = CALLCHAIN_DEFAULT_OPT;
const char *display = NULL; const char *display = NULL;
const char *coalesce = NULL; const char *coalesce = NULL;
bool no_source = false;
const struct option c2c_options[] = { const struct option c2c_options[] = {
OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name, OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name,
"file", "vmlinux pathname"), "file", "vmlinux pathname"),
...@@ -2510,6 +2511,8 @@ static int perf_c2c__report(int argc, const char **argv) ...@@ -2510,6 +2511,8 @@ static int perf_c2c__report(int argc, const char **argv)
"Use the stdio interface"), "Use the stdio interface"),
OPT_BOOLEAN(0, "full-symbols", &c2c.symbol_full, OPT_BOOLEAN(0, "full-symbols", &c2c.symbol_full,
"Display full length of symbols"), "Display full length of symbols"),
OPT_BOOLEAN(0, "no-source", &no_source,
"Do not display Source Line column"),
OPT_CALLBACK_DEFAULT('g', "call-graph", &callchain_param, OPT_CALLBACK_DEFAULT('g', "call-graph", &callchain_param,
"print_type,threshold[,print_limit],order,sort_key[,branch],value", "print_type,threshold[,print_limit],order,sort_key[,branch],value",
callchain_help, &parse_callchain_opt, callchain_help, &parse_callchain_opt,
...@@ -2545,7 +2548,7 @@ static int perf_c2c__report(int argc, const char **argv) ...@@ -2545,7 +2548,7 @@ static int perf_c2c__report(int argc, const char **argv)
if (err) if (err)
goto out; goto out;
err = setup_coalesce(coalesce); err = setup_coalesce(coalesce, no_source);
if (err) { if (err) {
pr_debug("Failed to initialize hists\n"); pr_debug("Failed to initialize hists\n");
goto out; goto out;
......
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