lkml.org 
[lkml]   [2014]   [Apr]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [GIT PULL 0/4] perf/urgent fixes

* Jiri Olsa <jolsa@redhat.com> wrote:

> On Wed, Apr 23, 2014 at 03:14:33PM +0200, Ingo Molnar wrote:
> >
> > (Just reporting two bugs I found today - unrelated to your the
> > perf/urgent pull request.)
> >
> > 1)
> >
> > Even when the most modern unwind library is found, the autodetection
> > is spammy:
> >
> >
> > Auto-detecting system features:
> > ... dwarf: [ on ]
> > ... glibc: [ on ]
> > ... gtk2: [ on ]
> > ... libaudit: [ on ]
> > ... libbfd: [ on ]
> > ... libelf: [ on ]
> > ... libnuma: [ on ]
> > ... libperl: [ on ]
> > ... libpython: [ on ]
> > ... libslang: [ on ]
> > ... libunwind: [ on ]
> > ... libdw-dwarf-unwind: [ on ]
> > ... DWARF post unwind library: libunwind
> >
> > The 'DWARF post unwind library' line is somewhat superfluous. I
> > realize that it prints out the library selected - but that's obvious
> > from the 'libdw-dwarf-unwind' line above it already, right?
> >
> > Furthermore, it breaks the autodetection output format.
> >
> > 2)
> >
> > On latest Ubuntu (14.04) the tip:master build fails with:
> >
> > /usr/bin/ld: cannot find -liberty
> > collect2: error: ld returned 1 exit status
> >
> > The autodetection sequence reports all green entries, so something's
> > funky going on there.
>
> we add -liberty once -lbfd is detected, I guess we
> assumed that was common case
>
> perf tools: fix BFD detection on opensuse
> commit 280e7c48c3b873e4987a63da276ecab25383f494
> Author: Andi Kleen <ak@linux.intel.com>
> Date: Sat Jan 11 11:42:51 2014 -0800
>
> could you please try patch below? it adds that only
> if thats detected
>
> 'make VF=1' should display more status now
>
> Andi,
> could you please try that on opensuse?
>
> thanks,
> jirka
>
>
> ---
> diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile
> index ee21fa9..f511658 100644
> --- a/tools/perf/config/Makefile
> +++ b/tools/perf/config/Makefile
> @@ -186,7 +186,10 @@ VF_FEATURE_TESTS = \
> stackprotector-all \
> timerfd \
> libunwind-debug-frame \
> - bionic
> + bionic \
> + liberty \
> + liberty-z \
> + cplus-demangle

In any case, your patch fixes the bug. With VF=1 I get this output:

Auto-detecting system features:
... dwarf: [ on ]
... glibc: [ on ]
... gtk2: [ on ]
... libaudit: [ on ]
... libbfd: [ on ]
... libelf: [ on ]
... libnuma: [ on ]
... libperl: [ on ]
... libpython: [ on ]
... libslang: [ on ]
... libunwind: [ on ]
... libdw-dwarf-unwind: [ on ]
... DWARF post unwind library: libunwind
... backtrace: [ on ]
... fortify-source: [ on ]
... gtk2-infobar: [ on ]
... libelf-getphdrnum: [ on ]
... libelf-mmap: [ on ]
... libpython-version: [ on ]
... on-exit: [ on ]
... stackprotector-all: [ on ]
... timerfd: [ on ]
... libunwind-debug-frame: [ OFF ]
... bionic: [ OFF ]
... liberty: [ OFF ]
... liberty-z: [ OFF ]
... cplus-demangle: [ OFF ]

So yes, your obervation that it's the -liberty +libbfd combination is
correct.

Tested-by: Ingo Molnar <mingo@kernel.org>

Btw., when reading the patch and the Makefile it was not obvious to me
what 'VF' stood for. It's pretty clear what CORE_FEATURE_TESTS and
LIB_FEATURE_TESTS means, but there's no comment for VF_FEATURE_TESTS
and the name is not self-explanatory.

I figured it out from a bit of git log digging that its purpose is to
generate the 'verbose feature check' output. But the variable is not
commented and the features it lists overlaps CORE_FEATURE_TESTS and
LIB_FEATURE_TESTS - so perhaps a bit more explanation (and possible
reduction in duplication) might be useful?

If it said ALL_FEATURE_TESTS and used $(CORE_FEATURE_TESTS) as a
baseline then that would be self-explanatory.

(In a separate patch from the fix.)

Thanks,

Ingo


\
 
 \ /
  Last update: 2014-04-24 09:01    [W:0.225 / U:0.416 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site