From: Jiri Olsa <jolsa@redhat.com>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Jiri Olsa <jolsa@kernel.org>, lkml <linux-kernel@vger.kernel.org>,
Ingo Molnar <mingo@kernel.org>,
Namhyung Kim <namhyung@kernel.org>,
Alexander Shishkin <alexander.shishkin@linux.intel.com>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Michael Petlan <mpetlan@redhat.com>
Subject: Re: [PATCH 1/3] libperf: Move libperf under tools/lib/perf
Date: Fri, 6 Dec 2019 16:07:06 +0100 [thread overview]
Message-ID: <20191206150706.GD31721@krava> (raw)
In-Reply-To: <20191206142754.GC30698@kernel.org>
On Fri, Dec 06, 2019 at 11:27:54AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Fri, Dec 06, 2019 at 02:55:11PM +0100, Jiri Olsa escreveu:
> > Moving libperf from its current location under perf
> > to separate directory under tools/lib.
>
> Breaks the build/bisection:
yes, I noted that in the cover email, the 2nd patch repairs paths,
but those changes would get lost in the move.. I can squash it
if you want, but I thought this is more transparent despite the
one-commit-long broken bisect ;-)
jirka
>
> [acme@quaco perf]$ rm -rf /tmp/build/perf ; mkdir -p /tmp/build/perf ; make O=/tmp/build/perf -C tools/perf install-bin
> make: Entering directory '/home/acme/git/perf/tools/perf'
> BUILD: Doing 'make -j8' parallel build
> HOSTCC /tmp/build/perf/fixdep.o
> HOSTLD /tmp/build/perf/fixdep-in.o
> LINK /tmp/build/perf/fixdep
>
> Auto-detecting system features:
> ... dwarf: [ on ]
> ... dwarf_getlocations: [ on ]
> ... glibc: [ on ]
> ... gtk2: [ on ]
> ... libaudit: [ on ]
> ... libbfd: [ on ]
> ... libcap: [ on ]
> ... libelf: [ on ]
> ... libnuma: [ on ]
> ... numa_num_possible_cpus: [ on ]
> ... libperl: [ on ]
> ... libpython: [ on ]
> ... libcrypto: [ on ]
> ... libunwind: [ on ]
> ... libdw-dwarf-unwind: [ on ]
> ... zlib: [ on ]
> ... lzma: [ on ]
> ... get_cpuid: [ on ]
> ... bpf: [ on ]
> ... libaio: [ on ]
> ... libzstd: [ on ]
> ... disassembler-four-args: [ on ]
>
> GEN /tmp/build/perf/common-cmds.h
> make[3]: *** /home/acme/git/perf/tools/perf/lib/: No such file or directory. Stop.
> make[2]: *** [Makefile.perf:785: /tmp/build/perf/libperf.a] Error 2
> make[2]: *** Waiting for unfinished jobs....
> MKDIR /tmp/build/perf/fd/
> MKDIR /tmp/build/perf/fs/
> CC /tmp/build/perf/exec-cmd.o
> CC /tmp/build/perf/fd/array.o
> CC /tmp/build/perf/fs/fs.o
> CC /tmp/build/perf/cpu.o
> CC /tmp/build/perf/help.o
> LD /tmp/build/perf/fd/libapi-in.o
> CC /tmp/build/perf/event-parse.o
> CC /tmp/build/perf/event-plugin.o
> CC /tmp/build/perf/pager.o
> CC /tmp/build/perf/debug.o
> CC /tmp/build/perf/str_error_r.o
> CC /tmp/build/perf/trace-seq.o
> MKDIR /tmp/build/perf/fs/
> CC /tmp/build/perf/parse-filter.o
> CC /tmp/build/perf/parse-options.o
> MKDIR /tmp/build/perf/staticobjs/
> CC /tmp/build/perf/fs/tracing_path.o
> CC /tmp/build/perf/run-command.o
> CC /tmp/build/perf/staticobjs/libbpf.o
> CC /tmp/build/perf/parse-utils.o
> CC /tmp/build/perf/kbuffer-parse.o
> MKDIR /tmp/build/perf/staticobjs/
> CC /tmp/build/perf/sigchain.o
> LD /tmp/build/perf/fs/libapi-in.o
> CC /tmp/build/perf/staticobjs/bpf.o
> CC /tmp/build/perf/subcmd-config.o
> CC /tmp/build/perf/staticobjs/nlattr.o
> CC /tmp/build/perf/tep_strerror.o
> CC /tmp/build/perf/event-parse-api.o
> CC /tmp/build/perf/staticobjs/btf.o
> LD /tmp/build/perf/libapi-in.o
> LD /tmp/build/perf/libsubcmd-in.o
> CC /tmp/build/perf/staticobjs/libbpf_errno.o
> CC /tmp/build/perf/staticobjs/str_error.o
> CC /tmp/build/perf/staticobjs/netlink.o
> CC /tmp/build/perf/staticobjs/bpf_prog_linfo.o
> AR /tmp/build/perf/libapi.a
> LD /tmp/build/perf/libtraceevent-in.o
> CC /tmp/build/perf/staticobjs/libbpf_probes.o
> CC /tmp/build/perf/staticobjs/xsk.o
> AR /tmp/build/perf/libsubcmd.a
> CC /tmp/build/perf/staticobjs/btf_dump.o
> CC /tmp/build/perf/staticobjs/hashmap.o
> LINK /tmp/build/perf/libtraceevent.a
> LD /tmp/build/perf/staticobjs/libbpf-in.o
> LINK /tmp/build/perf/libbpf.a
> PERF_VERSION = 5.4.ge59599b355da
> make[1]: *** [Makefile.perf:225: sub-make] Error 2
> make: *** [Makefile:110: install-bin] Error 2
> make: Leaving directory '/home/acme/git/perf/tools/perf'
> [acme@quaco perf]$
>
next prev parent reply other threads:[~2019-12-06 15:07 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-06 13:55 [PATCH 0/3] perf/libperf move Jiri Olsa
2019-12-06 13:55 ` [PATCH 1/3] libperf: Move libperf under tools/lib/perf Jiri Olsa
2019-12-06 14:27 ` Arnaldo Carvalho de Melo
2019-12-06 15:07 ` Jiri Olsa [this message]
2019-12-06 15:33 ` Arnaldo Carvalho de Melo
2019-12-06 15:38 ` Jiri Olsa
2019-12-06 13:55 ` [PATCH 2/3] libperf: Additional fixes Jiri Olsa
2019-12-06 13:55 ` [PATCH 3/3] libperf: Add man pages Jiri Olsa
2019-12-06 14:21 ` [PATCH 0/3] perf/libperf move Arnaldo Carvalho de Melo
2019-12-06 14:34 ` Jiri Olsa
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20191206150706.GD31721@krava \
--to=jolsa@redhat.com \
--cc=a.p.zijlstra@chello.nl \
--cc=acme@kernel.org \
--cc=alexander.shishkin@linux.intel.com \
--cc=jolsa@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=mpetlan@redhat.com \
--cc=namhyung@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox