Re: [Qemu-devel] [PATCH] vhost-user: support cross-endianess negatiation

[Paolo Bonzini <pbonzini@redhat.com>]

On 14/06/2017 19:02, Felipe Franciosi wrote:
> 
>> On 14 Jun 2017, at 17:59, no-reply@patchew.org 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: 1497458486-15673-1-git-send-email-felipe@nutanix.com
>> 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 'docker-run-test-build@min-glib' failed
>> make: *** [docker-run-test-build@min-glib] 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.

Paolo