* Jiri Olsa <jo...@redhat.com> wrote: > On Thu, Dec 05, 2013 at 10:25:02AM +0100, Ingo Molnar wrote: > > > > * Jiri Olsa <jo...@redhat.com> wrote: > > > > > On Wed, Dec 04, 2013 at 03:50:24PM -0300, Arnaldo Carvalho de Melo wrote: > > > > Em Tue, Dec 03, 2013 at 02:09:42PM +0100, Jiri Olsa escreveu: > > > > > Adding udis86 disassembler feature check which support > > > > > is needed for kvm:kvm_emulate_insn tracepoint. > > > > > > > > > > +$(call feature_check,udis86) > > > > > +ifeq ($(feature-udis86), 1) > > > > > + LIBTRACEEVENT_CFLAGS += -DHAVE_UDIS86 > > > > > + EXTLIBS += -ludis86 > > > > > +else > > > > > + msg := $(warning No udis86 support.); > > > > > +endif > > > > > > > > That is really an incomplete message, what package should I install? > > > > Perhaps we should add this there then: > > > > > > > > http://bit.ly/1hyrN52 > > > > > > nice :-) > > > > > > > > > > > Wow, that was easy, but yeah, could be made easier 8-) ;-P > > > > > > so something like: > > > No udis86 found. Please install udis86-devel. > > > > > > or: > > > No udis86 found, disabling kvm tracepoints instruction disassembly. > > > Please install udis86-devel. > > > > 1) > > > > So, the first problem I can see is in the output: > > > > Auto-detecting system features: > > ... backtrace: [ on ] > > ... dwarf: [ on ] > > ... fortify-source: [ on ] > > ... glibc: [ on ] > > ... gtk2: [ on ] > > ... gtk2-infobar: [ on ] > > ... libaudit: [ on ] > > ... libbfd: [ on ] > > ... libelf: [ on ] > > ... libelf-getphdrnum: [ on ] > > ... libelf-mmap: [ on ] > > ... libnuma: [ on ] > > ... libperl: [ on ] > > ... libpython: [ on ] > > ... libpython-version: [ on ] > > ... libslang: [ on ] > > ... libunwind: [ on ] > > ... on-exit: [ on ] > > ... stackprotector-all: [ on ] > > ... timerfd: [ on ] > > > > config/Makefile:421: No udis86 support. > > > > such a 'no XYZ support' message should only occur if a feature test > > failed. _If_ we decide that 'udis86' support is required for a full > > perf build then there should be a new line saying something like: > > > > ... udis86: [ OFF ] > > hum, just sent v4 that added udis86 into above list ;-) > > So.. do we consider 'full perf build' to be build with all > possible libraries linked in?
So my worry is, how widely is libudis86 available in distros? What is the package name on Debian/Ubuntu for example? Thanks, Ingo -- 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/