Re: [PATCH v4 0/9] add xenalyze to staging

[Ian Campbell <ian.campbell@citrix.com>]

On Sat, 2015-05-23 at 08:24 +0000, Olaf Hering wrote:
> Having xenalyze in the source tree makes it much easier to keep private
> debug code in hypervisor and xenalyze in sync. It helped alot while
> debugging the root cause for commit 607e8494c42397fb249191904066cace6ac9a880.

I'm afraid it doesn't build on arm64.

Some of these actually look like non-arch specific failures (e.g.
conflicts with register_t from system headers) or issues which should
probably be addressed with xenalyze in tree (e.g. NR_CPUS ought to be
available directly now?) or with some trivial #ifdef modifications.

That said, I don't know that xentrace actually works on ARM nor that
xenalyze could analyse such traces even with the build issues addressed,
so I'd be equally happy if this was just made x86 only.

In file included from /usr/aarch64-linux-gnu/include/linux/types.h:4:0,
                 from /usr/aarch64-linux-gnu/include/asm/ptrace.h:22,
                 from /usr/aarch64-linux-gnu/include/sys/user.h:25,
                 from /usr/aarch64-linux-gnu/include/sys/procfs.h:34,
                 from /usr/aarch64-linux-gnu/include/sys/ucontext.h:26,
                 from /usr/aarch64-linux-gnu/include/signal.h:360,
                 from /usr/aarch64-linux-gnu/include/sys/wait.h:30,
                 from xenalyze.c:28:
/local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/asm/types.h:54:13: error: conflicting types for 'register_t'
 typedef u64 register_t;
             ^
In file included from /usr/aarch64-linux-gnu/include/stdlib.h:314:0,
                 from xenalyze.c:24:
/usr/aarch64-linux-gnu/include/sys/types.h:205:13: note: previous declaration of 'register_t' was here
 typedef int register_t __attribute__ ((__mode__ (__word__)));
             ^
In file included from xenalyze.c:33:0:
analyze.h:19:0: error: "NR_CPUS" redefined [-Werror]
 #define NR_CPUS 256
 ^
In file included from /local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/xen/config.h:13:0,
                 from /local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/asm/types.h:6,
                 from /usr/aarch64-linux-gnu/include/linux/types.h:4,
                 from /usr/aarch64-linux-gnu/include/asm/ptrace.h:22,
                 from /usr/aarch64-linux-gnu/include/sys/user.h:25,
                 from /usr/aarch64-linux-gnu/include/sys/procfs.h:34,
                 from /usr/aarch64-linux-gnu/include/sys/ucontext.h:26,
                 from /usr/aarch64-linux-gnu/include/signal.h:360,
                 from /usr/aarch64-linux-gnu/include/sys/wait.h:30,
                 from xenalyze.c:28:
/local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/asm/config.h:47:0: note: this is the location of the previous definition
 #define NR_CPUS 128
 ^
In file included from xenalyze.c:33:0:
analyze.h:23:0: error: "BITS_PER_LONG" redefined [-Werror]
 # define BITS_PER_LONG 32
 ^
In file included from /local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/xen/config.h:13:0,
                 from /local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/asm/types.h:6,
                 from /usr/aarch64-linux-gnu/include/linux/types.h:4,
                 from /usr/aarch64-linux-gnu/include/asm/ptrace.h:22,
                 from /usr/aarch64-linux-gnu/include/sys/user.h:25,
                 from /usr/aarch64-linux-gnu/include/sys/procfs.h:34,
                 from /usr/aarch64-linux-gnu/include/sys/ucontext.h:26,
                 from /usr/aarch64-linux-gnu/include/signal.h:360,
                 from /usr/aarch64-linux-gnu/include/sys/wait.h:30,
                 from xenalyze.c:28:
/local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../xen/include/asm/config.h:23:0: note: this is the location of the previous definition
 #define BITS_PER_LONG (BYTES_PER_LONG << 3)
 ^
aarch64-linux-gnu-gcc    -o xentrace_setsize setsize.o /local/scratch/ianc/devel/committer-arm64.git/tools/xentrace/../../tools/libxc/libxenctrl.so