linux-kernel - Re: [PATCHv2 00/15] tools build: Move features framework into tools/build

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 18 Mar 2015 19:59:16 +0100
From:	Jiri Olsa <jolsa@...hat.com>
To:	Arnaldo Carvalho de Melo <acme@...hat.com>
Cc:	Jiri Olsa <jolsa@...nel.org>, linux-kernel@...r.kernel.org,
	Corey Ashford <cjashfor@...ux.vnet.ibm.com>,
	David Ahern <david.ahern@...cle.com>,
	Ingo Molnar <mingo@...nel.org>,
	Namhyung Kim <namhyung@...nel.org>,
	Paul Mackerras <paulus@...ba.org>,
	Peter Zijlstra <peterz@...radead.org>
Subject: Re: [PATCHv2 00/15] tools build: Move features framework into
 tools/build

On Wed, Mar 18, 2015 at 03:50:48PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Wed, Mar 11, 2015 at 09:58:49AM +0100, Jiri Olsa escreveu:
> > hi,
> > moving features detection framework from perf to 'tools/build'
> > plus other minor fixes.
> > 
> > v2 changes:
> >   - omitted patch 1 from original version (Ingo)
> >   - added features build output dump (Arnaldo)
> >   - better scripting in features test (Namhyung)
> >   - fixed test-all for pthread-attr-setaffinity-np
> >   - removed main_test_libbabeltrace from test-all
> >   - patches reordered
> > 
> > It's also available in:
> >   git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf.git
> >   perf/build
> 
> Ok, so using the csets from the above branch, if I do:
> 
> [acme@...andy linux]$ rm -rf /tmp/build/perf ; mkdir -p /tmp/build/perf ; time make O=/tmp/build/perf -C tools/perf LIBBABELTRACE_DIR=/opt/libbabeltrace/ install-bin
> make: Entering directory `/home/acme/git/linux/tools/perf'
>   BUILD:   Doing 'make -j8' parallel build
> config/Makefile:423: No libunwind found. Please install libunwind-dev[el] >= 1.1 and/or set LIBUNWIND_DIR
> 
> 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: [ OFF ]
> ...            libdw-dwarf-unwind: [ on  ]
> ...                          zlib: [ on  ]
> ...     DWARF post unwind library: libdw
> 
> And then, despite not telling anything about babeltrace being linked,
> i.e. the no line above with a [ on ], we get:

libbabeltrace was taken out of the default display
of features as requested, because the requested
interface wasn't released yet

you enabled it manually via LIBBABELTRACE_DIR='...',
so it got linked.. but it wont appear in the default
display list anyway.. we dont have this kind of
functionality yet

jirka

> 
> [acme@...andy linux]$ ldd ~/bin/perf | grep babel
> 	libbabeltrace-ctf.so.1 => /opt/libbabeltrace/lib/libbabeltrace-ctf.so.1 (0x00007f5f7b758000)
> 	libbabeltrace.so.1 => /opt/libbabeltrace/lib/libbabeltrace.so.1 (0x00007f5f7acfb000)
> [acme@...andy linux]$
> 
> I.e. if a libbabeltrace-devel was installed and had what is needed to build it
> in, when there would be no need for the LIBBABELTRACE_DIR above and babeltrace
> wouldn't be either implicitely or explicitely specified as something we want to
> build against we would not show it as a feature being built in (no
> libbabeltrace: [ On ]) but would do it anyway, is that really the intention?
> 
> I tested in fedora, where no system auto-detection fails, i.e. test-all passes,
> and the result is the same.
> 
> Ingo, can you clarify here?
> 
> - Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/