From 978e305a8cb8533bef2c6238c88e96913f7d09d0 Mon Sep 17 00:00:00 2001 From: Mark McLoughlin Date: Mon, 29 Jun 2009 14:49:03 +0100 Subject: [PATCH] Fix linux-user build on ppc kvm-87 build fails on ppc: https://koji.fedoraproject.org/koji/getfile?taskID=1441042&name=build.log gcc -I. -I.. -I/builddir/build/BUILD/qemu-kvm-devel-87/target-i386 -I/builddir/build/BUILD/qemu-kvm-devel-87 -MMD -MT elfload.o -MP -DNEED_CPU_H -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D__user= -I/builddir/build/BUILD/qemu-kvm-devel-87/tcg -I/builddir/build/BUILD/qemu-kvm-devel-87/tcg/ppc64 -I/builddir/build/BUILD/qemu-kvm-devel-87/fpu -I/builddir/build/BUILD/qemu-kvm-devel-87/linux-user -I/builddir/build/BUILD/qemu-kvm-devel-87/linux-user/i386 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mminimal-toc -g -fno-strict-aliasing -O2 -Wall -Wundef -Wendif-labels -Wwrite-strings -Wmissing-prototypes -Wstrict-prototypes -Wredundant-decls -c -o elfload.o /builddir/build/BUILD/qemu-kvm-devel-87/linux-user/elfload.c /builddir/build/BUILD/qemu-kvm-devel-87/linux-user/elfload.c:214: error: conflicting types for 'elf_greg_t' /usr/include/asm/elf.h:123: note: previous declaration of 'elf_greg_t' was here /builddir/build/BUILD/qemu-kvm-devel-87/linux-user/elfload.c:220: error: conflicting types for 'elf_gregset_t' /usr/include/asm/elf.h:124: note: previous declaration of 'elf_gregset_t' was here In file included from /builddir/build/BUILD/qemu-kvm-devel-87/linux-user/elfload.c:697: ../elf.h:457:1: warning: "R_PPC_NUM" redefined In file included from /usr/include/asm/sigcontext.h:13, from /usr/include/bits/sigcontext.h:28, from /usr/include/signal.h:339, from /builddir/build/BUILD/qemu-kvm-devel-87/linux-user/qemu.h:4, from /builddir/build/BUILD/qemu-kvm-devel-87/linux-user/elfload.c:16: /usr/include/asm/elf.h:81:1: warning: this is the location of the previous definition Problem seems to be that signal.h is pulling in a bunch of ppc headers which expose elf_greg_t, R_PPC_* and PPC_FEATURE_*. Signed-off-by: Mark McLoughlin --- elf.h | 2 ++ linux-user/elfload.c | 10 ++++++++++ 2 files changed, 12 insertions(+), 0 deletions(-) diff --git a/elf.h b/elf.h index b042002..14f8aa1 100644 --- a/elf.h +++ b/elf.h @@ -454,7 +454,9 @@ typedef struct { #define R_PPC_SECTOFF_HI 35 #define R_PPC_SECTOFF_HA 36 /* Keep this the last entry. */ +#ifndef R_PPC_NUM #define R_PPC_NUM 37 +#endif /* R_PPC_NUM */ /* ARM specific declarations */ diff --git a/linux-user/elfload.c b/linux-user/elfload.c index d31cca7..3ccfdda 100644 --- a/linux-user/elfload.c +++ b/linux-user/elfload.c @@ -134,6 +134,7 @@ static inline void init_thread(struct target_pt_regs *regs, struct image_info *i regs->rip = infop->entry; } +#ifdef FIX_BROKEN_PPC_BUILD typedef target_ulong elf_greg_t; typedef uint32_t target_uid_t; typedef uint32_t target_gid_t; @@ -179,6 +180,7 @@ static void elf_core_copy_regs(elf_gregset_t *regs, const CPUState *env) (*regs)[25] = env->segs[R_FS].selector & 0xffff; (*regs)[26] = env->segs[R_GS].selector & 0xffff; } +#endif /* FIX_BROKEN_PPC_BUILD */ #else @@ -211,6 +213,7 @@ static inline void init_thread(struct target_pt_regs *regs, struct image_info *i regs->edx = 0; } +#ifdef FIX_BROKEN_PPC_BUILD typedef target_ulong elf_greg_t; typedef uint16_t target_uid_t; typedef uint16_t target_gid_t; @@ -246,9 +249,12 @@ static void elf_core_copy_regs(elf_gregset_t *regs, const CPUState *env) (*regs)[15] = env->regs[R_ESP]; (*regs)[16] = env->segs[R_SS].selector & 0xffff; } +#endif /* FIX_BROKEN_PPC_BUILD */ #endif +#ifdef FIX_BROKEN_PPC_BUILD #define USE_ELF_CORE_DUMP +#endif /* FIX_BROKEN_PPC_BUILD */ #define ELF_EXEC_PAGESIZE 4096 #endif @@ -286,6 +292,7 @@ static inline void init_thread(struct target_pt_regs *regs, struct image_info *i regs->ARM_r10 = infop->start_data; } +#ifdef FIX_BROKEN_PPC_BUILD typedef uint32_t elf_greg_t; typedef uint16_t target_uid_t; typedef uint16_t target_gid_t; @@ -318,6 +325,7 @@ static void elf_core_copy_regs(elf_gregset_t *regs, const CPUState *env) } #define USE_ELF_CORE_DUMP +#endif /* FIX_BROKEN_PPC_BUILD */ #define ELF_EXEC_PAGESIZE 4096 enum @@ -421,6 +429,7 @@ static inline void init_thread(struct target_pt_regs *regs, struct image_info *i /* Feature masks for the Aux Vector Hardware Capabilities (AT_HWCAP). See arch/powerpc/include/asm/cputable.h. */ +#ifndef PPC_FEATURE_32 enum { PPC_FEATURE_32 = 0x80000000, PPC_FEATURE_64 = 0x40000000, @@ -452,6 +461,7 @@ enum { PPC_FEATURE_TRUE_LE = 0x00000002, PPC_FEATURE_PPC_LE = 0x00000001, }; +#endif /* !defined(PPC_FEATURE_32) */ #define ELF_HWCAP get_elf_hwcap() -- 1.6.2.5