Em Fri, Sep 27, 2013 at 04:32:12PM +0200, Jiri Olsa escreveu:
> When building minimal perf via:
> NO_LIBPERL=1 NO_LIBPYTHON=1 NO_NEWT=1 NO_GTK2=1 NO_DEMANGLE=1
> NO_LIBELF=1 NO_LIBUNWIND=1 NO_BACKTRACE=1 NO_LIBNUMA=1 NO_LIBAUDIT=1
> NO_LIBBIONIC=1
>
> We dont link -ldl and that's causing error for gtk2 related code,
> which is disabled on command line (NO_GTK2), but not in the code
> (GTK2_SUPPORT).
>
> We probably need some global fix/change for this browser
> enabled/disable code.
But for now what you did is enough, since the only module that uses
the dl mechanism is the GTK+ one, so I'm applying it.
I should have detected this before pushing out perf/core with the
changes that introduced this problem, that is what:
make -C tools/perf -f tests/make
was made to catch, my bad, Namhyung, please use those tests next time
too :-)
> Tested just by building it corretly.
Yeah, now tests/make completes without problems.
Thanks,
- Arnaldo
> Signed-off-by: Jiri Olsa <jo...@redhat.com>
> Cc: Corey Ashford <cjash...@linux.vnet.ibm.com>
> Cc: Ingo Molnar <mi...@elte.hu>
> Cc: Namhyung Kim <namhy...@kernel.org>
> Cc: Paul Mackerras <pau...@samba.org>
> Cc: Peter Zijlstra <a.p.zijls...@chello.nl>
> Cc: Arnaldo Carvalho de Melo <a...@redhat.com>
> Cc: David Ahern <dsah...@gmail.com>
> ---
> tools/perf/builtin-annotate.c | 6 ++++++
> tools/perf/builtin-report.c | 2 ++
> 2 files changed, 8 insertions(+)
>
> diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c
> index 577c3a1..ddde407 100644
> --- a/tools/perf/builtin-annotate.c
> +++ b/tools/perf/builtin-annotate.c
> @@ -142,6 +142,7 @@ find_next:
> }
>
> if (use_browser == 2) {
> +#ifdef GTK2_SUPPORT
> int ret;
> int (*annotate)(struct hist_entry *he,
> struct perf_evsel *evsel,
> @@ -161,6 +162,9 @@ find_next:
> /* skip missing symbols */
> nd = rb_next(nd);
> } else if (use_browser == 1) {
> +#else
> + use_browser = 1;
> +#endif
> key = hist_entry__tui_annotate(he, evsel, NULL);
> switch (key) {
> case -1:
> @@ -258,6 +262,7 @@ static int __cmd_annotate(struct perf_annotate *ann)
> goto out_delete;
> }
>
> +#ifdef GTK2_SUPPORT
> if (use_browser == 2) {
> void (*show_annotations)(void);
>
> @@ -269,6 +274,7 @@ static int __cmd_annotate(struct perf_annotate *ann)
> }
> show_annotations();
> }
> +#endif
>
> out_delete:
> /*
> diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
> index 295025e..c902229 100644
> --- a/tools/perf/builtin-report.c
> +++ b/tools/perf/builtin-report.c
> @@ -591,6 +591,7 @@ static int __cmd_report(struct perf_report *rep)
> ret = 0;
>
> } else if (use_browser == 2) {
> +#ifdef GTK2_SUPPORT
> int (*hist_browser)(struct perf_evlist *,
> const char *,
> struct hist_browser_timer *,
> @@ -604,6 +605,7 @@ static int __cmd_report(struct perf_report *rep)
> }
> hist_browser(session->evlist, help, NULL,
> rep->min_percent);
> +#endif
> }
> } else
> perf_evlist__tty_browse_hists(session->evlist, rep, help);
> --
> 1.7.11.7
--
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/
