[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/20] tests: fix linking test-char on win32
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 01/20] tests: fix linking test-char on win32 |
Date: |
Fri, 6 Jan 2017 14:50:42 -0500 (EST) |
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 <address@hidden>
> > ---
> > 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 <address@hidden>
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
>
- [Qemu-devel] [PATCH 00/20] chardev: qom-ify, Marc-André Lureau, 2017/01/05
- [Qemu-devel] [PATCH 02/20] qemu-options: stdio is available on win32, Marc-André Lureau, 2017/01/05
- [Qemu-devel] [PATCH 03/20] char: add qemu_chr_fe_add_watch() Returns description, Marc-André Lureau, 2017/01/05
- [Qemu-devel] [PATCH 04/20] doc: fix spelling, Marc-André Lureau, 2017/01/05
- [Qemu-devel] [PATCH 05/20] char: use a const CharDriver, Marc-André Lureau, 2017/01/05