* [PATCH] perf tools: Add proper header file for symbols
@ 2019-02-08 22:35 ` Mathieu Poirier
0 siblings, 0 replies; 6+ messages in thread
From: Mathieu Poirier @ 2019-02-08 22:35 UTC (permalink / raw)
To: acme; +Cc: suzuki.poulose, peterz, linux-kernel, mingo, jolsa,
linux-arm-kernel
After 'commit e22c1c751140 ("perf thread: Don't include symbol.h,
symbol_conf.h is enough")'
Compilation of the perf tools is broken when using the functionality
provided by the openCSD library:
[...]
... timerfd: [ on ]
... sched_getcpu: [ on ]
... sdt: [ OFF ]
... setns: [ on ]
... libopencsd: [ on ]
[...]
CC util/arm-spe.o
CC util/arm-spe-pkt-decoder.o
CC util/s390-cpumsf.o
CC util/cs-etm.o
CC util/parse-branch-options.o
util/cs-etm.c: In function ‘cs_etm__mem_access’:
util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
struct addr_location al;
And rightly so since file cs-etm.c doesn't include symbol.h, something
that is rectified in this patch.
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
---
tools/perf/util/cs-etm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
index 8b3f882d6e2f..0b11d653cfbe 100644
--- a/tools/perf/util/cs-etm.c
+++ b/tools/perf/util/cs-etm.c
@@ -25,6 +25,7 @@
#include "machine.h"
#include "map.h"
#include "perf.h"
+#include "symbol.h"
#include "thread.h"
#include "thread_map.h"
#include "thread-stack.h"
--
2.17.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH] perf tools: Add proper header file for symbols
@ 2019-02-08 22:35 ` Mathieu Poirier
0 siblings, 0 replies; 6+ messages in thread
From: Mathieu Poirier @ 2019-02-08 22:35 UTC (permalink / raw)
To: acme; +Cc: suzuki.poulose, peterz, mingo, jolsa, linux-arm-kernel,
linux-kernel
After 'commit e22c1c751140 ("perf thread: Don't include symbol.h,
symbol_conf.h is enough")'
Compilation of the perf tools is broken when using the functionality
provided by the openCSD library:
[...]
... timerfd: [ on ]
... sched_getcpu: [ on ]
... sdt: [ OFF ]
... setns: [ on ]
... libopencsd: [ on ]
[...]
CC util/arm-spe.o
CC util/arm-spe-pkt-decoder.o
CC util/s390-cpumsf.o
CC util/cs-etm.o
CC util/parse-branch-options.o
util/cs-etm.c: In function ‘cs_etm__mem_access’:
util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
struct addr_location al;
And rightly so since file cs-etm.c doesn't include symbol.h, something
that is rectified in this patch.
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
---
tools/perf/util/cs-etm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
index 8b3f882d6e2f..0b11d653cfbe 100644
--- a/tools/perf/util/cs-etm.c
+++ b/tools/perf/util/cs-etm.c
@@ -25,6 +25,7 @@
#include "machine.h"
#include "map.h"
#include "perf.h"
+#include "symbol.h"
#include "thread.h"
#include "thread_map.h"
#include "thread-stack.h"
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] perf tools: Add proper header file for symbols
2019-02-08 22:35 ` Mathieu Poirier
@ 2019-02-11 11:46 ` Arnaldo Carvalho de Melo
-1 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-02-11 11:46 UTC (permalink / raw)
To: Mathieu Poirier
Cc: suzuki.poulose, peterz, linux-kernel, mingo, jolsa,
linux-arm-kernel
Em Fri, Feb 08, 2019 at 03:35:43PM -0700, Mathieu Poirier escreveu:
> After 'commit e22c1c751140 ("perf thread: Don't include symbol.h,
> symbol_conf.h is enough")'
>
> Compilation of the perf tools is broken when using the functionality
> provided by the openCSD library:
>
> [...]
>
> ... timerfd: [ on ]
> ... sched_getcpu: [ on ]
> ... sdt: [ OFF ]
> ... setns: [ on ]
> ... libopencsd: [ on ]
>
> [...]
>
> CC util/arm-spe.o
> CC util/arm-spe-pkt-decoder.o
> CC util/s390-cpumsf.o
> CC util/cs-etm.o
> CC util/parse-branch-options.o
> util/cs-etm.c: In function ‘cs_etm__mem_access’:
> util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
> struct addr_location al;
>
> And rightly so since file cs-etm.c doesn't include symbol.h, something
> that is rectified in this patch.
Thanks, I'll make sure the openCSD library is present in one of my
containers doing arm64 cross builds so that this gets caught in the
future.
- Arnaldo
> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---
> tools/perf/util/cs-etm.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
> index 8b3f882d6e2f..0b11d653cfbe 100644
> --- a/tools/perf/util/cs-etm.c
> +++ b/tools/perf/util/cs-etm.c
> @@ -25,6 +25,7 @@
> #include "machine.h"
> #include "map.h"
> #include "perf.h"
> +#include "symbol.h"
> #include "thread.h"
> #include "thread_map.h"
> #include "thread-stack.h"
> --
> 2.17.1
--
- Arnaldo
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] perf tools: Add proper header file for symbols
@ 2019-02-11 11:46 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-02-11 11:46 UTC (permalink / raw)
To: Mathieu Poirier
Cc: suzuki.poulose, peterz, mingo, jolsa, linux-arm-kernel,
linux-kernel
Em Fri, Feb 08, 2019 at 03:35:43PM -0700, Mathieu Poirier escreveu:
> After 'commit e22c1c751140 ("perf thread: Don't include symbol.h,
> symbol_conf.h is enough")'
>
> Compilation of the perf tools is broken when using the functionality
> provided by the openCSD library:
>
> [...]
>
> ... timerfd: [ on ]
> ... sched_getcpu: [ on ]
> ... sdt: [ OFF ]
> ... setns: [ on ]
> ... libopencsd: [ on ]
>
> [...]
>
> CC util/arm-spe.o
> CC util/arm-spe-pkt-decoder.o
> CC util/s390-cpumsf.o
> CC util/cs-etm.o
> CC util/parse-branch-options.o
> util/cs-etm.c: In function ‘cs_etm__mem_access’:
> util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
> struct addr_location al;
>
> And rightly so since file cs-etm.c doesn't include symbol.h, something
> that is rectified in this patch.
Thanks, I'll make sure the openCSD library is present in one of my
containers doing arm64 cross builds so that this gets caught in the
future.
- Arnaldo
> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---
> tools/perf/util/cs-etm.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
> index 8b3f882d6e2f..0b11d653cfbe 100644
> --- a/tools/perf/util/cs-etm.c
> +++ b/tools/perf/util/cs-etm.c
> @@ -25,6 +25,7 @@
> #include "machine.h"
> #include "map.h"
> #include "perf.h"
> +#include "symbol.h"
> #include "thread.h"
> #include "thread_map.h"
> #include "thread-stack.h"
> --
> 2.17.1
--
- Arnaldo
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] perf tools: Add proper header file for symbols
2019-02-11 11:46 ` Arnaldo Carvalho de Melo
(?)
@ 2019-02-11 14:22 ` Arnaldo Carvalho de Melo
-1 siblings, 0 replies; 6+ messages in thread
From: Arnaldo Carvalho de Melo @ 2019-02-11 14:22 UTC (permalink / raw)
To: Mathieu Poirier
Cc: suzuki.poulose, peterz, linux-kernel, mingo, jolsa,
linux-arm-kernel
Em Mon, Feb 11, 2019 at 08:46:03AM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Fri, Feb 08, 2019 at 03:35:43PM -0700, Mathieu Poirier escreveu:
> > After 'commit e22c1c751140 ("perf thread: Don't include symbol.h,
> > symbol_conf.h is enough")'
> >
> > Compilation of the perf tools is broken when using the functionality
> > provided by the openCSD library:
> >
> > [...]
> >
> > ... timerfd: [ on ]
> > ... sched_getcpu: [ on ]
> > ... sdt: [ OFF ]
> > ... setns: [ on ]
> > ... libopencsd: [ on ]
> >
> > [...]
> >
> > CC util/arm-spe.o
> > CC util/arm-spe-pkt-decoder.o
> > CC util/s390-cpumsf.o
> > CC util/cs-etm.o
> > CC util/parse-branch-options.o
> > util/cs-etm.c: In function ‘cs_etm__mem_access’:
> > util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
> > struct addr_location al;
> >
> > And rightly so since file cs-etm.c doesn't include symbol.h, something
> > that is rectified in this patch.
>
> Thanks, I'll make sure the openCSD library is present in one of my
> containers doing arm64 cross builds so that this gets caught in the
> future.
So, without your patch:
17.52 ubuntu:19.04-x-arm64 : FAIL gcc (Ubuntu 8.2.0-19ubuntu1) 8.2.0
CC /tmp/build/perf/util/parse-branch-options.o
util/cs-etm.c: In function 'cs_etm__mem_access':
util/cs-etm.c:326:24: error: storage size of 'al' isn't known
struct addr_location al;
^~
util/cs-etm.c:326:24: error: unused variable 'al' [-Werror=unused-variable]
util/cs-etm.c: In function 'cs_etm__alloc_queue':
util/cs-etm.c:384:22: error: invalid application of 'sizeof' to incomplete type 'struct branch_stack'
size_t sz = sizeof(struct branch_stack);
^~~~~~
util/cs-etm.c:387:16: error: invalid application of 'sizeof' to incomplete type 'struct branch_entry'
sizeof(struct branch_entry);
^~~~~~
util/cs-etm.c: In function 'cs_etm__copy_last_branch_rb':
util/cs-etm.c:541:8: error: dereferencing pointer to incomplete type 'struct branch_stack'
bs_dst->nr = bs_src->nr;
^~
util/cs-etm.c:557:16: error: invalid application of 'sizeof' to incomplete type 'struct branch_entry'
sizeof(struct branch_entry) * nr);
^~~~~~
util/cs-etm.c:569:17: error: invalid application of 'sizeof' to incomplete type 'struct branch_entry'
sizeof(struct branch_entry) * etmq->last_branch_pos);
^~~~~~
util/cs-etm.c: In function 'cs_etm__update_last_branch_rb':
util/cs-etm.c:646:4: error: dereferencing pointer to incomplete type 'struct branch_entry'
be->from = cs_etm__last_executed_instr(etmq->prev_packet);
^~
CC /tmp/build/perf/util/dump-insn.o
util/cs-etm.c: In function 'cs_etm__synth_branch_sample':
util/cs-etm.c:792:23: error: field 'entries' has incomplete type
struct branch_entry entries;
^~~~~~~
util/cs-etm.c:820:5: error: field name not in record or union initializer
.from = sample.ip,
^
util/cs-etm.c:820:5: note: (near initialization for '(anonymous).entries')
util/cs-etm.c:821:5: error: field name not in record or union initializer
.to = sample.addr,
^
util/cs-etm.c:821:5: note: (near initialization for '(anonymous).entries')
cc1: all warnings being treated as errors
mv: cannot stat '/tmp/build/perf/util/.cs-etm.o.tmp': No such file or directory
make[4]: *** [/git/linux/tools/build/Makefile.build:97: /tmp/build/perf/util/cs-etm.o] Error 1
----------------------------------------------------
After your patch:
[root@quaco x-arm64]# time dm ubuntu:19.04-x-arm64
1 44.59 ubuntu:19.04-x-arm64 : Ok gcc (Ubuntu 8.2.0-19ubuntu1) 8.2.0
further output below, but now I have this in this ubuntu 19.04 arm64
cross build container:
[root@quaco x-arm64]# egrep -B2 -A2 'CSD|CROSS' Dockerfile
ENV ARCH arm64
ENV TARGET aarch64-linux-gnu
ENV CROSS_COMPILE=${TARGET}-
RUN apt-get -y update && \
apt-get -y install make gcc-${TARGET} g++-${TARGET} flex bison git && \
--
export PATH=$INSTALLDIR/bin:$PATH && \
export TARGETMACH=${TARGET} && \
export CROSS=${TARGET}- && \
export CC=${CROSS}gcc && \
export LD=${CROSS}ld && \
export AS=${CROSS}as && \
wget -q http://zlib.net/zlib-${ZLIB_VER}.tar.gz && \
wget -q https://fedorahosted.org/releases/e/l/elfutils/${ELFUTILS_VER}/elfutils-${ELFUTILS_VER}.tar.bz2 && \
--
cd .. && \
rm -rf elfutils-${ELFUTILS_VER} && \
git clone https://github.com/Linaro/OpenCSD.git && \
make -C OpenCSD/decoder/build/linux/ CROSS_COMPILE=${CROSS} install && \
rm -rf OpenCSD && \
apt-get -y remove wget bzip2 && \
apt-get -y clean && \
rm -rf /usr/share/doc /usr/share/gtk-doc /usr/share/locale /usr/share/man && \
unset TARGET INSTALLDIR TARGETMACH CROSS CC LD AS
RUN mkdir -m 777 -p /git /tmp/build/perf /tmp/build/objtool /tmp/build/linux && \
groupadd -r perfbuilder && \
[root@quaco x-arm64]#
[root@quaco x-arm64]# grep -B3 -A3 cs-etm /tmp/dm.log/ubuntu\:19.04-x-arm64
LINK /tmp/build/perf/plugin_hrtimer.so
CC /tmp/build/perf/arch/arm64/util/../../arm/util/auxtrace.o
LINK /tmp/build/perf/plugin_kmem.so
CC /tmp/build/perf/arch/arm64/util/../../arm/util/cs-etm.o
LINK /tmp/build/perf/plugin_kvm.so
CC /tmp/build/perf/libbpf_errno.o
LINK /tmp/build/perf/plugin_mac80211.so
--
MKDIR /tmp/build/perf/util/intel-pt-decoder/
GEN /tmp/build/perf/util/intel-pt-decoder/inat-tables.c
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-log.o
MKDIR /tmp/build/perf/util/cs-etm-decoder/
CC /tmp/build/perf/util/cs-etm-decoder/cs-etm-decoder.o
CC /tmp/build/perf/tests/hists_output.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-decoder.o
CC /tmp/build/perf/tests/hists_cumulate.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-insn-decoder.o
CC /tmp/build/perf/tests/python-use.o
LD /tmp/build/perf/util/cs-etm-decoder/libperf-in.o
MKDIR /tmp/build/perf/util/scripting-engines/
LD /tmp/build/perf/util/scripting-engines/libperf-in.o
CC /tmp/build/perf/tests/bp_signal.o
--
CC /tmp/build/perf/util/s390-cpumsf.o
CC /tmp/build/perf/tests/code-reading.o
CC /tmp/build/perf/tests/sample-parsing.o
CC /tmp/build/perf/util/cs-etm.o
CC /tmp/build/perf/tests/parse-no-sample-id-all.o
CC /tmp/build/perf/util/parse-branch-options.o
CC /tmp/build/perf/tests/kmod-path.o
--
CC /tmp/build/perf/trace/beauty/statx.o
CC /tmp/build/perf/arch/arm64/util/../../arm/util/auxtrace.o
CC /tmp/build/perf/ui/util.o
CC /tmp/build/perf/arch/arm64/util/../../arm/util/cs-etm.o
CC /tmp/build/perf/ui/hist.o
CC /tmp/build/perf/ui/stdio/hist.o
LD /tmp/build/perf/trace/beauty/libperf-in.o
--
CC /tmp/build/perf/util/thread-stack.o
CC /tmp/build/perf/util/auxtrace.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-pkt-decoder.o
CC /tmp/build/perf/util/cs-etm-decoder/cs-etm-decoder.o
CC /tmp/build/perf/util/intel-pt.o
CC /tmp/build/perf/util/intel-bts.o
CC /tmp/build/perf/tests/python-use.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-log.o
CC /tmp/build/perf/tests/bp_signal.o
LD /tmp/build/perf/util/cs-etm-decoder/libperf-in.o
CC /tmp/build/perf/util/arm-spe.o
CC /tmp/build/perf/util/intel-pt-decoder/intel-pt-decoder.o
CC /tmp/build/perf/tests/bp_signal_overflow.o
--
CC /tmp/build/perf/tests/bp_account.o
CC /tmp/build/perf/tests/wp.o
CC /tmp/build/perf/util/s390-cpumsf.o
CC /tmp/build/perf/util/cs-etm.o
CC /tmp/build/perf/util/parse-branch-options.o
CC /tmp/build/perf/tests/task-exit.o
CC /tmp/build/perf/tests/sw-clock.o
[root@quaco x-arm64]#
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 6+ messages in thread
* [tip:perf/core] perf cs-etm: Add proper header file for symbols
2019-02-08 22:35 ` Mathieu Poirier
(?)
(?)
@ 2019-02-15 9:25 ` tip-bot for Mathieu Poirier
-1 siblings, 0 replies; 6+ messages in thread
From: tip-bot for Mathieu Poirier @ 2019-02-15 9:25 UTC (permalink / raw)
To: linux-tip-commits
Cc: suzuki.poulose, hpa, linux-kernel, tglx, mingo, acme, peterz,
mathieu.poirier, jolsa
Commit-ID: 859dcf64389c93a647f230a7cfd206d30bc9d286
Gitweb: https://git.kernel.org/tip/859dcf64389c93a647f230a7cfd206d30bc9d286
Author: Mathieu Poirier <mathieu.poirier@linaro.org>
AuthorDate: Fri, 8 Feb 2019 15:35:43 -0700
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Thu, 14 Feb 2019 13:30:52 -0300
perf cs-etm: Add proper header file for symbols
After 'commit e22c1c751140 ("perf thread: Don't include symbol.h,
symbol_conf.h is enough")'
Compilation of the perf tools is broken when using the functionality
provided by the openCSD library:
[...]
... timerfd: [ on ]
... sched_getcpu: [ on ]
... sdt: [ OFF ]
... setns: [ on ]
... libopencsd: [ on ]
[...]
CC util/arm-spe.o
CC util/arm-spe-pkt-decoder.o
CC util/s390-cpumsf.o
CC util/cs-etm.o
CC util/parse-branch-options.o
util/cs-etm.c: In function ‘cs_etm__mem_access’:
util/cs-etm.c:297:24: error: storage size of ‘al’ isn’t known
struct addr_location al;
And rightly so since file cs-etm.c doesn't include symbol.h, something
that is rectified in this patch.
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Suzuki K Poulouse <suzuki.poulose@arm.com>
Cc: linux-arm-kernel@lists.infradead.org
Link: http://lkml.kernel.org/r/20190208223543.31836-1-mathieu.poirier@linaro.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/util/cs-etm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/perf/util/cs-etm.c b/tools/perf/util/cs-etm.c
index 8b3f882d6e2f..0b11d653cfbe 100644
--- a/tools/perf/util/cs-etm.c
+++ b/tools/perf/util/cs-etm.c
@@ -25,6 +25,7 @@
#include "machine.h"
#include "map.h"
#include "perf.h"
+#include "symbol.h"
#include "thread.h"
#include "thread_map.h"
#include "thread-stack.h"
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-02-15 9:26 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-02-08 22:35 [PATCH] perf tools: Add proper header file for symbols Mathieu Poirier
2019-02-08 22:35 ` Mathieu Poirier
2019-02-11 11:46 ` Arnaldo Carvalho de Melo
2019-02-11 11:46 ` Arnaldo Carvalho de Melo
2019-02-11 14:22 ` Arnaldo Carvalho de Melo
2019-02-15 9:25 ` [tip:perf/core] perf cs-etm: " tip-bot for Mathieu Poirier
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.