qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PULL 00/44] ppc-for-2.8 queue 20160922


From: David Gibson
Subject: Re: [Qemu-devel] [PULL 00/44] ppc-for-2.8 queue 20160922
Date: Fri, 23 Sep 2016 12:40:18 +1000
User-agent: Mutt/1.7.0 (2016-08-17)

On Thu, Sep 22, 2016 at 03:03:50PM +0100, Peter Maydell wrote:
> On 22 September 2016 at 07:36, David Gibson <address@hidden> wrote:
> > The following changes since commit a008535b9fa396226ff9cf78b8ac5f3584bda58e:
> >
> >   build-sys: fix make install regression (2016-09-20 11:32:43 +0100)
> >
> > are available in the git repository at:
> >
> >   git://github.com/dgibson/qemu.git tags/ppc-for-2.8-20160922
> >
> > for you to fetch changes up to 2832da4b6fc549d5feb2cf9fe53ad98cee894327:
> >
> >   monitor: fix crash for platforms without a CPU 0 (2016-09-22 15:53:01 
> > +1000)
> >
> > ----------------------------------------------------------------
> > ppc patch queue 2016-09-22
> >
> > This is my second pull request of ppc and spapr related patches for
> > qemu-2.8.  Included here are
> >     * TCG implementations for more POWER9 instructions
> >     * Some preliminary XICS fixes in preparataion for the pnv machine type
> >     * A significant ADB (Macintosh kbd/mouse) cleanup
> >     * Some conversions to use trace instead of debug macros
> >     * Fixes to correctly handle global TLB flush synchronization in
> >       TCG.  This is already a bug, but it will have much more impact
> >       when we get MTTCG
> >     * Add more qtest testcases for Power
> >     * Some MAINTAINERS updates
> >     * Assorted bugfixes
> >
> > This touches some test files and monitor.c which are technically
> > outside the ppc code, but coming through this tree because the changes
> > are primarily of interest to ppc.
> >
> > ----------------------------------------------------------------
> 
> I'm afraid this fails to build with clang:
> 
> /home/petmay01/linaro/qemu-for-merges/target-ppc/translate.c:532:16:
> error: unused function 'L' [-Werro
> r,-Wunused-function]
> EXTRACT_HELPER(L, 16, 2);
>                ^
> 1 error generated.

Drat, I wonder why travis didn't catch that for me.

Anyway, I've added an extra ifdef to address this and will send a new
pull request shortly.

-- 
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

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]