Paolo Bonzini <pbonz...@redhat.com> wrote: > On 14/06/2017 19:02, Felipe Franciosi wrote: >> >>> On 14 Jun 2017, at 17:59, no-re...@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-fel...@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.
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.