Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1236529

[PATCH perf/core 5/5] perf-probe: Improve error message when %return is on inlined function

Show all headers | View raw

perf probe shows more precisely message when it finds given
%return target function is inlined.

Without this fix:
  ----
  # ./perf probe -V getname_flags%return
  Return probe must be on the head of a real function.
  Debuginfo analysis failed.
    Error: Failed to show vars.
  ----

With this fix:
  ----
  # ./perf probe -V getname_flags%return
  Failed to find "getname_flags%return",
   because getname_flags is an inlined function and has no return point.
  Debuginfo analysis failed.
    Error: Failed to show vars.
  ----

Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Suggested-by: Arnaldo Carvalho de Melo <acme@kernel.org>
---
 tools/perf/util/probe-finder.c |   13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/tools/perf/util/probe-finder.c b/tools/perf/util/probe-finder.c
index f0708ff..bd8f03d 100644
--- a/tools/perf/util/probe-finder.c
+++ b/tools/perf/util/probe-finder.c
@@ -594,6 +594,7 @@ static int find_variable(Dwarf_Die *sc_die, struct probe_finder *pf)
 /* Convert subprogram DIE to trace point */
 static int convert_to_trace_point(Dwarf_Die *sp_die, Dwfl_Module *mod,
 				  Dwarf_Addr paddr, bool retprobe,
+				  const char *function,
 				  struct probe_trace_point *tp)
 {
 	Dwarf_Addr eaddr, highaddr;
@@ -637,8 +638,10 @@ static int convert_to_trace_point(Dwarf_Die *sp_die, Dwfl_Module *mod,
 	/* Return probe must be on the head of a subprogram */
 	if (retprobe) {
 		if (eaddr != paddr) {
-			pr_warning("Return probe must be on the head of"
-				   " a real function.\n");
+			pr_warning("Failed to find \"%s%%return\",\n"
+				   " because %s is an inlined function and"
+				   " has no return point.\n", function,
+				   function);
 			return -EINVAL;
 		}
 		tp->retprobe = true;
@@ -1178,6 +1181,7 @@ static int add_probe_trace_event(Dwarf_Die *sc_die, struct probe_finder *pf)
 {
 	struct trace_event_finder *tf =
 			container_of(pf, struct trace_event_finder, pf);
+	struct perf_probe_point *pp = &pf->pev->point;
 	struct probe_trace_event *tev;
 	struct perf_probe_arg *args;
 	int ret, i;
@@ -1192,7 +1196,7 @@ static int add_probe_trace_event(Dwarf_Die *sc_die, struct probe_finder *pf)
 
 	/* Trace point should be converted from subprogram DIE */
 	ret = convert_to_trace_point(&pf->sp_die, tf->mod, pf->addr,
-				     pf->pev->point.retprobe, &tev->point);
+				     pp->retprobe, pp->function, &tev->point);
 	if (ret < 0)
 		return ret;
 
@@ -1322,6 +1326,7 @@ static int add_available_vars(Dwarf_Die *sc_die, struct probe_finder *pf)
 {
 	struct available_var_finder *af =
 			container_of(pf, struct available_var_finder, pf);
+	struct perf_probe_point *pp = &pf->pev->point;
 	struct variable_list *vl;
 	Dwarf_Die die_mem;
 	int ret;
@@ -1335,7 +1340,7 @@ static int add_available_vars(Dwarf_Die *sc_die, struct probe_finder *pf)
 
 	/* Trace point should be converted from subprogram DIE */
 	ret = convert_to_trace_point(&pf->sp_die, af->mod, pf->addr,
-				     pf->pev->point.retprobe, &vl->point);
+				     pp->retprobe, pp->function, &vl->point);
 	if (ret < 0)
 		return ret;
 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH perf/core  0/5] perf-probe bugfixes Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> - 2015-09-30 18:50 +0200
  [PATCH perf/core 5/5] perf-probe: Improve error message when  %return is on inlined function Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> - 2015-09-30 18:50 +0200
    [tip:perf/core] perf probe:   Improve error message when %return is on inlined function tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2015-10-01 09:20 +0200
  [PATCH perf/core 3/5] [BUGFIX] perf-probe: Show correct source  lines of probes on kmodules Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> - 2015-09-30 18:50 +0200
    [tip:perf/core] perf probe:   Show correct source lines of probes on kmodules tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2015-10-01 09:20 +0200
  [PATCH perf/core 4/5] [BUGFIX] perf-probe: Fix a segfault bug in  debuginfo_cache Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> - 2015-09-30 18:50 +0200
    [tip:perf/core] perf probe: Fix a segfault bug in debuginfo_cache tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2015-10-01 09:20 +0200
  [PATCH perf/core 2/5] [BUGFIX] perf-probe: Fix to remove dot suffix  from second or latter events Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> - 2015-09-30 18:50 +0200
    [tip:perf/core] perf probe:   Fix to remove dot suffix from second or latter events tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2015-10-01 09:20 +0200
  [PATCH perf/core 1/5] [BUGFIX] perf-probe: Begin and end report  session correctly Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> - 2015-09-30 18:50 +0200
    [tip:perf/core] perf probe:   Begin and end libdwfl report session correctly tip-bot for Masami Hiramatsu <tipbot@zytor.com> - 2015-10-01 09:20 +0200
  Re: [PATCH perf/core  0/5] perf-probe bugfixes Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-09-30 22:20 +0200

csiph-web