Re: [Qemu-devel] [PATCH 01/20] tests: fix linking test-char on win32

["Marc-André Lureau" <mlureau@redhat.com>]

Hi

----- Original Message -----
> On 01/05/2017 10:53 AM, Marc-André Lureau wrote:
> > test-char.exe needs main-loop.o symbols, among others. Linking with
> > $(test-block-obj-y) brings what's necessary. We could try to eventually
> > strip to the minimum if needed.
> > 
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> > ---
> >  tests/Makefile.include | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Can I have a bit more background? I'm guessing this doesn't fix an
> actual failure at the moment (probably due to one of the other files
> dragging in stuff indirectly), but rather cleans up a latent problem
> that would otherwise be exposed with patches later in the series (where
> splitting files means the stuff is no longer dragged in indirectly, so
> we need to mention it explicitly)?

It does fix current linking failure:
x86_64-w64-mingw32-g++ -I/usr/x86_64-w64-mingw32/sys-root/mingw/include/pixman-1  -I/home/elmarco/src/qemu/dtc/libfdt -Werror -mms-bitfields -I/usr/x86_64-w64-mingw32/sys-root/mingw/include/glib-2.0 -I/usr/x86_64-w64-mingw32/sys-root/mingw/lib/glib-2.0/include -I/usr/x86_64-w64-mingw32/sys-root/mingw/include  -m64 -mcx16 -mthreads -D__USE_MINGW_ANSI_STDIO=1 -DWIN32_LEAN_AND_MEAN -DWINVER=0x501 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv  -Wendif-labels -Wno-shift-negative-value -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration -Wold-style-definition -Wtype-limits -fstack-protector-strong  -I/usr/x86_64-w64-mingw32/sys-root/mingw/include/libpng16  -I/home/elmarco/src/qemu/tests -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -g  -Wl,--nxcompat -Wl,--no-seh -Wl,--dynamicbase -Wl,--warn-common -m64 -g  -o tests/test-char.exe tests/test-char.o qemu-char.o qemu-timer.o io/channel.o io/channel-buffer.o io/channel-command.o io/channel-file.o io/channel-socket.o io/channel-tls.o io/channel-watch.o io/channel-websock.o io/channel-util.o io/task.o crypto/init.o crypto/hash.o crypto/hash-glib.o crypto/hmac.o crypto/hmac-glib.o crypto/aes.o crypto/desrfb.o crypto/cipher.o crypto/tlscreds.o crypto/tlscredsanon.o crypto/tlscredsx509.o crypto/tlssession.o crypto/secret.o crypto/random-platform.o crypto/pbkdf.o crypto/ivgen.o crypto/ivgen-essiv.o crypto/ivgen-plain.o crypto/ivgen-plain64.o crypto/afsplit.o crypto/xts.o crypto/block.o crypto/block-qcow.o crypto/block-luks.o qom/object.o qom/container.o qom/qom-qobject.o qom/object_interfaces.o  libqemuutil.a libqemustub.a /home/elmarco/src/qemu/winb/version.o  -L/usr/x86_64-w64-mingw32/sys-root/mingw/lib -lgthread-2.0 -lglib-2.0 -lintl  -lwinmm -lws2_32 -liphlpapi  -lz -lz 
qemu-char.o: In function `win_chr_free':
/home/elmarco/src/qemu/qemu-char.c:2149: undefined reference to `qemu_del_polling_cb'
/home/elmarco/src/qemu/qemu-char.c:2151: undefined reference to `qemu_del_polling_cb'
qemu-char.o: In function `win_stdio_thread':
/home/elmarco/src/qemu/qemu-char.c:2568: undefined reference to `qemu_del_wait_object'
qemu-char.o: In function `qemu_chr_open_stdio':
/home/elmarco/src/qemu/qemu-char.c:2661: undefined reference to `qemu_add_wait_object'
/home/elmarco/src/qemu/qemu-char.c:2646: undefined reference to `qemu_add_wait_object'
/home/elmarco/src/qemu/qemu-char.c:2696: undefined reference to `qemu_del_wait_object'
/home/elmarco/src/qemu/qemu-char.c:2691: undefined reference to `qemu_del_wait_object'
qemu-char.o: In function `win_chr_pipe_init':
/home/elmarco/src/qemu/qemu-char.c:2413: undefined reference to `qemu_add_polling_cb'
qemu-char.o: In function `win_chr_init':
/home/elmarco/src/qemu/qemu-char.c:2215: undefined reference to `qemu_add_polling_cb'
qemu-char.o: In function `win_stdio_wait_func':
/home/elmarco/src/qemu/qemu-char.c:2512: undefined reference to `qemu_del_wait_object'
collect2: error: ld returned 1 exit status
/home/elmarco/src/qemu/rules.mak:109: recipe for target 'tests/test-char.exe' failed
make: *** [tests/test-char.exe] Error 1

I can add this linking error to the commit log if it helps.

> 
> Even mentioning _which_ main-loop.o symbol to be grepping for in
> test-char.c might be helpful to the review.

It's from qemu-char.c.

> 
> > 
> > diff --git a/tests/Makefile.include b/tests/Makefile.include
> > index 4841d582a1..a8f3d12e49 100644
> > --- a/tests/Makefile.include
> > +++ b/tests/Makefile.include
> > @@ -489,7 +489,7 @@ tests/check-qjson$(EXESUF): tests/check-qjson.o
> > $(test-util-obj-y)
> >  tests/check-qom-interface$(EXESUF): tests/check-qom-interface.o
> >  $(test-qom-obj-y)
> >  tests/check-qom-proplist$(EXESUF): tests/check-qom-proplist.o
> >  $(test-qom-obj-y)
> >  
> > -tests/test-char$(EXESUF): tests/test-char.o qemu-char.o qemu-timer.o
> > $(test-util-obj-y) $(qtest-obj-y) $(test-io-obj-y)
> > +tests/test-char$(EXESUF): tests/test-char.o qemu-char.o qemu-timer.o
> > $(test-util-obj-y) $(qtest-obj-y) $(test-block-obj-y)
> 
> At any rate, test-block-obj-y is simply test-io-obj-y plus the
> additional block-obj-y, so you're merely adding an additional dependency
> on block-obj-y files.  While a nicer commit message may help, I don't
> see any technical problem in the change, so here's a weak:
> 
> Reviewed-by: Eric Blake <eblake@redhat.com>
> 
> --
> Eric Blake   eblake redhat com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
> 
>