On Tue, Jun 16, 2020 at 02:52:16PM +0800, Coiby Xu wrote: > On Sun, Jun 14, 2020 at 12:16:28PM -0700, no-re...@patchew.org wrote: > > Patchew URL: > > https://patchew.org/QEMU/20200614183907.514282-1-coiby...@gmail.com/ > > > > > > > > Hi, > > > > This series failed the asan build test. Please find the testing commands and > > their output below. If you have Docker installed, you can probably > > reproduce it > > locally. > > > > === TEST SCRIPT BEGIN === > > #!/bin/bash > > export ARCH=x86_64 > > make docker-image-fedora V=1 NETWORK=1 > > time make docker-test-debug@fedora TARGET_LIST=x86_64-softmmu J=14 NETWORK=1 > > === TEST SCRIPT END === > > > > CC stubs/vm-stop.o > > CC ui/input-keymap.o > > CC qemu-keymap.o > > /tmp/qemu-test/src/util/vhost-user-server.c:142:30: error: use of > > undeclared identifier 'VHOST_MEMORY_MAX_NREGIONS' > > VHOST_MEMORY_MAX_NREGIONS, nfds + nfds_t); > > ^ > > > > The full log is available at > > http://patchew.org/logs/20200614183907.514282-1-coiby...@gmail.com/testing.asan/?type=message. > > I couldn't re-produce this error locally for both docker-test-quick@centos7 > and this docker test. And I can't see any reason for this error to occur since > VHOST_MEMORY_MAX_NREGIONS is defined in contrib/libvhost-user/libvhost-user.h > which has been included by util/vhost-user-server.h.
Using G_N_ELEMENTS(vmsg->fds) instead of VHOST_MEMORY_MAX_NREGIONS is an even cleaner fix. Stefan
signature.asc
Description: PGP signature
