[PATCH 2/3] perf probe: Make --source avaiable when probe with lazy_line

[He Kuang]

Use get_real_path() to enable --source option when probe with lazy_line
pattern.

Before this patch:

  $ perf probe -s ./kernel_src/ -k ./vmlinux --add='fs/super.c;s->s_count=1;'
  Failed to open fs/super.c: No such file or directory
    Error: Failed to add events.

After this patch:

  $ perf probe -s ./kernel_src/ -k ./vmlinux  --add='fs/super.c;s->s_count=1;'
  Added new events:
    probe:_stext         (on @fs/super.c)
    probe:_stext_1       (on @fs/super.c)
  ...

Signed-off-by: He Kuang <heku...@huawei.com>
---
 tools/perf/util/probe-event.c  |  2 +-
 tools/perf/util/probe-event.h  |  2 ++
 tools/perf/util/probe-finder.c | 18 +++++++++++++++---
 3 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c
index 5483d98..35ee51a 100644
--- a/tools/perf/util/probe-event.c
+++ b/tools/perf/util/probe-event.c
@@ -661,7 +661,7 @@ static int try_to_find_probe_trace_events(struct 
perf_probe_event *pev,
  * a newly allocated path on success.
  * Return 0 if file was found and readable, -errno otherwise.
  */
-static int get_real_path(const char *raw_path, const char *comp_dir,
+int get_real_path(const char *raw_path, const char *comp_dir,
                         char **new_path)
 {
        const char *prefix = symbol_conf.source_prefix;
diff --git a/tools/perf/util/probe-event.h b/tools/perf/util/probe-event.h
index d6b7834..21809ea 100644
--- a/tools/perf/util/probe-event.h
+++ b/tools/perf/util/probe-event.h
@@ -135,6 +135,8 @@ extern int show_available_vars(struct perf_probe_event 
*pevs, int npevs,
                               struct strfilter *filter, bool externs);
 extern int show_available_funcs(const char *module, struct strfilter *filter,
                                bool user);
+extern int get_real_path(const char *raw_path, const char *comp_dir,
+                       char **new_path);
 
 /* Maximum index number of event-name postfix */
 #define MAX_EVENT_INDEX        1024
diff --git a/tools/perf/util/probe-finder.c b/tools/perf/util/probe-finder.c
index 7831e2d..431c12d 100644
--- a/tools/perf/util/probe-finder.c
+++ b/tools/perf/util/probe-finder.c
@@ -791,11 +791,20 @@ static int find_lazy_match_lines(struct intlist *list,
        ssize_t len;
        int count = 0, linenum = 1;
        char sbuf[STRERR_BUFSIZE];
+       char *realname = NULL;
+       int ret;
 
-       fp = fopen(fname, "r");
+       ret = get_real_path(fname, NULL, &realname);
+       if (ret < 0) {
+               pr_warning("Failed to find source file %s.\n", fname);
+               return ret;
+       }
+
+       fp = fopen(realname, "r");
        if (!fp) {
-               pr_warning("Failed to open %s: %s\n", fname,
+               pr_warning("Failed to open %s: %s\n", realname,
                           strerror_r(errno, sbuf, sizeof(sbuf)));
+               free(realname);
                return -errno;
        }
 
@@ -817,7 +826,10 @@ static int find_lazy_match_lines(struct intlist *list,
        fclose(fp);
 
        if (count == 0)
-               pr_debug("No matched lines found in %s.\n", fname);
+               pr_debug("No matched lines found in %s.\n", realname);
+
+       free(realname);
+
        return count;
 }
 
-- 
2.3.3.220.g9ab698f

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