Re: [PATCH 09/11] perf python: Link with libtraceevent

From: Namhyung Kim
Date: Wed Oct 17 2012 - 21:38:26 EST


On Wed, 17 Oct 2012 14:19:45 -0300, Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
>
> The evsel methods to read tracepoint fields uses libtraceevent
> functions, becoming needed by the python binding as well.

I got a build error after adding 'extern unsigned int page_size' to
util/python.c - please see my previous reply.

namhyung@sejong:perf$ make
SUBDIR ../lib/traceevent/
LINK perf
GEN python/perf.so
gcc: error: python_ext_build/tmp//../../libtraceevent.a: No such file or directory
error: command 'gcc' failed with exit status 1
cp: cannot stat `python_ext_build/lib/perf.so': No such file or directory
make: *** [python/perf.so] Error 1

The libtraceevent.a will be in
python_ext_build/tmp/../../../lib/traceevent/libtraceevent.a if make was
invoked without the O.

Makefile contains following:

TRACE_EVENT_DIR = ../lib/traceevent/

ifneq ($(OUTPUT),)
TE_PATH=$(OUTPUT)
else
TE_PATH=$(TRACE_EVENT_DIR)
endif

LIBTRACEEVENT = $(TE_PATH)libtraceevent.a
TE_LIB := -L$(TE_PATH) -ltraceevent

Thanks,
Namhyung

>
> Cc: David Ahern <dsahern@xxxxxxxxx>
> Cc: Frederic Weisbecker <fweisbec@xxxxxxxxx>
> Cc: Jiri Olsa <jolsa@xxxxxxxxxx>
> Cc: Mike Galbraith <efault@xxxxxx>
> Cc: Namhyung Kim <namhyung@xxxxxxxxx>
> Cc: Paul Mackerras <paulus@xxxxxxxxx>
> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Cc: Stephane Eranian <eranian@xxxxxxxxxx>
> Link: http://lkml.kernel.org/n/tip-j3o4v7jyvp9ke9n230l96a1m@xxxxxxxxxxxxxx
> Signed-off-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
> ---
> tools/perf/util/setup.py | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/tools/perf/util/setup.py b/tools/perf/util/setup.py
> index d0f9f29..09c3cea 100644
> --- a/tools/perf/util/setup.py
> +++ b/tools/perf/util/setup.py
> @@ -31,6 +31,7 @@ perf = Extension('perf',
> sources = ext_sources,
> include_dirs = ['util/include'],
> extra_compile_args = cflags,
> + extra_objects = [build_tmp + '/../../libtraceevent.a'],
> )
>
> setup(name='perf',
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/