* [Qemu-devel] [PATCH v1 0/8] testing/next
@ 2019-02-20 17:28 Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed Alex Bennée
` (9 more replies)
0 siblings, 10 replies; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Alex Bennée
Hi,
Here is the current status of testing/next which fixes some of the
current instability as well as some totally broken builds. However we
have two failures I'm still trying to track down:
tests/boot-sector.c:161:boot_sector_test:
assertion failed (signature == SIGNATURE): (0x0000face == 0x0000dead)
I have seen this locally and got a core dump but it doesn't show much.
I'm going to see if I can get more out of a debug build. It is
generated by:
tests/cdrom-test -m=quick -k --tap
I think the subtest is:
/x86_64/cdrom/boot/isapc
Attaching to the child QEMU looks like it is eternally returning
-EINTR and looping around with the occasionally kvm_handle_io to port
146 and port 112. I guess under test conditions this eventually times
out and dies.
The other is the failure in the gprof build:
PASS 55 ahci-test /x86_64/ahci/flush/migrate
tests/libqtest.c:143: kill_qemu() detected QEMU death from signal 11
(Segmentation fault) (core dumped)
ERROR - too few tests run (expected 66, got 55)
Aborted (core dumped)
/home/travis/build/stsquad/qemu/tests/Makefile.include:857: recipe
for target 'check-qtest-x86_64' failed
So far attempts to re-create this locally have failed. It may be a
Travis related environment thing.
The following patches need review
: patch 0002/.travis.yml split debug builds.patch
: patch 0005/tests docker squash initial update and install st.patch
: patch 0006/tests docker peg netmap code to a specific versio.patch
: patch 0008/tests softfloat always do quick softfloat tests.patch
Alex Bennée (4):
.travis.yml: split debug builds
tests/docker: squash initial update and install step for docker9
tests/docker: peg netmap code to a specific version
tests/softfloat: always do quick softfloat tests
Dr. David Alan Gilbert (2):
.travis.yml: Test with disable-replication
.travis.yml: Remove disable-uuid
Paolo Bonzini (1):
.travis.yml: the xcode10 image seems to be hosed
Thomas Huth (1):
Add a gitlab-ci file for Continuous Integration testing on Gitlab
.gitlab-ci.yml | 73 ++++++++++++++++++++
.travis.yml | 16 ++---
MAINTAINERS | 5 ++
tests/Makefile.include | 6 +-
tests/docker/dockerfiles/debian-amd64.docker | 1 +
tests/docker/dockerfiles/debian9.docker | 4 +-
6 files changed, 90 insertions(+), 15 deletions(-)
create mode 100644 .gitlab-ci.yml
--
2.20.1
^ permalink raw reply [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 19:08 ` Philippe Mathieu-Daudé
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 2/8] .travis.yml: split debug builds Alex Bennée
` (8 subsequent siblings)
9 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Alex Bennée
From: Paolo Bonzini <pbonzini@redhat.com>
It fails to install homebrew. Unfortunately we cannot mark
it as an expected failure because Travis does not match
allow_failures rows against include rows (only against the
main test matrix, which we do not use at all), so just disable
it.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <20190220105131.23479-1-pbonzini@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.travis.yml | 7 -------
1 file changed, 7 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index baa06b976a..866a1872b1 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -174,13 +174,6 @@ matrix:
compiler: clang
- - env:
- - CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
- os: osx
- osx_image: xcode10
- compiler: clang
-
-
# Python builds
- env:
- CONFIG="--target-list=x86_64-softmmu"
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 2/8] .travis.yml: split debug builds
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 18:35 ` Richard Henderson
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 3/8] .travis.yml: Test with disable-replication Alex Bennée
` (7 subsequent siblings)
9 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Alex Bennée
The builds are reaching the magic 50 minute limit with regularity so
lets split them up. Rather than doing a full debug build on both just
enable debug tcg for linux-user.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.travis.yml | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 866a1872b1..36ed7ec3f9 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -86,7 +86,12 @@ matrix:
- env:
- - CONFIG="--enable-debug --enable-debug-tcg"
+ - CONFIG="--enable-debug --enable-debug-tcg --disable-user"
+
+
+ # TCG debug can be run just on it's own and is mostly agnostic to user/softmmu distinctions
+ - env:
+ - CONFIG="--enable-debug-tcg --disable-system"
- env:
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 3/8] .travis.yml: Test with disable-replication
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 2/8] .travis.yml: split debug builds Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 4/8] .travis.yml: Remove disable-uuid Alex Bennée
` (6 subsequent siblings)
9 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel
Cc: pbonzini, peter.maydell, Dr. David Alan Gilbert,
Alex Bennée, Thomas Huth
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
We've had the build break with replication disabled, so lets
test that case in travis.
Suggsted-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20190215094502.32149-1-dgilbert@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 36ed7ec3f9..61b086b8b8 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -95,7 +95,7 @@ matrix:
- env:
- - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --disable-user"
+ - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --disable-user --disable-replication"
- env:
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 4/8] .travis.yml: Remove disable-uuid
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (2 preceding siblings ...)
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 3/8] .travis.yml: Test with disable-replication Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 5/8] tests/docker: squash initial update and install step for docker9 Alex Bennée
` (5 subsequent siblings)
9 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel
Cc: pbonzini, peter.maydell, Dr. David Alan Gilbert, Thomas Huth,
Alex Bennée
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Commit 315d3184525 turned --disable-uuid into a warning only; remove
the check from Travis.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20190215094502.32149-2-dgilbert@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index 61b086b8b8..cca57f4314 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -95,7 +95,7 @@ matrix:
- env:
- - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --disable-user --disable-replication"
+ - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-libusb --disable-user --disable-replication"
- env:
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 5/8] tests/docker: squash initial update and install step for docker9
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (3 preceding siblings ...)
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 4/8] .travis.yml: Remove disable-uuid Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 18:04 ` Paolo Bonzini
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 6/8] tests/docker: peg netmap code to a specific version Alex Bennée
` (4 subsequent siblings)
9 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Alex Bennée
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/docker/dockerfiles/debian9.docker | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/docker/dockerfiles/debian9.docker b/tests/docker/dockerfiles/debian9.docker
index 154ae2a455..5f23a35404 100644
--- a/tests/docker/dockerfiles/debian9.docker
+++ b/tests/docker/dockerfiles/debian9.docker
@@ -13,8 +13,8 @@ FROM debian:stretch-slim
RUN cat /etc/apt/sources.list | sed "s/^deb\ /deb-src /" >> /etc/apt/sources.list
# Install common build utilities
-RUN apt update
-RUN DEBIAN_FRONTEND=noninteractive apt install -yy eatmydata
+RUN apt-get update && \
+ DEBIAN_FRONTEND=noninteractive apt install -yy eatmydata
RUN DEBIAN_FRONTEND=noninteractive eatmydata \
apt install -y --no-install-recommends \
bison \
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 6/8] tests/docker: peg netmap code to a specific version
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (4 preceding siblings ...)
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 5/8] tests/docker: squash initial update and install step for docker9 Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 18:42 ` Richard Henderson
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 7/8] Add a gitlab-ci file for Continuous Integration testing on Gitlab Alex Bennée
` (3 subsequent siblings)
9 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Alex Bennée
Tracking head is always going to be at the whims of the upstream.
Let's use a defined release so things don't magically change under us.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/docker/dockerfiles/debian-amd64.docker | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/docker/dockerfiles/debian-amd64.docker b/tests/docker/dockerfiles/debian-amd64.docker
index 954fcf9606..d770a11a52 100644
--- a/tests/docker/dockerfiles/debian-amd64.docker
+++ b/tests/docker/dockerfiles/debian-amd64.docker
@@ -33,6 +33,7 @@ RUN DEBIAN_FRONTEND=noninteractive eatmydata \
apt-get install -y --no-install-recommends \
linux-headers-amd64
RUN git clone https://github.com/luigirizzo/netmap.git /usr/src/netmap
+RUN cd /usr/src/netmap && git checkout v11.3
RUN cd /usr/src/netmap/LINUX && ./configure --no-drivers --no-apps --kernel-dir=$(ls -d /usr/src/linux-headers-*-amd64) && make install
ENV QEMU_CONFIGURE_OPTS --enable-netmap
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 7/8] Add a gitlab-ci file for Continuous Integration testing on Gitlab
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (5 preceding siblings ...)
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 6/8] tests/docker: peg netmap code to a specific version Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 8/8] tests/softfloat: always do quick softfloat tests Alex Bennée
` (2 subsequent siblings)
9 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Thomas Huth, Alex Bennée
From: Thomas Huth <thuth@redhat.com>
This is very convenient for people like me who store their QEMU git trees
on gitlab.com: Automatic CI pipelines are now run for each branch that is
pushed to the server - useful for some extra-testing before sending PULL-
requests for example. Since the runtime of the jobs is limited to 1h, the
jobs are distributed into multiple pipelines - this way everything finishs
fine within time (ca. 30 minutes currently).
Signed-off-by: Thomas Huth <thuth@redhat.com>
Message-Id: <1550058881-16351-1-git-send-email-thuth@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.gitlab-ci.yml | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++
MAINTAINERS | 5 ++++
2 files changed, 78 insertions(+)
create mode 100644 .gitlab-ci.yml
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
new file mode 100644
index 0000000000..79d02cf740
--- /dev/null
+++ b/.gitlab-ci.yml
@@ -0,0 +1,73 @@
+before_script:
+ - apt-get update -qq
+ - apt-get install -y -qq flex bison libglib2.0-dev libpixman-1-dev genisoimage
+
+build-system1:
+ script:
+ - apt-get install -y -qq libgtk-3-dev libvte-dev nettle-dev libcacard-dev
+ libusb-dev libvde-dev libspice-protocol-dev libgl1-mesa-dev
+ - ./configure --enable-werror --target-list="aarch64-softmmu alpha-softmmu
+ cris-softmmu hppa-softmmu lm32-softmmu moxie-softmmu microblazeel-softmmu
+ mips64el-softmmu m68k-softmmu ppc-softmmu riscv64-softmmu sparc-softmmu"
+ - make -j2
+ - make -j2 check
+
+build-system2:
+ script:
+ - apt-get install -y -qq libsdl2-dev libgcrypt-dev libbrlapi-dev libaio-dev
+ libfdt-dev liblzo2-dev librdmacm-dev libibverbs-dev libibumad-dev
+ - ./configure --enable-werror --target-list="tricore-softmmu unicore32-softmmu
+ microblaze-softmmu mips-softmmu riscv32-softmmu s390x-softmmu sh4-softmmu
+ sparc64-softmmu x86_64-softmmu xtensa-softmmu nios2-softmmu or1k-softmmu"
+ - make -j2
+ - make -j2 check
+
+build-disabled:
+ script:
+ - ./configure --enable-werror --disable-rdma --disable-slirp --disable-curl
+ --disable-capstone --disable-live-block-migration --disable-glusterfs
+ --disable-replication --disable-coroutine-pool --disable-smartcard
+ --disable-guest-agent --disable-curses --disable-libxml2 --disable-tpm
+ --disable-qom-cast-debug --disable-spice --disable-vhost-vsock
+ --disable-vhost-net --disable-vhost-crypto --disable-vhost-user
+ --target-list="i386-softmmu ppc64-softmmu mips64-softmmu i386-linux-user"
+ - make -j2
+ - make -j2 check-qtest SPEED=slow
+
+build-tcg-disabled:
+ script:
+ - apt-get install -y -qq clang libgtk-3-dev libbluetooth-dev libusb-dev
+ - ./configure --cc=clang --enable-werror --disable-tcg --audio-drv-list=""
+ - make -j2
+ - make check-unit
+ - make check-qapi-schema
+ - cd tests/qemu-iotests/
+ - ./check -raw 001 002 003 004 005 008 009 010 011 012 021 025 032 033 048
+ 052 063 077 086 101 104 106 113 147 148 150 151 152 157 159 160
+ 163 170 171 183 184 192 194 197 205 208 215 221 222 226 227 236
+ - ./check -qcow2 001 002 003 004 005 007 008 009 010 011 012 013 017 018 019
+ 020 021 022 024 025 027 028 029 031 032 033 034 035 036 037 038
+ 039 040 042 043 046 047 048 049 050 051 052 053 054 056 057 058
+ 060 061 062 063 065 066 067 068 069 071 072 073 074 079 080 082
+ 085 086 089 090 091 095 096 097 098 099 102 103 104 105 107 108
+ 110 111 114 117 120 122 124 126 127 129 130 132 133 134 137 138
+ 139 140 141 142 143 144 145 147 150 151 152 154 155 156 157 158
+ 161 165 170 172 174 176 177 179 184 186 187 190 192 194 195 196
+ 197 200 202 203 205 208 209 214 215 216 217 218 222 226 227 229 234
+
+build-user:
+ script:
+ - ./configure --enable-werror --disable-system --disable-guest-agent
+ --disable-capstone --disable-slirp --disable-fdt
+ - make -j2
+ - make run-tcg-tests-i386-linux-user run-tcg-tests-x86_64-linux-user
+
+build-clang:
+ script:
+ - apt-get install -y -qq clang libsdl2-dev
+ xfslibs-dev libiscsi-dev libnfs-dev libseccomp-dev gnutls-dev librbd-dev
+ - ./configure --cc=clang --cxx=clang++ --enable-werror
+ --target-list="alpha-softmmu arm-softmmu m68k-softmmu mips64-softmmu
+ ppc-softmmu s390x-softmmu x86_64-softmmu arm-linux-user"
+ - make -j2
+ - make -j2 check
diff --git a/MAINTAINERS b/MAINTAINERS
index b1d786cfd8..c6f2976c18 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2479,6 +2479,11 @@ S: Maintained
F: .cirrus.yml
W: https://cirrus-ci.com/github/qemu/qemu
+GitLab Continuous Integration
+M: Thomas Huth <thuth@redhat.com>
+S: Maintained
+F: .gitlab-ci.yml
+
Guest Test Compilation Support
M: Alex Bennée <alex.bennee@linaro.org>
R: Philippe Mathieu-Daudé <f4bug@amsat.org>
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [Qemu-devel] [PATCH v1 8/8] tests/softfloat: always do quick softfloat tests
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (6 preceding siblings ...)
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 7/8] Add a gitlab-ci file for Continuous Integration testing on Gitlab Alex Bennée
@ 2019-02-20 17:28 ` Alex Bennée
2019-02-20 18:43 ` Richard Henderson
2019-02-20 19:07 ` [Qemu-devel] [PATCH v1 0/8] testing/next Philippe Mathieu-Daudé
2019-02-20 20:52 ` Alex Bennée
9 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 17:28 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell, Alex Bennée
Some operations take a long time and enabling "-l 2 -r all" can take
more than a day which is stretching the definition of a "slow" test.
Lets default to the quick test and leave a note for those who wish to
run by hand.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/Makefile.include | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/tests/Makefile.include b/tests/Makefile.include
index b39e989f72..3741f8f6dd 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -896,11 +896,9 @@ $(FP_TEST_BIN):
"BUILD", "$(notdir $@)")
# The full test suite can take a bit of time, default to a quick run
-ifeq ($(SPEED), quick)
+# "-l 2 -r all" can take more than a day for some operations and is best
+# run manually
FP_TL=-l 1
-else
-FP_TL=-l 2 -r all
-endif
# $1 = tests, $2 = description
test-softfloat = $(call quiet-command, \
--
2.20.1
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 5/8] tests/docker: squash initial update and install step for docker9
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 5/8] tests/docker: squash initial update and install step for docker9 Alex Bennée
@ 2019-02-20 18:04 ` Paolo Bonzini
0 siblings, 0 replies; 19+ messages in thread
From: Paolo Bonzini @ 2019-02-20 18:04 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: peter.maydell
Typo in subject, should be debian9
Paolo "no this is not top posting"
On 20/02/19 18:28, Alex Bennée wrote:
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/docker/dockerfiles/debian9.docker | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/docker/dockerfiles/debian9.docker b/tests/docker/dockerfiles/debian9.docker
> index 154ae2a455..5f23a35404 100644
> --- a/tests/docker/dockerfiles/debian9.docker
> +++ b/tests/docker/dockerfiles/debian9.docker
> @@ -13,8 +13,8 @@ FROM debian:stretch-slim
> RUN cat /etc/apt/sources.list | sed "s/^deb\ /deb-src /" >> /etc/apt/sources.list
>
> # Install common build utilities
> -RUN apt update
> -RUN DEBIAN_FRONTEND=noninteractive apt install -yy eatmydata
> +RUN apt-get update && \
> + DEBIAN_FRONTEND=noninteractive apt install -yy eatmydata
> RUN DEBIAN_FRONTEND=noninteractive eatmydata \
> apt install -y --no-install-recommends \
> bison \
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 2/8] .travis.yml: split debug builds
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 2/8] .travis.yml: split debug builds Alex Bennée
@ 2019-02-20 18:35 ` Richard Henderson
0 siblings, 0 replies; 19+ messages in thread
From: Richard Henderson @ 2019-02-20 18:35 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: pbonzini, peter.maydell
On 2/20/19 9:28 AM, Alex Bennée wrote:
> The builds are reaching the magic 50 minute limit with regularity so
> lets split them up. Rather than doing a full debug build on both just
> enable debug tcg for linux-user.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .travis.yml | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 6/8] tests/docker: peg netmap code to a specific version
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 6/8] tests/docker: peg netmap code to a specific version Alex Bennée
@ 2019-02-20 18:42 ` Richard Henderson
0 siblings, 0 replies; 19+ messages in thread
From: Richard Henderson @ 2019-02-20 18:42 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: pbonzini, peter.maydell
On 2/20/19 9:28 AM, Alex Bennée wrote:
> Tracking head is always going to be at the whims of the upstream.
> Let's use a defined release so things don't magically change under us.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/docker/dockerfiles/debian-amd64.docker | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 8/8] tests/softfloat: always do quick softfloat tests
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 8/8] tests/softfloat: always do quick softfloat tests Alex Bennée
@ 2019-02-20 18:43 ` Richard Henderson
0 siblings, 0 replies; 19+ messages in thread
From: Richard Henderson @ 2019-02-20 18:43 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: pbonzini, peter.maydell
On 2/20/19 9:28 AM, Alex Bennée wrote:
> Some operations take a long time and enabling "-l 2 -r all" can take
> more than a day which is stretching the definition of a "slow" test.
> Lets default to the quick test and leave a note for those who wish to
> run by hand.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/Makefile.include | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 0/8] testing/next
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (7 preceding siblings ...)
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 8/8] tests/softfloat: always do quick softfloat tests Alex Bennée
@ 2019-02-20 19:07 ` Philippe Mathieu-Daudé
2019-02-20 20:52 ` Alex Bennée
9 siblings, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-02-20 19:07 UTC (permalink / raw)
To: Alex Bennée, qemu-devel, pbonzini, Stefan Hajnoczi; +Cc: peter.maydell
On 2/20/19 6:28 PM, Alex Bennée wrote:
> Hi,
>
> Here is the current status of testing/next which fixes some of the
> current instability as well as some totally broken builds. However we
> have two failures I'm still trying to track down:
>
> tests/boot-sector.c:161:boot_sector_test:
> assertion failed (signature == SIGNATURE): (0x0000face == 0x0000dead)
>
> I have seen this locally and got a core dump but it doesn't show much.
> I'm going to see if I can get more out of a debug build. It is
> generated by:
>
> tests/cdrom-test -m=quick -k --tap
>
> I think the subtest is:
>
> /x86_64/cdrom/boot/isapc
>
> Attaching to the child QEMU looks like it is eternally returning
> -EINTR and looping around with the occasionally kvm_handle_io to port
> 146 and port 112. I guess under test conditions this eventually times
> out and dies.
>
> The other is the failure in the gprof build:
>
> PASS 55 ahci-test /x86_64/ahci/flush/migrate
> tests/libqtest.c:143: kill_qemu() detected QEMU death from signal 11
> (Segmentation fault) (core dumped)
> ERROR - too few tests run (expected 66, got 55)
> Aborted (core dumped)
> /home/travis/build/stsquad/qemu/tests/Makefile.include:857: recipe
> for target 'check-qtest-x86_64' failed
>
> So far attempts to re-create this locally have failed. It may be a
> Travis related environment thing.
On OSX (Xcode9.4) I still have:
ERROR:tests/test-aio.c:501:test_timer_schedule: assertion failed:
(aio_poll(ctx, true))
ERROR - Bail out! ERROR:tests/test-aio.c:501:test_timer_schedule:
assertion failed: (aio_poll(ctx, true))
make: *** [check-unit] Error 1
make: *** Waiting for unfinished jobs....
https://travis-ci.org/philmd/qemu/jobs/495761523
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed Alex Bennée
@ 2019-02-20 19:08 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-02-20 19:08 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: pbonzini, peter.maydell
On 2/20/19 6:28 PM, Alex Bennée wrote:
> From: Paolo Bonzini <pbonzini@redhat.com>
>
> It fails to install homebrew. Unfortunately we cannot mark
$ git config protocol.version
2
This is weird, Travis now enforce git protocol v2, but the brew version
they provide doesn't support it.
Maybe they don't care because now provide the Xcode 10.2 (as beta):
https://blog.travis-ci.com/2019-02-12-xcode-10-2-beta-2-is-now-available
Successful build:
https://travis-ci.org/philmd/qemu/jobs/496139928
I'll send a patch FWIW.
> it as an expected failure because Travis does not match
> allow_failures rows against include rows (only against the
> main test matrix, which we do not use at all), so just disable
> it.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Message-Id: <20190220105131.23479-1-pbonzini@redhat.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .travis.yml | 7 -------
> 1 file changed, 7 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index baa06b976a..866a1872b1 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -174,13 +174,6 @@ matrix:
> compiler: clang
>
>
> - - env:
> - - CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
> - os: osx
> - osx_image: xcode10
> - compiler: clang
> -
> -
> # Python builds
> - env:
> - CONFIG="--target-list=x86_64-softmmu"
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 0/8] testing/next
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
` (8 preceding siblings ...)
2019-02-20 19:07 ` [Qemu-devel] [PATCH v1 0/8] testing/next Philippe Mathieu-Daudé
@ 2019-02-20 20:52 ` Alex Bennée
2019-02-21 8:51 ` Thomas Huth
9 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-20 20:52 UTC (permalink / raw)
To: qemu-devel; +Cc: pbonzini, peter.maydell
Alex Bennée <alex.bennee@linaro.org> writes:
> Hi,
>
> tests/boot-sector.c:161:boot_sector_test:
> assertion failed (signature == SIGNATURE): (0x0000face == 0x0000dead)
>
> I have seen this locally and got a core dump but it doesn't show much.
> I'm going to see if I can get more out of a debug build. It is
> generated by:
>
> tests/cdrom-test -m=quick -k --tap
>
> I think the subtest is:
>
> /x86_64/cdrom/boot/isapc
>
> Attaching to the child QEMU looks like it is eternally returning
> -EINTR and looping around with the occasionally kvm_handle_io to port
> 146 and port 112. I guess under test conditions this eventually times
> out and dies.
<snip>
This is load related. I can run:
retry.py --timeout 600 -n 500 -c -- \
env QTEST_QEMU_BINARY=x86_64-softmmu/qemu-system-x86_64 \
QTEST_QEMU_IMG=qemu-img ./tests/cdrom-test \
-m=quick -p /x86_64/cdrom/boot/isapc
and then all 500 run fine. If I do the same while running a make check
check-tcg -j9 in another build directory it hangs within 20 odd
attempts.
Can anyone else reproduce this?
--
Alex Bennée
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 0/8] testing/next
2019-02-20 20:52 ` Alex Bennée
@ 2019-02-21 8:51 ` Thomas Huth
2019-02-21 13:00 ` Alex Bennée
0 siblings, 1 reply; 19+ messages in thread
From: Thomas Huth @ 2019-02-21 8:51 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: pbonzini, peter.maydell
On 20/02/2019 21.52, Alex Bennée wrote:
>
> Alex Bennée <alex.bennee@linaro.org> writes:
>
>> Hi,
>>
>> tests/boot-sector.c:161:boot_sector_test:
>> assertion failed (signature == SIGNATURE): (0x0000face == 0x0000dead)
>>
>> I have seen this locally and got a core dump but it doesn't show much.
>> I'm going to see if I can get more out of a debug build. It is
>> generated by:
>>
>> tests/cdrom-test -m=quick -k --tap
>>
>> I think the subtest is:
>>
>> /x86_64/cdrom/boot/isapc
>>
>> Attaching to the child QEMU looks like it is eternally returning
>> -EINTR and looping around with the occasionally kvm_handle_io to port
>> 146 and port 112. I guess under test conditions this eventually times
>> out and dies.
> <snip>
>
> This is load related. I can run:
>
> retry.py --timeout 600 -n 500 -c -- \
> env QTEST_QEMU_BINARY=x86_64-softmmu/qemu-system-x86_64 \
> QTEST_QEMU_IMG=qemu-img ./tests/cdrom-test \
> -m=quick -p /x86_64/cdrom/boot/isapc
>
> and then all 500 run fine. If I do the same while running a make check
> check-tcg -j9 in another build directory it hangs within 20 odd
> attempts.
>
> Can anyone else reproduce this?
I can't reproduce it here. Might be worth a try to check the BIOS output
in that case. Add this patch:
diff --git a/tests/cdrom-test.c b/tests/cdrom-test.c
index 14bd981..c38e016 100644
--- a/tests/cdrom-test.c
+++ b/tests/cdrom-test.c
@@ -132,7 +132,7 @@ static void add_x86_tests(void)
qtest_add_data_func("cdrom/boot/virtio-scsi",
"-device virtio-scsi -device scsi-cd,drive=cdr "
"-blockdev file,node-name=cdr,filename=", test_cdboot);
- qtest_add_data_func("cdrom/boot/isapc", "-M isapc "
+ qtest_add_data_func("cdrom/boot/isapc", "-M isapc -vga none -device sga -serial file:/tmp/stdio "
"-drive if=ide,media=cdrom,file=", test_cdboot);
qtest_add_data_func("cdrom/boot/am53c974",
"-device am53c974 -device scsi-cd,drive=cd1 "
... then check /tmp/stdio when it hangs.
Thomas
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 0/8] testing/next
2019-02-21 8:51 ` Thomas Huth
@ 2019-02-21 13:00 ` Alex Bennée
2019-02-21 16:21 ` Thomas Huth
0 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2019-02-21 13:00 UTC (permalink / raw)
To: Thomas Huth, John Snow, qemu-block, rth, ehabkost
Cc: qemu-devel, pbonzini, peter.maydell
Thomas Huth <thuth@redhat.com> writes:
> On 20/02/2019 21.52, Alex Bennée wrote:
>>
>> Alex Bennée <alex.bennee@linaro.org> writes:
>>
>>> Hi,
>>>
>>> tests/boot-sector.c:161:boot_sector_test:
>>> assertion failed (signature == SIGNATURE): (0x0000face == 0x0000dead)
>>>
>>> I have seen this locally and got a core dump but it doesn't show much.
>>> I'm going to see if I can get more out of a debug build. It is
>>> generated by:
>>>
>>> tests/cdrom-test -m=quick -k --tap
>>>
>>> I think the subtest is:
>>>
>>> /x86_64/cdrom/boot/isapc
>>>
>>> Attaching to the child QEMU looks like it is eternally returning
>>> -EINTR and looping around with the occasionally kvm_handle_io to port
>>> 146 and port 112. I guess under test conditions this eventually times
>>> out and dies.
>> <snip>
>>
>> This is load related. I can run:
>>
>> retry.py --timeout 600 -n 500 -c -- \
>> env QTEST_QEMU_BINARY=x86_64-softmmu/qemu-system-x86_64 \
>> QTEST_QEMU_IMG=qemu-img ./tests/cdrom-test \
>> -m=quick -p /x86_64/cdrom/boot/isapc
>>
>> and then all 500 run fine. If I do the same while running a make check
>> check-tcg -j9 in another build directory it hangs within 20 odd
>> attempts.
>>
>> Can anyone else reproduce this?
I should also mention my current config for the reproducer test is:
'../../configure' '--python=python3' '--disable-docs' '--disable-tools' '--disable-libusb' '--disable-vte' '--disable-xen' '--enable-debug' '--extra-cflags=-O0 -g3' '--target-list=x86_64-softmmu'
>
> I can't reproduce it here. Might be worth a try to check the BIOS output
> in that case. Add this patch:
>
> diff --git a/tests/cdrom-test.c b/tests/cdrom-test.c
> index 14bd981..c38e016 100644
> --- a/tests/cdrom-test.c
> +++ b/tests/cdrom-test.c
> @@ -132,7 +132,7 @@ static void add_x86_tests(void)
> qtest_add_data_func("cdrom/boot/virtio-scsi",
> "-device virtio-scsi -device scsi-cd,drive=cdr "
> "-blockdev file,node-name=cdr,filename=", test_cdboot);
> - qtest_add_data_func("cdrom/boot/isapc", "-M isapc "
> + qtest_add_data_func("cdrom/boot/isapc", "-M isapc -vga none -device sga -serial file:/tmp/stdio "
> "-drive if=ide,media=cdrom,file=", test_cdboot);
> qtest_add_data_func("cdrom/boot/am53c974",
> "-device am53c974 -device scsi-cd,drive=cd1 "
>
> ... then check /tmp/stdio when it hangs.
This gets us:
SeaBIOS (version rel-1.12.0-0-ga698c8995f-prebuilt.qemu.org)
Booting from Floppy...
Boot failed: could not read the boot disk
Booting from DVD/CD...
Boot failed: Could not read from CDROM (code 000c)
Booting from DVD/CD...
Boot failed: Could not read from CDROM (code 0003)
Booting from Hard Disk...
Boot failed: could not read the boot disk
No bootable device.
I tried to bisect but this occurs even in v3.0.0
SeaBIOS (version rel-1.11.2-0-gf9626ccb91-prebuilt.qemu-project.org)
Booting from Floppy...
Boot failed: could not read the boot disk
Booting from DVD/CD...
Boot failed: Could not read from CDROM (code 000c)
Booting from DVD/CD...
Boot failed: Could not read from CDROM (code 0003)
Booting from Hard Disk...
Boot failed: could not read the boot disk
No bootable device.
Can I pass this over to the block/bios people to look at. I think at
this point I think the test has always been unstable but tends only to
trigger the fault on Travis because it is highly loaded.
In the meantime I'll see if I can disable the test for the CI runs.
--
Alex Bennée
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [Qemu-devel] [PATCH v1 0/8] testing/next
2019-02-21 13:00 ` Alex Bennée
@ 2019-02-21 16:21 ` Thomas Huth
0 siblings, 0 replies; 19+ messages in thread
From: Thomas Huth @ 2019-02-21 16:21 UTC (permalink / raw)
To: Alex Bennée, John Snow, qemu-block, rth, ehabkost
Cc: qemu-devel, pbonzini, peter.maydell
On 21/02/2019 14.00, Alex Bennée wrote:
>
> Thomas Huth <thuth@redhat.com> writes:
>
>> On 20/02/2019 21.52, Alex Bennée wrote:
>>>
>>> Alex Bennée <alex.bennee@linaro.org> writes:
>>>
>>>> Hi,
>>>>
>>>> tests/boot-sector.c:161:boot_sector_test:
>>>> assertion failed (signature == SIGNATURE): (0x0000face == 0x0000dead)
>>>>
>>>> I have seen this locally and got a core dump but it doesn't show much.
>>>> I'm going to see if I can get more out of a debug build. It is
>>>> generated by:
>>>>
>>>> tests/cdrom-test -m=quick -k --tap
>>>>
>>>> I think the subtest is:
>>>>
>>>> /x86_64/cdrom/boot/isapc
>>>>
>>>> Attaching to the child QEMU looks like it is eternally returning
>>>> -EINTR and looping around with the occasionally kvm_handle_io to port
>>>> 146 and port 112. I guess under test conditions this eventually times
>>>> out and dies.
>>> <snip>
>>>
>>> This is load related. I can run:
>>>
>>> retry.py --timeout 600 -n 500 -c -- \
>>> env QTEST_QEMU_BINARY=x86_64-softmmu/qemu-system-x86_64 \
>>> QTEST_QEMU_IMG=qemu-img ./tests/cdrom-test \
>>> -m=quick -p /x86_64/cdrom/boot/isapc
>>>
>>> and then all 500 run fine. If I do the same while running a make check
>>> check-tcg -j9 in another build directory it hangs within 20 odd
>>> attempts.
>>>
>>> Can anyone else reproduce this?
>
> I should also mention my current config for the reproducer test is:
>
> '../../configure' '--python=python3' '--disable-docs' '--disable-tools' '--disable-libusb' '--disable-vte' '--disable-xen' '--enable-debug' '--extra-cflags=-O0 -g3' '--target-list=x86_64-softmmu'
>
>>
>> I can't reproduce it here. Might be worth a try to check the BIOS output
>> in that case. Add this patch:
>>
>> diff --git a/tests/cdrom-test.c b/tests/cdrom-test.c
>> index 14bd981..c38e016 100644
>> --- a/tests/cdrom-test.c
>> +++ b/tests/cdrom-test.c
>> @@ -132,7 +132,7 @@ static void add_x86_tests(void)
>> qtest_add_data_func("cdrom/boot/virtio-scsi",
>> "-device virtio-scsi -device scsi-cd,drive=cdr "
>> "-blockdev file,node-name=cdr,filename=", test_cdboot);
>> - qtest_add_data_func("cdrom/boot/isapc", "-M isapc "
>> + qtest_add_data_func("cdrom/boot/isapc", "-M isapc -vga none -device sga -serial file:/tmp/stdio "
>> "-drive if=ide,media=cdrom,file=", test_cdboot);
>> qtest_add_data_func("cdrom/boot/am53c974",
>> "-device am53c974 -device scsi-cd,drive=cd1 "
>>
>> ... then check /tmp/stdio when it hangs.
>
> This gets us:
>
> SeaBIOS (version rel-1.12.0-0-ga698c8995f-prebuilt.qemu.org)
> Booting from Floppy...
> Boot failed: could not read the boot disk
>
> Booting from DVD/CD...
> Boot failed: Could not read from CDROM (code 000c)
> Booting from DVD/CD...
> Boot failed: Could not read from CDROM (code 0003)
> Booting from Hard Disk...
> Boot failed: could not read the boot disk
>
> No bootable device.
>
> I tried to bisect but this occurs even in v3.0.0
Weird, if it also occurs with 3.0 already, why didn't we notice this
earlier?
> In the meantime I'll see if I can disable the test for the CI runs.
Ok, but that likely means nobody is going to fix it anymore ... anyway,
if we still keep it for the SPEED=slow mode, we will at least still
notice if it breaks completely one day...
Thomas
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2019-02-21 16:21 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-20 17:28 [Qemu-devel] [PATCH v1 0/8] testing/next Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 1/8] .travis.yml: the xcode10 image seems to be hosed Alex Bennée
2019-02-20 19:08 ` Philippe Mathieu-Daudé
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 2/8] .travis.yml: split debug builds Alex Bennée
2019-02-20 18:35 ` Richard Henderson
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 3/8] .travis.yml: Test with disable-replication Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 4/8] .travis.yml: Remove disable-uuid Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 5/8] tests/docker: squash initial update and install step for docker9 Alex Bennée
2019-02-20 18:04 ` Paolo Bonzini
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 6/8] tests/docker: peg netmap code to a specific version Alex Bennée
2019-02-20 18:42 ` Richard Henderson
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 7/8] Add a gitlab-ci file for Continuous Integration testing on Gitlab Alex Bennée
2019-02-20 17:28 ` [Qemu-devel] [PATCH v1 8/8] tests/softfloat: always do quick softfloat tests Alex Bennée
2019-02-20 18:43 ` Richard Henderson
2019-02-20 19:07 ` [Qemu-devel] [PATCH v1 0/8] testing/next Philippe Mathieu-Daudé
2019-02-20 20:52 ` Alex Bennée
2019-02-21 8:51 ` Thomas Huth
2019-02-21 13:00 ` Alex Bennée
2019-02-21 16:21 ` Thomas Huth
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.