[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ppc: xics: fix compilation with CentOS 6
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH] ppc: xics: fix compilation with CentOS 6 |
Date: |
Mon, 8 May 2017 16:13:59 +1000 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Fri, May 05, 2017 at 11:48:50AM +0200, Paolo Bonzini wrote:
> The PowerPCCPU typedef is included twice if a file includes
> both hw/ppc/xics.h and target/ppc/cpu-qom.h.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
Applied to ppc-for-2.10, thanks.
> ---
> There are other failures but I don't have time to debug them today:
>
> In file included from /tmp/qemu-test/src/include/hw/ppc/pnv.h:26,
> from /tmp/qemu-test/src/hw/ppc/pnv.c:31:
> /tmp/qemu-test/src/include/hw/ppc/pnv_psi.h:50: error: redefinition of
> typedef ‘PnvPsi’
> /tmp/qemu-test/src/include/hw/ppc/pnv_lpc.h:26: note: previous
> declaration of ‘PnvPsi’ was here
> In file included from /tmp/qemu-test/src/include/hw/ppc/pnv.h:27,
> from /tmp/qemu-test/src/hw/ppc/pnv.c:31:
> /tmp/qemu-test/src/include/hw/ppc/pnv_occ.h:25: error: redefinition of
> typedef ‘PnvPsi’
> /tmp/qemu-test/src/include/hw/ppc/pnv_psi.h:50: note: previous
> declaration of ‘PnvPsi’ was here
> In file included from /tmp/qemu-test/src/hw/ppc/pnv.c:39:
> /tmp/qemu-test/src/include/hw/ipmi/ipmi.h:188: error: redefinition of
> typedef ‘IPMIBmc’
> /tmp/qemu-test/src/include/hw/ppc/pnv.h:121: note: previous declaration
> of ‘IPMIBmc’ was here
> make[1]: *** [hw/ppc/pnv.o] Error 1
>
> Please start including Docker-based tests in your pre-pull-request
> tests.
Um.. is there some doco on how to execute these. I tried "make
docker-test" and "make docker-image" and they both just seem to hang
at:
umbus:~/src/qemu (ppc-for-2.10)$ make docker-image
BUILD travis
strace shows docker blocked on a futex.
> include/hw/ppc/xics.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/hw/ppc/xics.h b/include/hw/ppc/xics.h
> index c215dc7..ce1942b 100644
> --- a/include/hw/ppc/xics.h
> +++ b/include/hw/ppc/xics.h
> @@ -29,6 +29,7 @@
> #define XICS_H
>
> #include "hw/qdev.h"
> +#include "target/ppc/cpu-qom.h"
>
> #define XICS_IPI 0x2
> #define XICS_BUID 0x1
> @@ -46,7 +47,6 @@ typedef struct ICSStateClass ICSStateClass;
> typedef struct ICSState ICSState;
> typedef struct ICSIRQState ICSIRQState;
> typedef struct XICSFabric XICSFabric;
> -typedef struct PowerPCCPU PowerPCCPU;
>
> #define TYPE_ICP "icp"
> #define ICP(obj) OBJECT_CHECK(ICPState, (obj), TYPE_ICP)
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature