LKML Archive on lore.kernel.org help / color / mirror / Atom feed
From: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> To: Naohiro Aota <naota@elisp.net> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>, Arnaldo Carvalho de Melo <acme@kernel.org>, Namhyung Kim <namhyung@kernel.org>, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] perf probe: export get_real_path Date: Thu, 26 Feb 2015 16:50:22 +0900 [thread overview] Message-ID: <54EED03E.9030601@hitachi.com> (raw) In-Reply-To: <1424934769-19481-1-git-send-email-naota@elisp.net> (2015/02/26 16:12), Naohiro Aota wrote: > Export it to use from util/probe-finder.c Please fold this in to the next patch, since this exported symbol is not used until applying the next one. BTW, since get_real_path is compiled only when HAVE_DWARF_SUPPORT=y, we can also move it into probe-finder.c. Could you also move it into probe-finder.c and export it at probe-finder.h? Thank you, > > Signed-off-by: Naohiro Aota <naota@elisp.net> > --- > tools/perf/util/probe-event.c | 2 +- > tools/perf/util/probe-event.h | 2 ++ > 2 files changed, 3 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/util/probe-event.c b/tools/perf/util/probe-event.c > index 919937e..1d0d505 100644 > --- a/tools/perf/util/probe-event.c > +++ b/tools/perf/util/probe-event.c > @@ -520,7 +520,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 e01e994..30a3391 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 > -- Masami HIRAMATSU Software Platform Research Dept. Linux Technology Research Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com
next prev parent reply other threads:[~2015-02-26 7:50 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-02-26 7:12 [PATCH 1/2] perf probe: export get_real_path Naohiro Aota 2015-02-26 7:12 ` [PATCH 2/2] perf probe: Find compilation directory path for lazy matching Naohiro Aota 2015-02-26 8:08 ` Masami Hiramatsu 2015-02-26 8:25 ` [PATCH perf/core ] [BUGFIX] perf-probe: Fix get_real_path to free allocated memory in error path Masami Hiramatsu 2015-02-26 14:46 ` Arnaldo Carvalho de Melo 2015-02-27 0:58 ` Masami Hiramatsu 2015-02-28 9:31 ` [tip:perf/core] perf probe: " tip-bot for Masami Hiramatsu 2015-02-26 7:50 ` Masami Hiramatsu [this message] 2015-03-04 7:52 ` [PATCH v2] perf probe: Find compilation directory path for lazy matching Naohiro Aota 2015-03-04 12:34 ` Masami Hiramatsu 2015-03-11 13:30 ` Arnaldo Carvalho de Melo 2015-03-12 1:42 ` Masami Hiramatsu 2015-03-13 5:13 ` Naohiro Aota 2015-03-13 5:18 ` [PATCH v3] " Naohiro Aota 2015-03-13 12:21 ` Masami Hiramatsu 2015-04-13 23:10 ` Arnaldo Carvalho de Melo 2015-04-14 6:35 ` Masami Hiramatsu 2015-04-14 12:17 ` [tip:perf/urgent] " tip-bot for Naohiro Aota
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=54EED03E.9030601@hitachi.com \ --to=masami.hiramatsu.pt@hitachi.com \ --cc=a.p.zijlstra@chello.nl \ --cc=acme@kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=namhyung@kernel.org \ --cc=naota@elisp.net \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).