[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] vhost-user: support cross-endianess negatiation
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH] vhost-user: support cross-endianess negatiation |
Date: |
Wed, 14 Jun 2017 19:29:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Paolo Bonzini <address@hidden> wrote:
> On 14/06/2017 19:02, Felipe Franciosi wrote:
>>
>>> On 14 Jun 2017, at 17:59, address@hidden wrote:
>>>
>>> Hi,
>>>
>>> This series failed automatic build test. Please find the testing commands
>>> and
>>> their output below. If you have docker installed, you can probably
>>> reproduce it
>>> locally.
>>>
>>> Message-id: address@hidden
>>> Type: series
>>> Subject: [Qemu-devel] [PATCH] vhost-user: support cross-endianess
>>> negatiation
>>>
>> ...
>>>
>>> CC hw/pci-bridge/pci_expander_bridge.o
>>> In file included from /tmp/qemu-test/src/hw/net/vmxnet3.c:30:
>>> /tmp/qemu-test/src/include/migration/register.h:18: error:
>>> redefinition of typedef ‘LoadStateHandler’
>>> /tmp/qemu-test/src/include/migration/vmstate.h:32: note: previous
>>> declaration of ‘LoadStateHandler’ was here
>>> make: *** [hw/net/vmxnet3.o] Error 1
>>> make: *** Waiting for unfinished jobs....
>>> tests/docker/Makefile.include:118: recipe for target 'docker-run' failed
>>> make[1]: *** [docker-run] Error 2
>>> make[1]: Leaving directory '/var/tmp/patchew-tester-tmp-enl3ixyr/src'
>>> tests/docker/Makefile.include:149: recipe for target
>>> 'address@hidden' failed
>>> make: *** address@hidden Error 2
>>> === OUTPUT END ===
>>>
>>> Test command exited with code: 2
>>
>> That seems completely unrelated to what our patch touched... I'm
>> gonna guess this is a known issue and I can safely ignore it?
>
> Yes.
>
> Juan, can you take a look? Just removing the typedef from
> VMStateDescription, and instead just using int(*load_state_old)(etc...)
> should do it.
There is a patch already on list, reviewed, and on a PULL request.
Problem is that new complires don't complain about a repeated typedef.
Older ones do. I compile on F25, and there it don't warn/error.
Sorry for the noise.
Later, Juan.