Message ID | 20200527145455.2550-1-alex.bennee@linaro.org |
---|---|
Headers | show |
Series | testing and plugin fixes | expand |
On Wed, 27 May 2020 at 15:54, Alex Bennée <alex.bennee@linaro.org> wrote: > > The following changes since commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: > > Merge remote-tracking branch 'remotes/gkurz/tags/9p-next-2020-05-26' into staging (2020-05-26 14:05:53 +0100) > > are available in the Git repository at: > > https://github.com/stsquad/qemu.git tags/pull-testing-tcg-plugins-270520-1 > > for you to fetch changes up to 919bfbf5d6569b63a374332292cf3d2355a6d6c3: > > tests/tcg: add new threadcount test (2020-05-27 14:26:49 +0100) > > ---------------------------------------------------------------- > Testing and one plugin fix: > > - support alternates for genisoimage to test/vm > - add clang++ to clang tests > - fix record/replay smoke test > - enable more softfloat tests > - better detection of hung gdb > - upgrade aarch64 tcg test x-compile to gcc-10 > - fix plugin cpu_index clash vs threads This makes the NetBSD tests/vm setup spit out a huge pile of extra warnings: every link has something like: ld: warning: libintl.so.1, needed by /usr/pkg/lib/libglib-2.0.so, may conflict with libintl.so.8 or some variant like: ld: warning: libintl.so.1, needed by /usr/pkg/lib/libcurl.so, may conflict with libintl.so.8 ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 thanks -- PMM
Peter Maydell <peter.maydell@linaro.org> writes: > On Wed, 27 May 2020 at 15:54, Alex Bennée <alex.bennee@linaro.org> wrote: >> >> The following changes since commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: >> >> Merge remote-tracking branch 'remotes/gkurz/tags/9p-next-2020-05-26' into staging (2020-05-26 14:05:53 +0100) >> >> are available in the Git repository at: >> >> https://github.com/stsquad/qemu.git tags/pull-testing-tcg-plugins-270520-1 >> >> for you to fetch changes up to 919bfbf5d6569b63a374332292cf3d2355a6d6c3: >> >> tests/tcg: add new threadcount test (2020-05-27 14:26:49 +0100) >> >> ---------------------------------------------------------------- >> Testing and one plugin fix: >> >> - support alternates for genisoimage to test/vm >> - add clang++ to clang tests >> - fix record/replay smoke test >> - enable more softfloat tests >> - better detection of hung gdb >> - upgrade aarch64 tcg test x-compile to gcc-10 >> - fix plugin cpu_index clash vs threads > > This makes the NetBSD tests/vm setup spit out a huge pile of extra warnings: > every link has something like: > > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libglib-2.0.so, may > conflict with libintl.so.8 > > or some variant like: > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libcurl.so, may > conflict with libintl.so.8 > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may > conflict with libintl.so.8 Weird... I'll have a look at whats going on. > > thanks > -- PMM -- Alex Bennée
Peter Maydell <peter.maydell@linaro.org> writes: > On Wed, 27 May 2020 at 15:54, Alex Bennée <alex.bennee@linaro.org> wrote: >> >> The following changes since commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: >> >> Merge remote-tracking branch 'remotes/gkurz/tags/9p-next-2020-05-26' into staging (2020-05-26 14:05:53 +0100) >> >> are available in the Git repository at: >> >> https://github.com/stsquad/qemu.git tags/pull-testing-tcg-plugins-270520-1 >> >> for you to fetch changes up to 919bfbf5d6569b63a374332292cf3d2355a6d6c3: >> >> tests/tcg: add new threadcount test (2020-05-27 14:26:49 +0100) >> >> ---------------------------------------------------------------- >> Testing and one plugin fix: >> >> - support alternates for genisoimage to test/vm >> - add clang++ to clang tests >> - fix record/replay smoke test >> - enable more softfloat tests >> - better detection of hung gdb >> - upgrade aarch64 tcg test x-compile to gcc-10 >> - fix plugin cpu_index clash vs threads > > This makes the NetBSD tests/vm setup spit out a huge pile of extra warnings: > every link has something like: > > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libglib-2.0.so, may > conflict with libintl.so.8 > > or some variant like: > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libcurl.so, may > conflict with libintl.so.8 > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may > conflict with libintl.so.8 I get these on commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: LINK arm-softmmu/qemu-system-arm ld: /usr/lib/libc.so and /usr/lib/crt0.o: warning: multiple common of `environ' ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 which makes me think the build environment has changed and it was simply that my PR triggered a re-build of the NetBSD VM. -- Alex Bennée
On Thu, 28 May 2020 at 15:57, Alex Bennée <alex.bennee@linaro.org> wrote: > I get these on commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: > > LINK arm-softmmu/qemu-system-arm > ld: /usr/lib/libc.so and /usr/lib/crt0.o: warning: multiple common of `environ' > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > > which makes me think the build environment has changed and it was simply > that my PR triggered a re-build of the NetBSD VM. Yes. I see them also now on the next pullreq I'm trying to process. How do I get back to an environment that doesn't produce all these warnings? thanks -- PMM
+Gerd On 5/28/20 5:09 PM, Peter Maydell wrote: > On Thu, 28 May 2020 at 15:57, Alex Bennée <alex.bennee@linaro.org> wrote: >> I get these on commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: >> >> LINK arm-softmmu/qemu-system-arm >> ld: /usr/lib/libc.so and /usr/lib/crt0.o: warning: multiple common of `environ' >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 >> >> which makes me think the build environment has changed and it was simply >> that my PR triggered a re-build of the NetBSD VM. > > Yes. I see them also now on the next pullreq I'm trying to > process. How do I get back to an environment that doesn't > produce all these warnings? Maybe building back NetBSD 8.1? $ git checkout 2cc3e591b3321a~ tests/vm/netbsd $ make vm-build-netbsd $ git checkout HEAD tests/vm/netbsd > > thanks > -- PMM >
On Thu, 28 May 2020 at 16:18, Philippe Mathieu-Daudé <philmd@redhat.com> wrote: > > +Gerd > > On 5/28/20 5:09 PM, Peter Maydell wrote: > > Yes. I see them also now on the next pullreq I'm trying to > > process. How do I get back to an environment that doesn't > > produce all these warnings? > > Maybe building back NetBSD 8.1? Did that change not itself trigger the VM to be rebuilt ? If not, can we make it so that updates to the images always do trigger rebuilds, so that we can catch new warnings that they introduce at the time rather than later? thanks -- PMM
On 5/28/20 5:20 PM, Peter Maydell wrote: > On Thu, 28 May 2020 at 16:18, Philippe Mathieu-Daudé <philmd@redhat.com> wrote: >> >> +Gerd >> >> On 5/28/20 5:09 PM, Peter Maydell wrote: >>> Yes. I see them also now on the next pullreq I'm trying to >>> process. How do I get back to an environment that doesn't >>> produce all these warnings? >> >> Maybe building back NetBSD 8.1? > > Did that change not itself trigger the VM to be rebuilt ? > If not, can we make it so that updates to the images always > do trigger rebuilds, so that we can catch new warnings > that they introduce at the time rather than later? This is how it should work... How do you run your VM builds?
On Thu, May 28, 2020 at 05:18:42PM +0200, Philippe Mathieu-Daudé wrote: > +Gerd > > On 5/28/20 5:09 PM, Peter Maydell wrote: > > On Thu, 28 May 2020 at 15:57, Alex Bennée <alex.bennee@linaro.org> wrote: > >> I get these on commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: > >> > >> LINK arm-softmmu/qemu-system-arm > >> ld: /usr/lib/libc.so and /usr/lib/crt0.o: warning: multiple common of `environ' > >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 > >> ld: warning: libintl.so.1, needed by /usr/pkg/lib/libgio-2.0.so, may conflict with libintl.so.8 Hmm, on a quick glance this looks like a netbsd packaging bug ... > Maybe building back NetBSD 8.1? > > $ git checkout 2cc3e591b3321a~ tests/vm/netbsd > $ make vm-build-netbsd > $ git checkout HEAD tests/vm/netbsd The *nsd installs fetch and install a bunch of addon packages, so when these are updated a netbsd image rebuild will use the newer versions. IIRC both tests/vm/netbsd and tests/vm/basevm.py changes will trigger and netbsd image rebuild. If you want avoid that you have to touch the disk image after git checkout so make thinks it is up-to-date and doesn't try to rebuild it. take care, Gerd
On Thu, 28 May 2020 at 16:26, Philippe Mathieu-Daudé <philmd@redhat.com> wrote: > > Did that change not itself trigger the VM to be rebuilt ? > > If not, can we make it so that updates to the images always > > do trigger rebuilds, so that we can catch new warnings > > that they introduce at the time rather than later? > > This is how it should work... How do you run your VM builds? git fetch pmaydell git checkout --quiet remotes/pmaydell/staging make -C build vm-build-netbsd -j8 V=1 thanks -- PMM
Peter Maydell <peter.maydell@linaro.org> writes: > On Thu, 28 May 2020 at 16:18, Philippe Mathieu-Daudé <philmd@redhat.com> wrote: >> >> +Gerd >> >> On 5/28/20 5:09 PM, Peter Maydell wrote: >> > Yes. I see them also now on the next pullreq I'm trying to >> > process. How do I get back to an environment that doesn't >> > produce all these warnings? >> >> Maybe building back NetBSD 8.1? > > Did that change not itself trigger the VM to be rebuilt ? > If not, can we make it so that updates to the images always > do trigger rebuilds, so that we can catch new warnings > that they introduce at the time rather than later? $(IMAGES_DIR)/%.img: $(SRC_PATH)/tests/vm/% \ So netbsdb.img should have been rebuilt when tests/vm/netbsd was touched. However maybe the packages have been updated upstream since? As we also have: $(SRC_PATH)/tests/vm/basevm.py \ $(SRC_PATH)/tests/vm/Makefile.include I suspect the genisoimage change to basevm triggered a re-build grabbing the latest packages. -- Alex Bennée
On Wed, 27 May 2020 at 15:54, Alex Bennée <alex.bennee@linaro.org> wrote: > > The following changes since commit ddc760832fa8cf5e93b9d9e6e854a5114ac63510: > > Merge remote-tracking branch 'remotes/gkurz/tags/9p-next-2020-05-26' into staging (2020-05-26 14:05:53 +0100) > > are available in the Git repository at: > > https://github.com/stsquad/qemu.git tags/pull-testing-tcg-plugins-270520-1 > > for you to fetch changes up to 919bfbf5d6569b63a374332292cf3d2355a6d6c3: > > tests/tcg: add new threadcount test (2020-05-27 14:26:49 +0100) > > ---------------------------------------------------------------- > Testing and one plugin fix: > > - support alternates for genisoimage to test/vm > - add clang++ to clang tests > - fix record/replay smoke test > - enable more softfloat tests > - better detection of hung gdb > - upgrade aarch64 tcg test x-compile to gcc-10 > - fix plugin cpu_index clash vs threads Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1 for any user-visible changes. -- PMM