[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL v4 00/11] Net patches
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PULL v4 00/11] Net patches |
Date: |
Thu, 12 Mar 2015 19:58:30 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Thu, Mar 12, 2015 at 03:03:45PM +0000, Peter Maydell wrote:
> On 12 March 2015 at 14:43, Peter Maydell <address@hidden> wrote:
> > On 12 March 2015 at 14:33, Stefan Hajnoczi <address@hidden> wrote:
> >> v4:
> >> * Drop clang 3.5.0 fixes since they break with older clang and w32 [Peter]
> >>
> >> v3:
> >> * Add clang 3.5.0 warning fixes
> >> * Squash David Ahern's clang struct definition warnings fix
> >>
> >> v2:
> >> * Squash in Jiri's fix for rocker format string specifiers [Peter]
> >> * Squash in Windows build fix [Peter]
> >> * Both build fixes are described in "rocker: add new rocker switch device"
> >>
> >> The following changes since commit
> >> ee74801035b0b5f1fdfd4e31d3a53f511f91c804:
> >>
> >> Merge remote-tracking branch 'remotes/lalrae/tags/mips-20150311' into
> >> staging (2015-03-11 18:22:15 +0000)
> >>
> >> are available in the git repository at:
> >>
> >> git://github.com/stefanha/qemu.git tags/net-pull-request
> >>
> >> for you to fetch changes up to 8d3f4f565fb4adf940a8824882a7f549304f34fd:
> >>
> >> rocker: timestamp on the debug logs helps correlate with events in the
> >> VM (2015-03-12 14:18:11 +0000)
> >
> > w32 build failure :-(
> >
> > cc1: warnings being treated as errors
> > /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker.c: In
> > function 'rocker_io_readq':
> > /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker.c:1109:
> > warning: integer constant is too large for 'long' type
> >
> > (64 bit constant without ULL suffix.)
>
> Also fails to build with our minimum glib version:
>
> /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker_of_dpa.c:
> In function 'of_dpa_init':
> /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker_of_dpa.c:2273:
> error: 'g_int64_hash' undeclared (first use in this function)
> /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker_of_dpa.c:2273:
> error: (Each undeclared identifier is reported only once
> /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker_of_dpa.c:2273:
> error: for each function it appears in.)
> /home/petmay01/linaro/qemu-for-merges/hw/net/rocker/rocker_of_dpa.c:2273:
> error: 'g_int64_equal' undeclared (first use in this function)
>
> (Both first introduced in glib 2.22.)
I'm dropping the rocker patch series.
David, Jiri, Scott: The minimum required glib version is 2.12 so glib
APIs that were added later cannot be used. Also note the Windows
integer constant warning above.
My build environments didn't hit these issues so we need to go around
one more time. Please post fixes.
Stefan
pgpiMZFutpMVu.pgp
Description: PGP signature
- [Qemu-devel] [PULL v4 09/11] rocker: add tests, (continued)
- [Qemu-devel] [PULL v4 09/11] rocker: add tests, Stefan Hajnoczi, 2015/03/12
- [Qemu-devel] [PULL v4 11/11] rocker: timestamp on the debug logs helps correlate with events in the VM, Stefan Hajnoczi, 2015/03/12
- [Qemu-devel] [PULL v4 10/11] MAINTAINERS: add rocker, Stefan Hajnoczi, 2015/03/12
- [Qemu-devel] [PULL v4 08/11] rocker: add new rocker switch device, Stefan Hajnoczi, 2015/03/12
- Re: [Qemu-devel] [PULL v4 00/11] Net patches, Peter Maydell, 2015/03/12
- Re: [Qemu-devel] [PULL v4 00/11] Net patches,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PULL v4 00/11] Net patches, Paolo Bonzini, 2015/03/12
- Re: [Qemu-devel] [PULL v4 00/11] Net patches, Scott Feldman, 2015/03/14
- Re: [Qemu-devel] [PULL v4 00/11] Net patches, Peter Maydell, 2015/03/16