[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/57] Migration pull
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PULL 00/57] Migration pull |
Date: |
Mon, 09 Nov 2015 19:49:03 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Peter Maydell <address@hidden> wrote:
> On 9 November 2015 at 17:28, Juan Quintela <address@hidden> wrote:
>>
>> Hi
>>
>> two items:
>>
>> - Postcopy pull request (dave)
>> - fix for qemu_completion (Denis)
>>
>> Please apply.
>>
>> Thanks, Juan.
>>
>>
>> The following changes since commit ce278618b088afd10b91a05311eaeb6401bb5004:
>>
>> configure: Don't disable optimization for non-fortify builds
>> (2015-11-09 16:28:09 +0000)
>>
>> are available in the git repository at:
>>
>> git://github.com/juanquintela/qemu.git tags/migration/20151109
>>
>> for you to fetch changes up to d1de612d6ed199a52a6f4823d89c1731c74edcb6:
>>
>> migration: qemu_savevm_state_cleanup becomes mandatory operation
>> (2015-11-09 18:26:06 +0100)
>>
>> ----------------------------------------------------------------
>> migration/next for 20151109
>>
>> ----------------------------------------------------------------
>
> Fails to build on W32:
>
> cc1: warnings being treated as errors
> /home/petmay01/linaro/qemu-for-merges/migration/migration.c: In
> function ‘source_return_path_thread’:
> /home/petmay01/linaro/qemu-for-merges/migration/migration.c:1289:
> warning: format ‘%zd’ expects type ‘signed size_t’, but argument 5 has
> type ‘ssize_t’
After asking around on irc, "poll" favoured %zd for printing ssize_t
values. But clearly your crosscompiler don't agree. So what to do:
- another pull request with %ld .... (long) .... cast? Ugly but should
work.
- you change it yourself?
- any other suggestion?
I test compiled it on x86_64 and i686 linux, and with w32 and w64
compilers, so I thought that I have covered the normal cases.
Later, Juan.
> CC migration/block.o
> make: *** [migration/migration.o] Error 1
> make: *** Waiting for unfinished jobs....
> CC net/net.o
> cc1: warnings being treated as errors
> /home/petmay01/linaro/qemu-for-merges/migration/qemu-file-unix.c: In
> function ‘socket_writev_buffer’:
> /home/petmay01/linaro/qemu-for-merges/migration/qemu-file-unix.c:59:
> warning: format ‘%zd’ expects type ‘signed size_t’, but argument 3 has
> type ‘ssize_t’
> /home/petmay01/linaro/qemu-for-merges/migration/qemu-file-unix.c:59:
> warning: format ‘%zd’ expects type ‘signed size_t’, but argument 4 has
> type ‘ssize_t’
> make: *** [migration/qemu-file-unix.o] Error 1
>
> thanks
> -- PMM
- Re: [Qemu-devel] [PULL 00/57] Migration pull, (continued)
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Peter Maydell, 2015/11/09
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Juan Quintela, 2015/11/09
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Eric Blake, 2015/11/09
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Peter Maydell, 2015/11/10
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Peter Maydell, 2015/11/10
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Dr. David Alan Gilbert, 2015/11/10
- Re: [Qemu-devel] [PULL 00/57] Migration pull, Peter Maydell, 2015/11/10
Re: [Qemu-devel] [PULL 00/57] Migration pull,
Juan Quintela <=
Re: [Qemu-devel] [PULL 00/57] Migration pull, Markus Armbruster, 2015/11/09