* [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting)
@ 2022-07-25 14:05 Alex Bennée
2022-07-25 14:05 ` [PATCH v1 01/13] tests: refresh to latest libvirt-ci module Alex Bennée
` (12 more replies)
0 siblings, 13 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Alex Bennée
As per usual I've opened up a tree for fixes for the 7.1 release. It
started as testing/next but I've included some fixes for semihosting
and a few minor doc updates as well. I'll roll a PR from this at the
end of the week (unless it doesn't meet the bar for missing rc0
tomorrow).
Alex Bennée (2):
docs/devel: fix description of OBJECT_DECLARE_SIMPLE_TYPE
qemu-options: bring the kernel and image options together
Bin Meng (2):
.cirrus.yml: Change winsymlinks to 'native'
.gitlab-ci.d/windows.yml: Enable native Windows symlink
Daniel P. Berrangé (3):
tests: refresh to latest libvirt-ci module
gitlab: show testlog.txt contents when cirrus/custom-runner jobs fail
gitlab: drop 'containers-layer2' stage
Ilya Leoshkevich (2):
qapi: Add exit-failure PanicAction
tests/tcg/s390x: Test unaligned accesses to lowcore
Peter Maydell (4):
semihosting: Don't return negative values on
qemu_semihosting_console_write() failure
semihosting: Don't copy buffer after console_write()
semihosting: Check for errors on SET_ARG()
semihosting: Fix handling of buffer in TARGET_SYS_TMPNAM
docs/devel/qom.rst | 3 +-
qapi/run-state.json | 4 +-
include/sysemu/sysemu.h | 2 +-
semihosting/arm-compat-semi.c | 29 +++++-
semihosting/console.c | 3 +-
semihosting/syscalls.c | 2 +-
softmmu/main.c | 6 +-
softmmu/runstate.c | 17 +++-
.cirrus.yml | 2 +-
.gitlab-ci.d/cirrus/build.yml | 3 +-
.gitlab-ci.d/cirrus/freebsd-12.vars | 3 +-
.gitlab-ci.d/cirrus/freebsd-13.vars | 3 +-
.gitlab-ci.d/cirrus/macos-11.vars | 4 +-
.gitlab-ci.d/container-cross.yml | 24 ++---
.../custom-runners/centos-stream-8-x86_64.yml | 2 +
.../custom-runners/ubuntu-20.04-aarch32.yml | 2 +
.../custom-runners/ubuntu-20.04-aarch64.yml | 12 +++
.../custom-runners/ubuntu-20.04-s390x.yml | 12 +++
.gitlab-ci.d/stages.yml | 1 -
.gitlab-ci.d/windows.yml | 2 +
qemu-options.hx | 98 +++++++++++++++----
tests/docker/dockerfiles/alpine.docker | 4 +-
tests/docker/dockerfiles/centos8.docker | 6 +-
tests/docker/dockerfiles/debian-amd64.docker | 2 +
.../dockerfiles/debian-arm64-cross.docker | 2 +
.../dockerfiles/debian-armel-cross.docker | 2 +
.../dockerfiles/debian-armhf-cross.docker | 2 +
.../dockerfiles/debian-mips64el-cross.docker | 2 +
.../dockerfiles/debian-mipsel-cross.docker | 2 +
.../dockerfiles/debian-ppc64el-cross.docker | 2 +
.../dockerfiles/debian-s390x-cross.docker | 2 +
tests/docker/dockerfiles/fedora.docker | 3 +-
tests/docker/dockerfiles/opensuse-leap.docker | 7 +-
tests/docker/dockerfiles/ubuntu2004.docker | 2 +
tests/lcitool/libvirt-ci | 2 +-
tests/lcitool/projects/qemu.yml | 6 +-
tests/lcitool/refresh | 4 +-
tests/tcg/s390x/Makefile.softmmu-target | 9 ++
tests/tcg/s390x/unaligned-lowcore.S | 19 ++++
39 files changed, 242 insertions(+), 70 deletions(-)
create mode 100644 tests/tcg/s390x/Makefile.softmmu-target
create mode 100644 tests/tcg/s390x/unaligned-lowcore.S
--
2.30.2
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH v1 01/13] tests: refresh to latest libvirt-ci module
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 18:56 ` Richard Henderson
2022-07-26 14:34 ` Philippe Mathieu-Daudé via
2022-07-25 14:05 ` [PATCH v1 02/13] gitlab: show testlog.txt contents when cirrus/custom-runner jobs fail Alex Bennée
` (11 subsequent siblings)
12 siblings, 2 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Thomas Huth, Alex Bennée, Ed Maste,
Li-Wen Hsu, Wainer dos Santos Moschetta, Beraldo Leal
From: Daniel P. Berrangé <berrange@redhat.com>
Notable changes:
- libvirt-ci source tree was re-arranged, so the script we
run now lives in a bin/ sub-dir
- opensuse 15.2 is replaced by opensuse 15.3
- libslirp is temporarily dropped on opensuse as the
libslirp-version.h is broken
https://bugzilla.opensuse.org/show_bug.cgi?id=1201551
- The incorrectly named python3-virtualenv module was
changed to python3-venv, but most distros don't need
any package as 'venv' is a standard part of python
- glibc-static was renamed to libc-static, to reflect
fact that it isn't going to be glibc on all distros
- The cmocka/json-c deps that were manually added to
the centos dockerfile and are now consistently added
to all targets
Acked-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20220722130431.2319019-2-berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.gitlab-ci.d/cirrus/freebsd-12.vars | 3 +--
.gitlab-ci.d/cirrus/freebsd-13.vars | 3 +--
.gitlab-ci.d/cirrus/macos-11.vars | 4 ++--
tests/docker/dockerfiles/alpine.docker | 4 +++-
tests/docker/dockerfiles/centos8.docker | 6 +++---
tests/docker/dockerfiles/debian-amd64.docker | 2 ++
tests/docker/dockerfiles/debian-arm64-cross.docker | 2 ++
tests/docker/dockerfiles/debian-armel-cross.docker | 2 ++
tests/docker/dockerfiles/debian-armhf-cross.docker | 2 ++
tests/docker/dockerfiles/debian-mips64el-cross.docker | 2 ++
tests/docker/dockerfiles/debian-mipsel-cross.docker | 2 ++
tests/docker/dockerfiles/debian-ppc64el-cross.docker | 2 ++
tests/docker/dockerfiles/debian-s390x-cross.docker | 2 ++
tests/docker/dockerfiles/fedora.docker | 3 ++-
tests/docker/dockerfiles/opensuse-leap.docker | 7 ++++---
tests/docker/dockerfiles/ubuntu2004.docker | 2 ++
tests/lcitool/libvirt-ci | 2 +-
tests/lcitool/projects/qemu.yml | 6 ++++--
tests/lcitool/refresh | 4 ++--
19 files changed, 41 insertions(+), 19 deletions(-)
diff --git a/.gitlab-ci.d/cirrus/freebsd-12.vars b/.gitlab-ci.d/cirrus/freebsd-12.vars
index f59263731f..8fa5a320e9 100644
--- a/.gitlab-ci.d/cirrus/freebsd-12.vars
+++ b/.gitlab-ci.d/cirrus/freebsd-12.vars
@@ -1,5 +1,4 @@
# THIS FILE WAS AUTO-GENERATED
-# ... and then edited to fix py39, pending proper lcitool update.
#
# $ lcitool variables freebsd-12 qemu
#
@@ -12,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
NINJA='/usr/local/bin/ninja'
PACKAGING_COMMAND='pkg'
PIP3='/usr/local/bin/pip-3.8'
-PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-virtualenv py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
+PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage cmocka ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 json-c libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
PYPI_PKGS=''
PYTHON='/usr/local/bin/python3'
diff --git a/.gitlab-ci.d/cirrus/freebsd-13.vars b/.gitlab-ci.d/cirrus/freebsd-13.vars
index 40fc961398..8ed7e33a77 100644
--- a/.gitlab-ci.d/cirrus/freebsd-13.vars
+++ b/.gitlab-ci.d/cirrus/freebsd-13.vars
@@ -1,5 +1,4 @@
# THIS FILE WAS AUTO-GENERATED
-# ... and then edited to fix py39, pending proper lcitool update.
#
# $ lcitool variables freebsd-13 qemu
#
@@ -12,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
NINJA='/usr/local/bin/ninja'
PACKAGING_COMMAND='pkg'
PIP3='/usr/local/bin/pip-3.8'
-PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-virtualenv py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
+PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage cmocka ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 json-c libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
PYPI_PKGS=''
PYTHON='/usr/local/bin/python3'
diff --git a/.gitlab-ci.d/cirrus/macos-11.vars b/.gitlab-ci.d/cirrus/macos-11.vars
index cfe9181fd4..bec6e862d4 100644
--- a/.gitlab-ci.d/cirrus/macos-11.vars
+++ b/.gitlab-ci.d/cirrus/macos-11.vars
@@ -11,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
NINJA='/usr/local/bin/ninja'
PACKAGING_COMMAND='brew'
PIP3='/usr/local/bin/pip3'
-PKGS='bash bc bzip2 capstone ccache ctags curl dbus diffutils dtc gcovr gettext git glib gnu-sed gnutls gtk+3 jemalloc jpeg-turbo libepoxy libffi libgcrypt libiscsi libnfs libpng libslirp libssh libtasn1 libusb llvm lzo make meson ncurses nettle ninja perl pixman pkg-config python3 rpm2cpio sdl2 sdl2_image snappy sparse spice-protocol tesseract texinfo usbredir vde vte3 zlib zstd'
-PYPI_PKGS='PyYAML numpy pillow sphinx sphinx-rtd-theme virtualenv'
+PKGS='bash bc bzip2 capstone ccache cmocka ctags curl dbus diffutils dtc gcovr gettext git glib gnu-sed gnutls gtk+3 jemalloc jpeg-turbo json-c libepoxy libffi libgcrypt libiscsi libnfs libpng libslirp libssh libtasn1 libusb llvm lzo make meson ncurses nettle ninja perl pixman pkg-config python3 rpm2cpio sdl2 sdl2_image snappy sparse spice-protocol tesseract texinfo usbredir vde vte3 zlib zstd'
+PYPI_PKGS='PyYAML numpy pillow sphinx sphinx-rtd-theme'
PYTHON='/usr/local/bin/python3'
diff --git a/tests/docker/dockerfiles/alpine.docker b/tests/docker/dockerfiles/alpine.docker
index 3f4c0f95cb..806cb19f17 100644
--- a/tests/docker/dockerfiles/alpine.docker
+++ b/tests/docker/dockerfiles/alpine.docker
@@ -21,6 +21,7 @@ RUN apk update && \
cdrkit \
ceph-dev \
clang \
+ cmocka-dev \
ctags \
curl-dev \
cyrus-sasl-dev \
@@ -39,6 +40,7 @@ RUN apk update && \
glib-static \
gnutls-dev \
gtk+3.0-dev \
+ json-c-dev \
libaio-dev \
libbpf-dev \
libcap-ng-dev \
@@ -64,6 +66,7 @@ RUN apk update && \
mesa-dev \
meson \
multipath-tools \
+ musl-dev \
ncurses-dev \
ndctl-dev \
net-tools \
@@ -81,7 +84,6 @@ RUN apk update && \
py3-pip \
py3-sphinx \
py3-sphinx_rtd_theme \
- py3-virtualenv \
py3-yaml \
python3 \
rpm2cpio \
diff --git a/tests/docker/dockerfiles/centos8.docker b/tests/docker/dockerfiles/centos8.docker
index 10618bfa83..3c29883332 100644
--- a/tests/docker/dockerfiles/centos8.docker
+++ b/tests/docker/dockerfiles/centos8.docker
@@ -6,11 +6,12 @@
FROM quay.io/centos/centos:stream8
-RUN dnf update -y && \
+RUN dnf distro-sync -y && \
dnf install 'dnf-command(config-manager)' -y && \
dnf config-manager --set-enabled -y powertools && \
dnf install -y centos-release-advanced-virtualization && \
dnf install -y epel-release && \
+ dnf install -y epel-next-release && \
dnf install -y \
SDL2-devel \
alsa-lib-devel \
@@ -45,6 +46,7 @@ RUN dnf update -y && \
gtk3-devel \
hostname \
jemalloc-devel \
+ json-c-devel \
libaio-devel \
libasan \
libattr-devel \
@@ -60,7 +62,6 @@ RUN dnf update -y && \
libgcrypt-devel \
libiscsi-devel \
libjpeg-devel \
- json-c-devel \
libnfs-devel \
libpmem-devel \
libpng-devel \
@@ -99,7 +100,6 @@ RUN dnf update -y && \
python3-pip \
python3-sphinx \
python3-sphinx_rtd_theme \
- python3-virtualenv \
rdma-core-devel \
rpm \
sed \
diff --git a/tests/docker/dockerfiles/debian-amd64.docker b/tests/docker/dockerfiles/debian-amd64.docker
index 503e282802..8d78ba2484 100644
--- a/tests/docker/dockerfiles/debian-amd64.docker
+++ b/tests/docker/dockerfiles/debian-amd64.docker
@@ -41,6 +41,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev \
libcap-ng-dev \
libcapstone-dev \
+ libcmocka-dev \
libcurl4-gnutls-dev \
libdaxctl-dev \
libdrm-dev \
@@ -59,6 +60,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev \
libjemalloc-dev \
libjpeg62-turbo-dev \
+ libjson-c-dev \
liblttng-ust-dev \
liblzo2-dev \
libncursesw5-dev \
diff --git a/tests/docker/dockerfiles/debian-arm64-cross.docker b/tests/docker/dockerfiles/debian-arm64-cross.docker
index b60426834c..b7ba2c527f 100644
--- a/tests/docker/dockerfiles/debian-arm64-cross.docker
+++ b/tests/docker/dockerfiles/debian-arm64-cross.docker
@@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:arm64 \
libcap-ng-dev:arm64 \
libcapstone-dev:arm64 \
+ libcmocka-dev:arm64 \
libcurl4-gnutls-dev:arm64 \
libdaxctl-dev:arm64 \
libdrm-dev:arm64 \
@@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:arm64 \
libjemalloc-dev:arm64 \
libjpeg62-turbo-dev:arm64 \
+ libjson-c-dev:arm64 \
liblttng-ust-dev:arm64 \
liblzo2-dev:arm64 \
libncursesw5-dev:arm64 \
diff --git a/tests/docker/dockerfiles/debian-armel-cross.docker b/tests/docker/dockerfiles/debian-armel-cross.docker
index a6153e5a83..9b1778261e 100644
--- a/tests/docker/dockerfiles/debian-armel-cross.docker
+++ b/tests/docker/dockerfiles/debian-armel-cross.docker
@@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:armel \
libcap-ng-dev:armel \
libcapstone-dev:armel \
+ libcmocka-dev:armel \
libcurl4-gnutls-dev:armel \
libdaxctl-dev:armel \
libdrm-dev:armel \
@@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:armel \
libjemalloc-dev:armel \
libjpeg62-turbo-dev:armel \
+ libjson-c-dev:armel \
liblttng-ust-dev:armel \
liblzo2-dev:armel \
libncursesw5-dev:armel \
diff --git a/tests/docker/dockerfiles/debian-armhf-cross.docker b/tests/docker/dockerfiles/debian-armhf-cross.docker
index a2ebce96f8..addbc9a793 100644
--- a/tests/docker/dockerfiles/debian-armhf-cross.docker
+++ b/tests/docker/dockerfiles/debian-armhf-cross.docker
@@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:armhf \
libcap-ng-dev:armhf \
libcapstone-dev:armhf \
+ libcmocka-dev:armhf \
libcurl4-gnutls-dev:armhf \
libdaxctl-dev:armhf \
libdrm-dev:armhf \
@@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:armhf \
libjemalloc-dev:armhf \
libjpeg62-turbo-dev:armhf \
+ libjson-c-dev:armhf \
liblttng-ust-dev:armhf \
liblzo2-dev:armhf \
libncursesw5-dev:armhf \
diff --git a/tests/docker/dockerfiles/debian-mips64el-cross.docker b/tests/docker/dockerfiles/debian-mips64el-cross.docker
index b02dcb7fd9..1bb7d8e184 100644
--- a/tests/docker/dockerfiles/debian-mips64el-cross.docker
+++ b/tests/docker/dockerfiles/debian-mips64el-cross.docker
@@ -83,6 +83,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:mips64el \
libcap-ng-dev:mips64el \
libcapstone-dev:mips64el \
+ libcmocka-dev:mips64el \
libcurl4-gnutls-dev:mips64el \
libdaxctl-dev:mips64el \
libdrm-dev:mips64el \
@@ -101,6 +102,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:mips64el \
libjemalloc-dev:mips64el \
libjpeg62-turbo-dev:mips64el \
+ libjson-c-dev:mips64el \
liblttng-ust-dev:mips64el \
liblzo2-dev:mips64el \
libncursesw5-dev:mips64el \
diff --git a/tests/docker/dockerfiles/debian-mipsel-cross.docker b/tests/docker/dockerfiles/debian-mipsel-cross.docker
index b6d99ae324..a94b459b23 100644
--- a/tests/docker/dockerfiles/debian-mipsel-cross.docker
+++ b/tests/docker/dockerfiles/debian-mipsel-cross.docker
@@ -83,6 +83,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:mipsel \
libcap-ng-dev:mipsel \
libcapstone-dev:mipsel \
+ libcmocka-dev:mipsel \
libcurl4-gnutls-dev:mipsel \
libdaxctl-dev:mipsel \
libdrm-dev:mipsel \
@@ -101,6 +102,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:mipsel \
libjemalloc-dev:mipsel \
libjpeg62-turbo-dev:mipsel \
+ libjson-c-dev:mipsel \
liblttng-ust-dev:mipsel \
liblzo2-dev:mipsel \
libncursesw5-dev:mipsel \
diff --git a/tests/docker/dockerfiles/debian-ppc64el-cross.docker b/tests/docker/dockerfiles/debian-ppc64el-cross.docker
index bcf04bc90b..c641fd5d0e 100644
--- a/tests/docker/dockerfiles/debian-ppc64el-cross.docker
+++ b/tests/docker/dockerfiles/debian-ppc64el-cross.docker
@@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:ppc64el \
libcap-ng-dev:ppc64el \
libcapstone-dev:ppc64el \
+ libcmocka-dev:ppc64el \
libcurl4-gnutls-dev:ppc64el \
libdaxctl-dev:ppc64el \
libdrm-dev:ppc64el \
@@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:ppc64el \
libjemalloc-dev:ppc64el \
libjpeg62-turbo-dev:ppc64el \
+ libjson-c-dev:ppc64el \
liblttng-ust-dev:ppc64el \
liblzo2-dev:ppc64el \
libncursesw5-dev:ppc64el \
diff --git a/tests/docker/dockerfiles/debian-s390x-cross.docker b/tests/docker/dockerfiles/debian-s390x-cross.docker
index ff79a2cc4f..c0cbe09e66 100644
--- a/tests/docker/dockerfiles/debian-s390x-cross.docker
+++ b/tests/docker/dockerfiles/debian-s390x-cross.docker
@@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev:s390x \
libcap-ng-dev:s390x \
libcapstone-dev:s390x \
+ libcmocka-dev:s390x \
libcurl4-gnutls-dev:s390x \
libdaxctl-dev:s390x \
libdrm-dev:s390x \
@@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev:s390x \
libjemalloc-dev:s390x \
libjpeg62-turbo-dev:s390x \
+ libjson-c-dev:s390x \
liblttng-ust-dev:s390x \
liblzo2-dev:s390x \
libncursesw5-dev:s390x \
diff --git a/tests/docker/dockerfiles/fedora.docker b/tests/docker/dockerfiles/fedora.docker
index 2e6a84abfd..b39d311bbc 100644
--- a/tests/docker/dockerfiles/fedora.docker
+++ b/tests/docker/dockerfiles/fedora.docker
@@ -53,12 +53,14 @@ exec "$@"' > /usr/bin/nosync && \
gtk3-devel \
hostname \
jemalloc-devel \
+ json-c-devel \
libaio-devel \
libasan \
libattr-devel \
libbpf-devel \
libcacard-devel \
libcap-ng-devel \
+ libcmocka-devel \
libcurl-devel \
libdrm-devel \
libepoxy-devel \
@@ -106,7 +108,6 @@ exec "$@"' > /usr/bin/nosync && \
python3-pip \
python3-sphinx \
python3-sphinx_rtd_theme \
- python3-virtualenv \
rdma-core-devel \
rpm \
sed \
diff --git a/tests/docker/dockerfiles/opensuse-leap.docker b/tests/docker/dockerfiles/opensuse-leap.docker
index 6f5993d602..047a435ab5 100644
--- a/tests/docker/dockerfiles/opensuse-leap.docker
+++ b/tests/docker/dockerfiles/opensuse-leap.docker
@@ -1,10 +1,10 @@
# THIS FILE WAS AUTO-GENERATED
#
-# $ lcitool dockerfile --layers all opensuse-leap-152 qemu
+# $ lcitool dockerfile --layers all opensuse-leap-153 qemu
#
# https://gitlab.com/libvirt/libvirt-ci
-FROM registry.opensuse.org/opensuse/leap:15.2
+FROM registry.opensuse.org/opensuse/leap:15.3
RUN zypper update -y && \
zypper install -y \
@@ -44,6 +44,7 @@ RUN zypper update -y && \
libbz2-devel \
libcacard-devel \
libcap-ng-devel \
+ libcmocka-devel \
libcurl-devel \
libdrm-devel \
libepoxy-devel \
@@ -53,6 +54,7 @@ RUN zypper update -y && \
libgnutls-devel \
libiscsi-devel \
libjpeg8-devel \
+ libjson-c-devel \
libndctl-devel \
libnettle-devel \
libnfs-devel \
@@ -94,7 +96,6 @@ RUN zypper update -y && \
python3-pip \
python3-setuptools \
python3-sphinx_rtd_theme \
- python3-virtualenv \
python3-wheel \
rdma-core-devel \
rpm \
diff --git a/tests/docker/dockerfiles/ubuntu2004.docker b/tests/docker/dockerfiles/ubuntu2004.docker
index a3b38884e3..99803b343b 100644
--- a/tests/docker/dockerfiles/ubuntu2004.docker
+++ b/tests/docker/dockerfiles/ubuntu2004.docker
@@ -40,6 +40,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libcacard-dev \
libcap-ng-dev \
libcapstone-dev \
+ libcmocka-dev \
libcurl4-gnutls-dev \
libdaxctl-dev \
libdrm-dev \
@@ -58,6 +59,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
libiscsi-dev \
libjemalloc-dev \
libjpeg-turbo8-dev \
+ libjson-c-dev \
liblttng-ust-dev \
liblzo2-dev \
libncursesw5-dev \
diff --git a/tests/lcitool/libvirt-ci b/tests/lcitool/libvirt-ci
index f83b916d5e..324355cf62 160000
--- a/tests/lcitool/libvirt-ci
+++ b/tests/lcitool/libvirt-ci
@@ -1 +1 @@
-Subproject commit f83b916d5efa4bd33fbf4b7ea41bf6d535cc63fb
+Subproject commit 324355cf62e86fb551408575afb123bac989ac37
diff --git a/tests/lcitool/projects/qemu.yml b/tests/lcitool/projects/qemu.yml
index d068a7a8de..d9f826f8eb 100644
--- a/tests/lcitool/projects/qemu.yml
+++ b/tests/lcitool/projects/qemu.yml
@@ -9,6 +9,7 @@ packages:
- capstone
- ccache
- clang
+ - cmocka
- column
- ctags
- cyrus-sasl
@@ -26,15 +27,16 @@ packages:
- genisoimage
- glib2
- glib2-static
- - glibc-static
- glusterfs
- gnutls
- gtk3
- hostname
+ - json-c
- libaio
- libattr
- libasan
- libbpf
+ - libc-static
- libcacard
- libcap-ng
- libcurl
@@ -90,7 +92,7 @@ packages:
- python3-pip
- python3-sphinx
- python3-sphinx-rtd-theme
- - python3-virtualenv
+ - python3-venv
- rpm2cpio
- sdl2
- sdl2-image
diff --git a/tests/lcitool/refresh b/tests/lcitool/refresh
index 5e260f8cd6..2a59150322 100755
--- a/tests/lcitool/refresh
+++ b/tests/lcitool/refresh
@@ -25,7 +25,7 @@ self_dir = Path(__file__).parent
src_dir = self_dir.parent.parent
dockerfiles_dir = Path(src_dir, "tests", "docker", "dockerfiles")
-lcitool_path = Path(self_dir, "libvirt-ci", "lcitool")
+lcitool_path = Path(self_dir, "libvirt-ci", "bin", "lcitool")
lcitool_cmd = [lcitool_path, "--data-dir", self_dir]
@@ -112,7 +112,7 @@ try:
generate_dockerfile("debian-amd64", "debian-11",
trailer="".join(debian11_extras))
generate_dockerfile("fedora", "fedora-35")
- generate_dockerfile("opensuse-leap", "opensuse-leap-152")
+ generate_dockerfile("opensuse-leap", "opensuse-leap-153")
generate_dockerfile("ubuntu2004", "ubuntu-2004",
trailer="".join(ubuntu2004_tsanhack))
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 02/13] gitlab: show testlog.txt contents when cirrus/custom-runner jobs fail
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
2022-07-25 14:05 ` [PATCH v1 01/13] tests: refresh to latest libvirt-ci module Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 03/13] gitlab: drop 'containers-layer2' stage Alex Bennée
` (10 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Thomas Huth, Alex Bennée,
Wainer dos Santos Moschetta, Beraldo Leal
From: Daniel P. Berrangé <berrange@redhat.com>
When tests fail meson just displays a summary and tells you to look at
the testlog.txt file for details. The native jobs on shared runners
publish testlog.txt as an artifact. For the Cirrus jobs and custom
runner jobs this is not currently possible. The best we can do is cat
the log contents on failure, to give maintainers a fighting chance
of diagnosing the problem.
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20220722130431.2319019-3-berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.gitlab-ci.d/cirrus/build.yml | 3 ++-
.../custom-runners/centos-stream-8-x86_64.yml | 2 ++
.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch32.yml | 2 ++
.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml | 12 ++++++++++++
.gitlab-ci.d/custom-runners/ubuntu-20.04-s390x.yml | 12 ++++++++++++
5 files changed, 30 insertions(+), 1 deletion(-)
diff --git a/.gitlab-ci.d/cirrus/build.yml b/.gitlab-ci.d/cirrus/build.yml
index c555f5d36e..7ef6af8d33 100644
--- a/.gitlab-ci.d/cirrus/build.yml
+++ b/.gitlab-ci.d/cirrus/build.yml
@@ -32,5 +32,6 @@ build_task:
- $MAKE -j$(sysctl -n hw.ncpu)
- for TARGET in $TEST_TARGETS ;
do
- $MAKE -j$(sysctl -n hw.ncpu) $TARGET V=1 ;
+ $MAKE -j$(sysctl -n hw.ncpu) $TARGET V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
done
diff --git a/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml b/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
index 49aa703f55..068b0c4335 100644
--- a/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
+++ b/.gitlab-ci.d/custom-runners/centos-stream-8-x86_64.yml
@@ -23,6 +23,8 @@ centos-stream-8-x86_64:
- mkdir build
- cd build
- ../scripts/ci/org.centos/stream/8/x86_64/configure
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make -j"$JOBS"
- make NINJA=":" check
+ || { cat meson-logs/testlog.txt; exit 1; } ;
- ../scripts/ci/org.centos/stream/8/x86_64/test-avocado
diff --git a/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch32.yml b/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch32.yml
index 1998460d06..cbfa9cc164 100644
--- a/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch32.yml
+++ b/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch32.yml
@@ -19,5 +19,7 @@ ubuntu-20.04-aarch32-all:
- mkdir build
- cd build
- ../configure --cross-prefix=arm-linux-gnueabihf-
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc --ignore=40`
- make --output-sync -j`nproc --ignore=40` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
diff --git a/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml b/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml
index 65718a188a..3d878914e7 100644
--- a/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml
+++ b/.gitlab-ci.d/custom-runners/ubuntu-20.04-aarch64.yml
@@ -17,9 +17,12 @@ ubuntu-20.04-aarch64-all-linux-static:
- mkdir build
- cd build
- ../configure --enable-debug --static --disable-system --disable-glusterfs --disable-libssh
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc --ignore=40`
- make --output-sync -j`nproc --ignore=40` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
- make --output-sync -j`nproc --ignore=40` check-tcg V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-aarch64-all:
needs: []
@@ -38,8 +41,10 @@ ubuntu-20.04-aarch64-all:
- mkdir build
- cd build
- ../configure --disable-libssh
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc --ignore=40`
- make --output-sync -j`nproc --ignore=40` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-aarch64-alldbg:
needs: []
@@ -54,9 +59,11 @@ ubuntu-20.04-aarch64-alldbg:
- mkdir build
- cd build
- ../configure --enable-debug --disable-libssh
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make clean
- make --output-sync -j`nproc --ignore=40`
- make --output-sync -j`nproc --ignore=40` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-aarch64-clang:
needs: []
@@ -75,8 +82,10 @@ ubuntu-20.04-aarch64-clang:
- mkdir build
- cd build
- ../configure --disable-libssh --cc=clang-10 --cxx=clang++-10 --enable-sanitizers
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc --ignore=40`
- make --output-sync -j`nproc --ignore=40` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-aarch64-tci:
needs: []
@@ -95,6 +104,7 @@ ubuntu-20.04-aarch64-tci:
- mkdir build
- cd build
- ../configure --disable-libssh --enable-tcg-interpreter
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc --ignore=40`
ubuntu-20.04-aarch64-notcg:
@@ -114,5 +124,7 @@ ubuntu-20.04-aarch64-notcg:
- mkdir build
- cd build
- ../configure --disable-libssh --disable-tcg
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc --ignore=40`
- make --output-sync -j`nproc --ignore=40` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
diff --git a/.gitlab-ci.d/custom-runners/ubuntu-20.04-s390x.yml b/.gitlab-ci.d/custom-runners/ubuntu-20.04-s390x.yml
index 03e74c97db..0c835939db 100644
--- a/.gitlab-ci.d/custom-runners/ubuntu-20.04-s390x.yml
+++ b/.gitlab-ci.d/custom-runners/ubuntu-20.04-s390x.yml
@@ -17,9 +17,12 @@ ubuntu-20.04-s390x-all-linux-static:
- mkdir build
- cd build
- ../configure --enable-debug --static --disable-system --disable-glusterfs --disable-libssh
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc`
- make --output-sync -j`nproc` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
- make --output-sync -j`nproc` check-tcg V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-s390x-all:
needs: []
@@ -35,8 +38,10 @@ ubuntu-20.04-s390x-all:
- mkdir build
- cd build
- ../configure --disable-libssh
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc`
- make --output-sync -j`nproc` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-s390x-alldbg:
needs: []
@@ -55,9 +60,11 @@ ubuntu-20.04-s390x-alldbg:
- mkdir build
- cd build
- ../configure --enable-debug --disable-libssh
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make clean
- make --output-sync -j`nproc`
- make --output-sync -j`nproc` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-s390x-clang:
needs: []
@@ -76,8 +83,10 @@ ubuntu-20.04-s390x-clang:
- mkdir build
- cd build
- ../configure --disable-libssh --cc=clang --cxx=clang++ --enable-sanitizers
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc`
- make --output-sync -j`nproc` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
ubuntu-20.04-s390x-tci:
needs: []
@@ -96,6 +105,7 @@ ubuntu-20.04-s390x-tci:
- mkdir build
- cd build
- ../configure --disable-libssh --enable-tcg-interpreter
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc`
ubuntu-20.04-s390x-notcg:
@@ -115,5 +125,7 @@ ubuntu-20.04-s390x-notcg:
- mkdir build
- cd build
- ../configure --disable-libssh --disable-tcg
+ || { cat config.log meson-logs/meson-log.txt; exit 1; }
- make --output-sync -j`nproc`
- make --output-sync -j`nproc` check V=1
+ || { cat meson-logs/testlog.txt; exit 1; } ;
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 03/13] gitlab: drop 'containers-layer2' stage
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
2022-07-25 14:05 ` [PATCH v1 01/13] tests: refresh to latest libvirt-ci module Alex Bennée
2022-07-25 14:05 ` [PATCH v1 02/13] gitlab: show testlog.txt contents when cirrus/custom-runner jobs fail Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-26 14:30 ` Philippe Mathieu-Daudé via
2022-07-25 14:05 ` [PATCH v1 04/13] .cirrus.yml: Change winsymlinks to 'native' Alex Bennée
` (9 subsequent siblings)
12 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Thomas Huth, Alex Bennée,
Wainer dos Santos Moschetta, Beraldo Leal
From: Daniel P. Berrangé <berrange@redhat.com>
Since we express dependancies via a 'needs' clause, we don't need to
split container builds into separate stages. GitLab happily lets jobs
depend on other jobs in the same stage and will run them when possible.
Acked-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20220722130431.2319019-4-berrange@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.gitlab-ci.d/container-cross.yml | 24 ++++++++++++------------
.gitlab-ci.d/stages.yml | 1 -
2 files changed, 12 insertions(+), 13 deletions(-)
diff --git a/.gitlab-ci.d/container-cross.yml b/.gitlab-ci.d/container-cross.yml
index b7963498a3..505b267542 100644
--- a/.gitlab-ci.d/container-cross.yml
+++ b/.gitlab-ci.d/container-cross.yml
@@ -1,20 +1,20 @@
alpha-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-alpha-cross
amd64-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-amd64-cross
amd64-debian-user-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-all-test-cross
@@ -65,21 +65,21 @@ hexagon-cross-container:
hppa-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-hppa-cross
m68k-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-m68k-cross
mips64-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-mips64-cross
@@ -92,7 +92,7 @@ mips64el-debian-cross-container:
mips-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-mips-cross
@@ -105,7 +105,7 @@ mipsel-debian-cross-container:
powerpc-test-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian11-container']
variables:
NAME: debian-powerpc-test-cross
@@ -127,7 +127,7 @@ riscv64-debian-cross-container:
# we can however build TCG tests using a non-sid base
riscv64-debian-test-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian11-container']
variables:
NAME: debian-riscv64-test-cross
@@ -140,21 +140,21 @@ s390x-debian-cross-container:
sh4-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-sh4-cross
sparc64-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-sparc64-cross
tricore-debian-cross-container:
extends: .container_job_template
- stage: containers-layer2
+ stage: containers
needs: ['amd64-debian10-container']
variables:
NAME: debian-tricore-cross
diff --git a/.gitlab-ci.d/stages.yml b/.gitlab-ci.d/stages.yml
index f50826018d..f92f57a27d 100644
--- a/.gitlab-ci.d/stages.yml
+++ b/.gitlab-ci.d/stages.yml
@@ -3,6 +3,5 @@
# - test (for test stages, using build artefacts from a build stage)
stages:
- containers
- - containers-layer2
- build
- test
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 04/13] .cirrus.yml: Change winsymlinks to 'native'
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (2 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 03/13] gitlab: drop 'containers-layer2' stage Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 05/13] .gitlab-ci.d/windows.yml: Enable native Windows symlink Alex Bennée
` (8 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Bin Meng, Thomas Huth, Yonggang Luo,
Alex Bennée
From: Bin Meng <bin.meng@windriver.com>
At present winsymlinks is set to 'nativestrict', and its behavior is:
a) if native symlinks are enabled and <target> exists, creates
<destination> as a native Windows symlink;
b) else if native symlinks are not enabled or if <target> does
not exist, 'ln -s' fails.
This causes the following error message was seen during the configure:
"ln: failed to create symbolic link
'x86_64-softmmu/qemu-system-x86_64.exe': No such file or directory"
Change winsymlinks to 'native' whose behavior is most similar to the
behavior of 'ln -s' on *nix, that is:
a) if native symlinks are enabled, and whether <target> exists
or not, creates <destination> as a native Windows symlink;
b) else if native symlinks are not enabled, and whether <target>
exists or not, 'ln -s' creates as a Windows shortcut file.
Signed-off-by: Bin Meng <bin.meng@windriver.com>
Acked-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Yonggang Luo <luoyonggang@gmail.com>
Message-Id: <20220719161230.766063-1-bmeng.cn@gmail.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.cirrus.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.cirrus.yml b/.cirrus.yml
index 20843a420c..eac39024f2 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -10,7 +10,7 @@ windows_msys2_task:
memory: 8G
env:
CIRRUS_SHELL: powershell
- MSYS: winsymlinks:nativestrict
+ MSYS: winsymlinks:native
MSYSTEM: MINGW64
MSYS2_URL: https://github.com/msys2/msys2-installer/releases/download/2022-05-03/msys2-base-x86_64-20220503.sfx.exe
MSYS2_FINGERPRINT: 0
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 05/13] .gitlab-ci.d/windows.yml: Enable native Windows symlink
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (3 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 04/13] .cirrus.yml: Change winsymlinks to 'native' Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 06/13] semihosting: Don't return negative values on qemu_semihosting_console_write() failure Alex Bennée
` (7 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Bin Meng, Alex Bennée, Thomas Huth,
Wainer dos Santos Moschetta, Beraldo Leal
From: Bin Meng <bin.meng@windriver.com>
The following error message was seen during the configure:
"ln: failed to create symbolic link
'x86_64-softmmu/qemu-system-x86_64.exe': No such file or directory"
By default the MSYS environment variable is not defined, so the runtime
behavior of winsymlinks is: if <target> does not exist, 'ln -s' fails.
At the configure phase, the qemu-system-x86_64.exe has not been built
so creation of the symbolic link fails hence the error message.
Set winsymlinks to 'native' whose behavior is most similar to the
behavior of 'ln -s' on *nix, that is:
a) if native symlinks are enabled, and whether <target> exists
or not, creates <destination> as a native Windows symlink;
b) else if native symlinks are not enabled, and whether <target>
exists or not, 'ln -s' creates as a Windows shortcut file.
Signed-off-by: Bin Meng <bin.meng@windriver.com>
Message-Id: <20220725123000.807608-1-bmeng.cn@gmail.com>
---
.gitlab-ci.d/windows.yml | 2 ++
1 file changed, 2 insertions(+)
diff --git a/.gitlab-ci.d/windows.yml b/.gitlab-ci.d/windows.yml
index 1b2ede49e1..0b9572a8a3 100644
--- a/.gitlab-ci.d/windows.yml
+++ b/.gitlab-ci.d/windows.yml
@@ -57,6 +57,7 @@ msys2-64bit:
mingw-w64-x86_64-zstd "
- $env:CHERE_INVOKING = 'yes' # Preserve the current working directory
- $env:MSYSTEM = 'MINGW64' # Start a 64 bit Mingw environment
+ - $env:MSYS = 'winsymlinks:native' # Enable native Windows symlink
- .\msys64\usr\bin\bash -lc './configure --target-list=x86_64-softmmu
--enable-capstone --without-default-devices'
- .\msys64\usr\bin\bash -lc "sed -i '/^ROMS=/d' build/config-host.mak"
@@ -89,6 +90,7 @@ msys2-32bit:
mingw-w64-i686-usbredir "
- $env:CHERE_INVOKING = 'yes' # Preserve the current working directory
- $env:MSYSTEM = 'MINGW32' # Start a 32-bit MinG environment
+ - $env:MSYS = 'winsymlinks:native' # Enable native Windows symlink
- mkdir output
- cd output
- ..\msys64\usr\bin\bash -lc "../configure --target-list=ppc64-softmmu"
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 06/13] semihosting: Don't return negative values on qemu_semihosting_console_write() failure
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (4 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 05/13] .gitlab-ci.d/windows.yml: Enable native Windows symlink Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-26 14:31 ` Philippe Mathieu-Daudé via
2022-07-25 14:05 ` [PATCH v1 07/13] semihosting: Don't copy buffer after console_write() Alex Bennée
` (6 subsequent siblings)
12 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Richard Henderson, Alex Bennée
From: Peter Maydell <peter.maydell@linaro.org>
The documentation comment for qemu_semihosting_console_write() says
* Returns: number of bytes written -- this should only ever be short
* on some sort of i/o error.
and the callsites rely on this. However, the implementation code
path which sends console output to a chardev doesn't honour this,
and will return negative values on error. Bring it into line with
the other implementation codepaths and the documentation, so that
it returns 0 on error.
Spotted by Coverity, because console_write() passes the return value
to unlock_user(), which doesn't accept a negative length.
Resolves: Coverity CID 1490288
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220719121110.225657-2-peter.maydell@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
semihosting/console.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/semihosting/console.c b/semihosting/console.c
index 5b1ec0a1c3..0f976fe8cb 100644
--- a/semihosting/console.c
+++ b/semihosting/console.c
@@ -111,7 +111,8 @@ int qemu_semihosting_console_read(CPUState *cs, void *buf, int len)
int qemu_semihosting_console_write(void *buf, int len)
{
if (console.chr) {
- return qemu_chr_write_all(console.chr, (uint8_t *)buf, len);
+ int r = qemu_chr_write_all(console.chr, (uint8_t *)buf, len);
+ return r < 0 ? 0 : r;
} else {
return fwrite(buf, 1, len, stderr);
}
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 07/13] semihosting: Don't copy buffer after console_write()
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (5 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 06/13] semihosting: Don't return negative values on qemu_semihosting_console_write() failure Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 08/13] semihosting: Check for errors on SET_ARG() Alex Bennée
` (5 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Richard Henderson, Alex Bennée
From: Peter Maydell <peter.maydell@linaro.org>
The console_write() semihosting function outputs guest data from a
buffer; it doesn't update that buffer. It therefore doesn't need to
pass a length value to unlock_user(), but can pass 0, meaning "do not
copy any data back to the guest memory".
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220719121110.225657-3-peter.maydell@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
semihosting/syscalls.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/semihosting/syscalls.c b/semihosting/syscalls.c
index 4847f66c02..508a0ad88c 100644
--- a/semihosting/syscalls.c
+++ b/semihosting/syscalls.c
@@ -627,7 +627,7 @@ static void console_write(CPUState *cs, gdb_syscall_complete_cb complete,
}
ret = qemu_semihosting_console_write(ptr, len);
complete(cs, ret ? ret : -1, ret ? 0 : EIO);
- unlock_user(ptr, buf, ret);
+ unlock_user(ptr, buf, 0);
}
static void console_fstat(CPUState *cs, gdb_syscall_complete_cb complete,
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 08/13] semihosting: Check for errors on SET_ARG()
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (6 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 07/13] semihosting: Don't copy buffer after console_write() Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 09/13] semihosting: Fix handling of buffer in TARGET_SYS_TMPNAM Alex Bennée
` (4 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Richard Henderson, Alex Bennée
From: Peter Maydell <peter.maydell@linaro.org>
The SET_ARG() macro returns an error indication; we check this in the
TARGET_SYS_GET_CMDLINE case but not when we use it in implementing
TARGET_SYS_ELAPSED. Check for and handle the errors via the do_fault
codepath, and update the comment documenting the SET_ARG() and
GET_ARG() macros to note how they handle memory access errors.
Resolves: Coverity CID 1490287
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220719121110.225657-4-peter.maydell@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
semihosting/arm-compat-semi.c | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/semihosting/arm-compat-semi.c b/semihosting/arm-compat-semi.c
index 1a1e2a6960..d12288fc80 100644
--- a/semihosting/arm-compat-semi.c
+++ b/semihosting/arm-compat-semi.c
@@ -171,6 +171,12 @@ static LayoutInfo common_semi_find_bases(CPUState *cs)
* Read the input value from the argument block; fail the semihosting
* call if the memory read fails. Eventually we could use a generic
* CPUState helper function here.
+ * Note that GET_ARG() handles memory access errors by jumping to
+ * do_fault, so must be used as the first thing done in handling a
+ * semihosting call, to avoid accidentally leaking allocated resources.
+ * SET_ARG(), since it unavoidably happens late, instead returns an
+ * error indication (0 on success, non-0 for error) which the caller
+ * should check.
*/
#define GET_ARG(n) do { \
@@ -739,10 +745,14 @@ void do_common_semihosting(CPUState *cs)
case TARGET_SYS_ELAPSED:
elapsed = get_clock() - clock_start;
if (sizeof(target_ulong) == 8) {
- SET_ARG(0, elapsed);
+ if (SET_ARG(0, elapsed)) {
+ goto do_fault;
+ }
} else {
- SET_ARG(0, (uint32_t) elapsed);
- SET_ARG(1, (uint32_t) (elapsed >> 32));
+ if (SET_ARG(0, (uint32_t) elapsed) ||
+ SET_ARG(1, (uint32_t) (elapsed >> 32))) {
+ goto do_fault;
+ }
}
common_semi_set_ret(cs, 0);
break;
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 09/13] semihosting: Fix handling of buffer in TARGET_SYS_TMPNAM
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (7 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 08/13] semihosting: Check for errors on SET_ARG() Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 10/13] qapi: Add exit-failure PanicAction Alex Bennée
` (3 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Richard Henderson, Alex Bennée
From: Peter Maydell <peter.maydell@linaro.org>
The TARGET_SYS_TMPNAM implementation has two bugs spotted by
Coverity:
* confusion about whether 'len' has the length of the string
including or excluding the terminating NUL means we
lock_user() len bytes of memory but memcpy() len + 1 bytes
* In the error-exit cases we forget to free() the buffer
that asprintf() returned to us
Resolves: Coverity CID 1490285, 1490289
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220719121110.225657-5-peter.maydell@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
semihosting/arm-compat-semi.c | 13 +++++++++++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/semihosting/arm-compat-semi.c b/semihosting/arm-compat-semi.c
index d12288fc80..e741674238 100644
--- a/semihosting/arm-compat-semi.c
+++ b/semihosting/arm-compat-semi.c
@@ -504,16 +504,25 @@ void do_common_semihosting(CPUState *cs)
GET_ARG(1);
GET_ARG(2);
len = asprintf(&s, "/tmp/qemu-%x%02x", getpid(), (int)arg1 & 0xff);
+ if (len < 0) {
+ common_semi_set_ret(cs, -1);
+ break;
+ }
+
+ /* Allow for trailing NUL */
+ len++;
/* Make sure there's enough space in the buffer */
- if (len < 0 || len >= arg2) {
+ if (len > arg2) {
+ free(s);
common_semi_set_ret(cs, -1);
break;
}
p = lock_user(VERIFY_WRITE, arg0, len, 0);
if (!p) {
+ free(s);
goto do_fault;
}
- memcpy(p, s, len + 1);
+ memcpy(p, s, len);
unlock_user(p, arg0, len);
free(s);
common_semi_set_ret(cs, 0);
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 10/13] qapi: Add exit-failure PanicAction
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (8 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 09/13] semihosting: Fix handling of buffer in TARGET_SYS_TMPNAM Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 11/13] tests/tcg/s390x: Test unaligned accesses to lowcore Alex Bennée
` (2 subsequent siblings)
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Ilya Leoshkevich, David Hildenbrand,
Richard Henderson, Alex Bennée, Eric Blake,
Markus Armbruster
From: Ilya Leoshkevich <iii@linux.ibm.com>
Currently QEMU exits with code 0 on both panic an shutdown. For tests
it is useful to return 1 on panic, so that it counts as a test
failure.
Introduce a new exit-failure PanicAction that makes main() return
EXIT_FAILURE. Tests can use -action panic=exit-failure option to
activate this behavior.
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220722233614.7254-2-iii@linux.ibm.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
qapi/run-state.json | 4 +++-
include/sysemu/sysemu.h | 2 +-
softmmu/main.c | 6 ++++--
softmmu/runstate.c | 17 +++++++++++++----
qemu-options.hx | 2 +-
5 files changed, 22 insertions(+), 9 deletions(-)
diff --git a/qapi/run-state.json b/qapi/run-state.json
index 6e2162d7b3..d42c370c4f 100644
--- a/qapi/run-state.json
+++ b/qapi/run-state.json
@@ -364,10 +364,12 @@
#
# @shutdown: Shutdown the VM and exit, according to the shutdown action
#
+# @exit-failure: Shutdown the VM and exit with nonzero status
+#
# Since: 6.0
##
{ 'enum': 'PanicAction',
- 'data': [ 'pause', 'shutdown', 'none' ] }
+ 'data': [ 'pause', 'shutdown', 'exit-failure', 'none' ] }
##
# @watchdog-set-action:
diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
index 812f66a31a..31aa45160b 100644
--- a/include/sysemu/sysemu.h
+++ b/include/sysemu/sysemu.h
@@ -103,7 +103,7 @@ void qemu_boot_set(const char *boot_order, Error **errp);
bool defaults_enabled(void);
void qemu_init(int argc, char **argv, char **envp);
-void qemu_main_loop(void);
+int qemu_main_loop(void);
void qemu_cleanup(void);
extern QemuOptsList qemu_legacy_drive_opts;
diff --git a/softmmu/main.c b/softmmu/main.c
index c00432ff09..1b675a8c03 100644
--- a/softmmu/main.c
+++ b/softmmu/main.c
@@ -32,11 +32,13 @@
int qemu_main(int argc, char **argv, char **envp)
{
+ int status;
+
qemu_init(argc, argv, envp);
- qemu_main_loop();
+ status = qemu_main_loop();
qemu_cleanup();
- return 0;
+ return status;
}
#ifndef CONFIG_COCOA
diff --git a/softmmu/runstate.c b/softmmu/runstate.c
index 168e1b78a0..1e68680b9d 100644
--- a/softmmu/runstate.c
+++ b/softmmu/runstate.c
@@ -482,7 +482,8 @@ void qemu_system_guest_panicked(GuestPanicInformation *info)
qapi_event_send_guest_panicked(GUEST_PANIC_ACTION_PAUSE,
!!info, info);
vm_stop(RUN_STATE_GUEST_PANICKED);
- } else if (panic_action == PANIC_ACTION_SHUTDOWN) {
+ } else if (panic_action == PANIC_ACTION_SHUTDOWN ||
+ panic_action == PANIC_ACTION_EXIT_FAILURE) {
qapi_event_send_guest_panicked(GUEST_PANIC_ACTION_POWEROFF,
!!info, info);
vm_stop(RUN_STATE_GUEST_PANICKED);
@@ -662,7 +663,7 @@ void qemu_system_debug_request(void)
qemu_notify_event();
}
-static bool main_loop_should_exit(void)
+static bool main_loop_should_exit(int *status)
{
RunState r;
ShutdownCause request;
@@ -680,6 +681,10 @@ static bool main_loop_should_exit(void)
if (shutdown_action == SHUTDOWN_ACTION_PAUSE) {
vm_stop(RUN_STATE_SHUTDOWN);
} else {
+ if (request == SHUTDOWN_CAUSE_GUEST_PANIC &&
+ panic_action == PANIC_ACTION_EXIT_FAILURE) {
+ *status = EXIT_FAILURE;
+ }
return true;
}
}
@@ -715,12 +720,14 @@ static bool main_loop_should_exit(void)
return false;
}
-void qemu_main_loop(void)
+int qemu_main_loop(void)
{
+ int status = EXIT_SUCCESS;
#ifdef CONFIG_PROFILER
int64_t ti;
#endif
- while (!main_loop_should_exit()) {
+
+ while (!main_loop_should_exit(&status)) {
#ifdef CONFIG_PROFILER
ti = profile_getclock();
#endif
@@ -729,6 +736,8 @@ void qemu_main_loop(void)
dev_time += profile_getclock() - ti;
#endif
}
+
+ return status;
}
void qemu_add_exit_notifier(Notifier *notify)
diff --git a/qemu-options.hx b/qemu-options.hx
index 79e00916a1..8e17c5064a 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -4239,7 +4239,7 @@ DEF("action", HAS_ARG, QEMU_OPTION_action,
" action when guest reboots [default=reset]\n"
"-action shutdown=poweroff|pause\n"
" action when guest shuts down [default=poweroff]\n"
- "-action panic=pause|shutdown|none\n"
+ "-action panic=pause|shutdown|exit-failure|none\n"
" action when guest panics [default=shutdown]\n"
"-action watchdog=reset|shutdown|poweroff|inject-nmi|pause|debug|none\n"
" action when watchdog fires [default=reset]\n",
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 11/13] tests/tcg/s390x: Test unaligned accesses to lowcore
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (9 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 10/13] qapi: Add exit-failure PanicAction Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 15:17 ` Thomas Huth
2022-07-25 14:05 ` [PATCH v1 12/13] docs/devel: fix description of OBJECT_DECLARE_SIMPLE_TYPE Alex Bennée
2022-07-25 14:05 ` [PATCH v1 13/13] qemu-options: bring the kernel and image options together Alex Bennée
12 siblings, 1 reply; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Ilya Leoshkevich, Richard Henderson,
Alex Bennée, David Hildenbrand, Cornelia Huck, Thomas Huth,
open list:S390 TCG CPUs
From: Ilya Leoshkevich <iii@linux.ibm.com>
Add a small test to avoid regressions.
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Acked-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20220722233614.7254-3-iii@linux.ibm.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
tests/tcg/s390x/Makefile.softmmu-target | 9 +++++++++
tests/tcg/s390x/unaligned-lowcore.S | 19 +++++++++++++++++++
2 files changed, 28 insertions(+)
create mode 100644 tests/tcg/s390x/Makefile.softmmu-target
create mode 100644 tests/tcg/s390x/unaligned-lowcore.S
diff --git a/tests/tcg/s390x/Makefile.softmmu-target b/tests/tcg/s390x/Makefile.softmmu-target
new file mode 100644
index 0000000000..a34fa68473
--- /dev/null
+++ b/tests/tcg/s390x/Makefile.softmmu-target
@@ -0,0 +1,9 @@
+S390X_SRC=$(SRC_PATH)/tests/tcg/s390x
+VPATH+=$(S390X_SRC)
+QEMU_OPTS=-action panic=exit-failure -kernel
+
+%: %.S
+ $(CC) -march=z13 -m64 -nostartfiles -static -Wl,-Ttext=0 \
+ -Wl,--build-id=none $< -o $@
+
+TESTS += unaligned-lowcore
diff --git a/tests/tcg/s390x/unaligned-lowcore.S b/tests/tcg/s390x/unaligned-lowcore.S
new file mode 100644
index 0000000000..246b517f11
--- /dev/null
+++ b/tests/tcg/s390x/unaligned-lowcore.S
@@ -0,0 +1,19 @@
+ .org 0x1D0 /* program new PSW */
+ .quad 0x2000000000000, 0 /* disabled wait */
+ .org 0x200 /* lowcore padding */
+
+ .globl _start
+_start:
+ lctlg %c0,%c0,_c0
+ vst %v0,_unaligned
+ lpswe quiesce_psw
+
+ .align 8
+quiesce_psw:
+ .quad 0x2000000000000,0xfff /* see is_special_wait_psw() */
+_c0:
+ .quad 0x10060000 /* lowcore protection, AFP, VX */
+
+ .byte 0
+_unaligned:
+ .octa 0
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 12/13] docs/devel: fix description of OBJECT_DECLARE_SIMPLE_TYPE
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (10 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 11/13] tests/tcg/s390x: Test unaligned accesses to lowcore Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
2022-07-25 14:05 ` [PATCH v1 13/13] qemu-options: bring the kernel and image options together Alex Bennée
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Alex Bennée, Richard Henderson
Since 30b5707c26 (qom: Remove module_obj_name parameter from
OBJECT_DECLARE* macros) we don't need the additional two parameters.
Fix the documentation.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20220712103131.2006653-1-alex.bennee@linaro.org>
---
docs/devel/qom.rst | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/docs/devel/qom.rst b/docs/devel/qom.rst
index e5fe3597cd..0cf9a714f0 100644
--- a/docs/devel/qom.rst
+++ b/docs/devel/qom.rst
@@ -292,8 +292,7 @@ in the header file:
.. code-block:: c
:caption: Declaring a simple type
- OBJECT_DECLARE_SIMPLE_TYPE(MyDevice, my_device,
- MY_DEVICE, DEVICE)
+ OBJECT_DECLARE_SIMPLE_TYPE(MyDevice, MY_DEVICE)
This is equivalent to the following:
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH v1 13/13] qemu-options: bring the kernel and image options together
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
` (11 preceding siblings ...)
2022-07-25 14:05 ` [PATCH v1 12/13] docs/devel: fix description of OBJECT_DECLARE_SIMPLE_TYPE Alex Bennée
@ 2022-07-25 14:05 ` Alex Bennée
12 siblings, 0 replies; 19+ messages in thread
From: Alex Bennée @ 2022-07-25 14:05 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Alex Bennée, Cédric Le Goater
How to control the booting of QEMU is often a source of confusion for
users. Bring the options that control this together in the manual
pages and add some verbiage to describe when each option is
appropriate. This attempts to codify some of the knowledge expressed
in:
https://stackoverflow.com/questions/58420670/qemu-bios-vs-kernel-vs-device-loader-file/58434837#58434837
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20220707151037.397324-1-alex.bennee@linaro.org>
---
qemu-options.hx | 96 +++++++++++++++++++++++++++++++++++++++----------
1 file changed, 78 insertions(+), 18 deletions(-)
diff --git a/qemu-options.hx b/qemu-options.hx
index 8e17c5064a..3f23a42fa8 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -1585,13 +1585,6 @@ SRST
Use file as SecureDigital card image.
ERST
-DEF("pflash", HAS_ARG, QEMU_OPTION_pflash,
- "-pflash file use 'file' as a parallel flash image\n", QEMU_ARCH_ALL)
-SRST
-``-pflash file``
- Use file as a parallel flash image.
-ERST
-
DEF("snapshot", 0, QEMU_OPTION_snapshot,
"-snapshot write to temporary files instead of disk image files\n",
QEMU_ARCH_ALL)
@@ -3684,12 +3677,67 @@ DEFHEADING()
#endif
-DEFHEADING(Linux/Multiboot boot specific:)
+DEFHEADING(Boot Image or Kernel specific:)
+SRST
+There are broadly 4 ways you can boot a system with QEMU.
+
+ - specify a firmware and let it control finding a kernel
+ - specify a firmware and pass a hint to the kernel to boot
+ - direct kernel image boot
+ - manually load files into the guest's address space
+
+The third method is useful for quickly testing kernels but as there is
+no firmware to pass configuration information to the kernel the
+hardware must either be probeable, the kernel built for the exact
+configuration or passed some configuration data (e.g. a DTB blob)
+which tells the kernel what drivers it needs. This exact details are
+often hardware specific.
+
+The final method is the most generic way of loading images into the
+guest address space and used mostly for ``bare metal`` type
+development where the reset vectors of the processor are taken into
+account.
+
+ERST
+
SRST
-When using these options, you can use a given Linux or Multiboot kernel
-without installing it in the disk image. It can be useful for easier
-testing of various kernels.
+For x86 machines and some other architectures ``-bios`` will generally
+do the right thing with whatever it is given. For other machines the
+more strict ``-pflash`` option needs an image that is sized for the
+flash device for the given machine type.
+
+Please see the :ref:`system-targets-ref` section of the manual for
+more detailed documentation.
+
+ERST
+
+DEF("bios", HAS_ARG, QEMU_OPTION_bios, \
+ "-bios file set the filename for the BIOS\n", QEMU_ARCH_ALL)
+SRST
+``-bios file``
+ Set the filename for the BIOS.
+ERST
+
+DEF("pflash", HAS_ARG, QEMU_OPTION_pflash,
+ "-pflash file use 'file' as a parallel flash image\n", QEMU_ARCH_ALL)
+SRST
+``-pflash file``
+ Use file as a parallel flash image.
+ERST
+
+SRST
+
+The kernel options were designed to work with Linux kernels although
+other things (like hypervisors) can be packaged up as a kernel
+executable image. The exact format of a executable image is usually
+architecture specific.
+
+The way in which the kernel is started (what address it is loaded at,
+what if any information is passed to it via CPU registers, the state
+of the hardware when it is started, and so on) is also architecture
+specific. Typically it follows the specification laid down by the
+Linux kernel for how kernels for that architecture must be started.
ERST
@@ -3729,6 +3777,25 @@ SRST
kernel on boot.
ERST
+SRST
+
+Finally you can also manually load images directly into the address
+space of the guest. This is most useful for developers who already
+know the layout of their guest and take care to ensure something sane
+will happen when the reset vector executes.
+
+The generic loader can be invoked by using the loader device:
+
+``-device loader,addr=<addr>,data=<data>,data-len=<data-len>[,data-be=<data-be>][,cpu-num=<cpu-num>]``
+
+there is also the guest loader which operates in a similar way but
+tweaks the DTB so a hypervisor loaded via ``-kernel`` can find where
+the guest image is:
+
+``-device guest-loader,addr=<addr>[,kernel=<path>,[bootargs=<arguments>]][,initrd=<path>]``
+
+ERST
+
DEFHEADING()
DEFHEADING(Debug/Expert options:)
@@ -4179,13 +4246,6 @@ SRST
To list all the data directories, use ``-L help``.
ERST
-DEF("bios", HAS_ARG, QEMU_OPTION_bios, \
- "-bios file set the filename for the BIOS\n", QEMU_ARCH_ALL)
-SRST
-``-bios file``
- Set the filename for the BIOS.
-ERST
-
DEF("enable-kvm", 0, QEMU_OPTION_enable_kvm, \
"-enable-kvm enable KVM full virtualization support\n",
QEMU_ARCH_ARM | QEMU_ARCH_I386 | QEMU_ARCH_MIPS | QEMU_ARCH_PPC |
--
2.30.2
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH v1 11/13] tests/tcg/s390x: Test unaligned accesses to lowcore
2022-07-25 14:05 ` [PATCH v1 11/13] tests/tcg/s390x: Test unaligned accesses to lowcore Alex Bennée
@ 2022-07-25 15:17 ` Thomas Huth
0 siblings, 0 replies; 19+ messages in thread
From: Thomas Huth @ 2022-07-25 15:17 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, f4bug, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Ilya Leoshkevich, Richard Henderson,
David Hildenbrand, Cornelia Huck, open list:S390 TCG CPUs
On 25/07/2022 16.05, Alex Bennée wrote:
> From: Ilya Leoshkevich <iii@linux.ibm.com>
>
> Add a small test to avoid regressions.
>
> Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
> Acked-by: Richard Henderson <richard.henderson@linaro.org>
> Message-Id: <20220722233614.7254-3-iii@linux.ibm.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> tests/tcg/s390x/Makefile.softmmu-target | 9 +++++++++
> tests/tcg/s390x/unaligned-lowcore.S | 19 +++++++++++++++++++
> 2 files changed, 28 insertions(+)
> create mode 100644 tests/tcg/s390x/Makefile.softmmu-target
> create mode 100644 tests/tcg/s390x/unaligned-lowcore.S
>
> diff --git a/tests/tcg/s390x/Makefile.softmmu-target b/tests/tcg/s390x/Makefile.softmmu-target
> new file mode 100644
> index 0000000000..a34fa68473
> --- /dev/null
> +++ b/tests/tcg/s390x/Makefile.softmmu-target
> @@ -0,0 +1,9 @@
> +S390X_SRC=$(SRC_PATH)/tests/tcg/s390x
> +VPATH+=$(S390X_SRC)
> +QEMU_OPTS=-action panic=exit-failure -kernel
> +
> +%: %.S
> + $(CC) -march=z13 -m64 -nostartfiles -static -Wl,-Ttext=0 \
> + -Wl,--build-id=none $< -o $@
> +
> +TESTS += unaligned-lowcore
> diff --git a/tests/tcg/s390x/unaligned-lowcore.S b/tests/tcg/s390x/unaligned-lowcore.S
> new file mode 100644
> index 0000000000..246b517f11
> --- /dev/null
> +++ b/tests/tcg/s390x/unaligned-lowcore.S
> @@ -0,0 +1,19 @@
> + .org 0x1D0 /* program new PSW */
> + .quad 0x2000000000000, 0 /* disabled wait */
> + .org 0x200 /* lowcore padding */
> +
> + .globl _start
> +_start:
> + lctlg %c0,%c0,_c0
> + vst %v0,_unaligned
> + lpswe quiesce_psw
> +
> + .align 8
> +quiesce_psw:
> + .quad 0x2000000000000,0xfff /* see is_special_wait_psw() */
> +_c0:
> + .quad 0x10060000 /* lowcore protection, AFP, VX */
> +
> + .byte 0
> +_unaligned:
> + .octa 0
Acked-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v1 01/13] tests: refresh to latest libvirt-ci module
2022-07-25 14:05 ` [PATCH v1 01/13] tests: refresh to latest libvirt-ci module Alex Bennée
@ 2022-07-25 18:56 ` Richard Henderson
2022-07-26 14:34 ` Philippe Mathieu-Daudé via
1 sibling, 0 replies; 19+ messages in thread
From: Richard Henderson @ 2022-07-25 18:56 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: Daniel P. Berrange
On 7/25/22 07:05, Alex Bennée wrote:
> From: Daniel P. Berrangé <berrange@redhat.com>
>
> Notable changes:
>
> - libvirt-ci source tree was re-arranged, so the script we
> run now lives in a bin/ sub-dir
It would have been nice to keep the submodule update plus the bin/ sub-dir change in one
patch, and the generated file updates in a separate patch. But I won't insist.
Acked-by: Richard Henderson <richard.henderson@linaro.org>
r~
>
> - opensuse 15.2 is replaced by opensuse 15.3
>
> - libslirp is temporarily dropped on opensuse as the
> libslirp-version.h is broken
>
> https://bugzilla.opensuse.org/show_bug.cgi?id=1201551
>
> - The incorrectly named python3-virtualenv module was
> changed to python3-venv, but most distros don't need
> any package as 'venv' is a standard part of python
>
> - glibc-static was renamed to libc-static, to reflect
> fact that it isn't going to be glibc on all distros
>
> - The cmocka/json-c deps that were manually added to
> the centos dockerfile and are now consistently added
> to all targets
>
> Acked-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> Message-Id: <20220722130431.2319019-2-berrange@redhat.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .gitlab-ci.d/cirrus/freebsd-12.vars | 3 +--
> .gitlab-ci.d/cirrus/freebsd-13.vars | 3 +--
> .gitlab-ci.d/cirrus/macos-11.vars | 4 ++--
> tests/docker/dockerfiles/alpine.docker | 4 +++-
> tests/docker/dockerfiles/centos8.docker | 6 +++---
> tests/docker/dockerfiles/debian-amd64.docker | 2 ++
> tests/docker/dockerfiles/debian-arm64-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-armel-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-armhf-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-mips64el-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-mipsel-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-ppc64el-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-s390x-cross.docker | 2 ++
> tests/docker/dockerfiles/fedora.docker | 3 ++-
> tests/docker/dockerfiles/opensuse-leap.docker | 7 ++++---
> tests/docker/dockerfiles/ubuntu2004.docker | 2 ++
> tests/lcitool/libvirt-ci | 2 +-
> tests/lcitool/projects/qemu.yml | 6 ++++--
> tests/lcitool/refresh | 4 ++--
> 19 files changed, 41 insertions(+), 19 deletions(-)
>
> diff --git a/.gitlab-ci.d/cirrus/freebsd-12.vars b/.gitlab-ci.d/cirrus/freebsd-12.vars
> index f59263731f..8fa5a320e9 100644
> --- a/.gitlab-ci.d/cirrus/freebsd-12.vars
> +++ b/.gitlab-ci.d/cirrus/freebsd-12.vars
> @@ -1,5 +1,4 @@
> # THIS FILE WAS AUTO-GENERATED
> -# ... and then edited to fix py39, pending proper lcitool update.
> #
> # $ lcitool variables freebsd-12 qemu
> #
> @@ -12,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
> NINJA='/usr/local/bin/ninja'
> PACKAGING_COMMAND='pkg'
> PIP3='/usr/local/bin/pip-3.8'
> -PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-virtualenv py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
> +PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage cmocka ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 json-c libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
> PYPI_PKGS=''
> PYTHON='/usr/local/bin/python3'
> diff --git a/.gitlab-ci.d/cirrus/freebsd-13.vars b/.gitlab-ci.d/cirrus/freebsd-13.vars
> index 40fc961398..8ed7e33a77 100644
> --- a/.gitlab-ci.d/cirrus/freebsd-13.vars
> +++ b/.gitlab-ci.d/cirrus/freebsd-13.vars
> @@ -1,5 +1,4 @@
> # THIS FILE WAS AUTO-GENERATED
> -# ... and then edited to fix py39, pending proper lcitool update.
> #
> # $ lcitool variables freebsd-13 qemu
> #
> @@ -12,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
> NINJA='/usr/local/bin/ninja'
> PACKAGING_COMMAND='pkg'
> PIP3='/usr/local/bin/pip-3.8'
> -PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-virtualenv py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
> +PKGS='alsa-lib bash bzip2 ca_root_nss capstone4 ccache cdrkit-genisoimage cmocka ctags curl cyrus-sasl dbus diffutils dtc fusefs-libs3 gettext git glib gmake gnutls gsed gtk3 json-c libepoxy libffi libgcrypt libjpeg-turbo libnfs libspice-server libssh libtasn1 llvm lzo2 meson ncurses nettle ninja opencv perl5 pixman pkgconf png py39-numpy py39-pillow py39-pip py39-sphinx py39-sphinx_rtd_theme py39-yaml python3 rpm2cpio sdl2 sdl2_image snappy spice-protocol tesseract texinfo usbredir virglrenderer vte3 zstd'
> PYPI_PKGS=''
> PYTHON='/usr/local/bin/python3'
> diff --git a/.gitlab-ci.d/cirrus/macos-11.vars b/.gitlab-ci.d/cirrus/macos-11.vars
> index cfe9181fd4..bec6e862d4 100644
> --- a/.gitlab-ci.d/cirrus/macos-11.vars
> +++ b/.gitlab-ci.d/cirrus/macos-11.vars
> @@ -11,6 +11,6 @@ MAKE='/usr/local/bin/gmake'
> NINJA='/usr/local/bin/ninja'
> PACKAGING_COMMAND='brew'
> PIP3='/usr/local/bin/pip3'
> -PKGS='bash bc bzip2 capstone ccache ctags curl dbus diffutils dtc gcovr gettext git glib gnu-sed gnutls gtk+3 jemalloc jpeg-turbo libepoxy libffi libgcrypt libiscsi libnfs libpng libslirp libssh libtasn1 libusb llvm lzo make meson ncurses nettle ninja perl pixman pkg-config python3 rpm2cpio sdl2 sdl2_image snappy sparse spice-protocol tesseract texinfo usbredir vde vte3 zlib zstd'
> -PYPI_PKGS='PyYAML numpy pillow sphinx sphinx-rtd-theme virtualenv'
> +PKGS='bash bc bzip2 capstone ccache cmocka ctags curl dbus diffutils dtc gcovr gettext git glib gnu-sed gnutls gtk+3 jemalloc jpeg-turbo json-c libepoxy libffi libgcrypt libiscsi libnfs libpng libslirp libssh libtasn1 libusb llvm lzo make meson ncurses nettle ninja perl pixman pkg-config python3 rpm2cpio sdl2 sdl2_image snappy sparse spice-protocol tesseract texinfo usbredir vde vte3 zlib zstd'
> +PYPI_PKGS='PyYAML numpy pillow sphinx sphinx-rtd-theme'
> PYTHON='/usr/local/bin/python3'
> diff --git a/tests/docker/dockerfiles/alpine.docker b/tests/docker/dockerfiles/alpine.docker
> index 3f4c0f95cb..806cb19f17 100644
> --- a/tests/docker/dockerfiles/alpine.docker
> +++ b/tests/docker/dockerfiles/alpine.docker
> @@ -21,6 +21,7 @@ RUN apk update && \
> cdrkit \
> ceph-dev \
> clang \
> + cmocka-dev \
> ctags \
> curl-dev \
> cyrus-sasl-dev \
> @@ -39,6 +40,7 @@ RUN apk update && \
> glib-static \
> gnutls-dev \
> gtk+3.0-dev \
> + json-c-dev \
> libaio-dev \
> libbpf-dev \
> libcap-ng-dev \
> @@ -64,6 +66,7 @@ RUN apk update && \
> mesa-dev \
> meson \
> multipath-tools \
> + musl-dev \
> ncurses-dev \
> ndctl-dev \
> net-tools \
> @@ -81,7 +84,6 @@ RUN apk update && \
> py3-pip \
> py3-sphinx \
> py3-sphinx_rtd_theme \
> - py3-virtualenv \
> py3-yaml \
> python3 \
> rpm2cpio \
> diff --git a/tests/docker/dockerfiles/centos8.docker b/tests/docker/dockerfiles/centos8.docker
> index 10618bfa83..3c29883332 100644
> --- a/tests/docker/dockerfiles/centos8.docker
> +++ b/tests/docker/dockerfiles/centos8.docker
> @@ -6,11 +6,12 @@
>
> FROM quay.io/centos/centos:stream8
>
> -RUN dnf update -y && \
> +RUN dnf distro-sync -y && \
> dnf install 'dnf-command(config-manager)' -y && \
> dnf config-manager --set-enabled -y powertools && \
> dnf install -y centos-release-advanced-virtualization && \
> dnf install -y epel-release && \
> + dnf install -y epel-next-release && \
> dnf install -y \
> SDL2-devel \
> alsa-lib-devel \
> @@ -45,6 +46,7 @@ RUN dnf update -y && \
> gtk3-devel \
> hostname \
> jemalloc-devel \
> + json-c-devel \
> libaio-devel \
> libasan \
> libattr-devel \
> @@ -60,7 +62,6 @@ RUN dnf update -y && \
> libgcrypt-devel \
> libiscsi-devel \
> libjpeg-devel \
> - json-c-devel \
> libnfs-devel \
> libpmem-devel \
> libpng-devel \
> @@ -99,7 +100,6 @@ RUN dnf update -y && \
> python3-pip \
> python3-sphinx \
> python3-sphinx_rtd_theme \
> - python3-virtualenv \
> rdma-core-devel \
> rpm \
> sed \
> diff --git a/tests/docker/dockerfiles/debian-amd64.docker b/tests/docker/dockerfiles/debian-amd64.docker
> index 503e282802..8d78ba2484 100644
> --- a/tests/docker/dockerfiles/debian-amd64.docker
> +++ b/tests/docker/dockerfiles/debian-amd64.docker
> @@ -41,6 +41,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev \
> libcap-ng-dev \
> libcapstone-dev \
> + libcmocka-dev \
> libcurl4-gnutls-dev \
> libdaxctl-dev \
> libdrm-dev \
> @@ -59,6 +60,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev \
> libjemalloc-dev \
> libjpeg62-turbo-dev \
> + libjson-c-dev \
> liblttng-ust-dev \
> liblzo2-dev \
> libncursesw5-dev \
> diff --git a/tests/docker/dockerfiles/debian-arm64-cross.docker b/tests/docker/dockerfiles/debian-arm64-cross.docker
> index b60426834c..b7ba2c527f 100644
> --- a/tests/docker/dockerfiles/debian-arm64-cross.docker
> +++ b/tests/docker/dockerfiles/debian-arm64-cross.docker
> @@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:arm64 \
> libcap-ng-dev:arm64 \
> libcapstone-dev:arm64 \
> + libcmocka-dev:arm64 \
> libcurl4-gnutls-dev:arm64 \
> libdaxctl-dev:arm64 \
> libdrm-dev:arm64 \
> @@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:arm64 \
> libjemalloc-dev:arm64 \
> libjpeg62-turbo-dev:arm64 \
> + libjson-c-dev:arm64 \
> liblttng-ust-dev:arm64 \
> liblzo2-dev:arm64 \
> libncursesw5-dev:arm64 \
> diff --git a/tests/docker/dockerfiles/debian-armel-cross.docker b/tests/docker/dockerfiles/debian-armel-cross.docker
> index a6153e5a83..9b1778261e 100644
> --- a/tests/docker/dockerfiles/debian-armel-cross.docker
> +++ b/tests/docker/dockerfiles/debian-armel-cross.docker
> @@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:armel \
> libcap-ng-dev:armel \
> libcapstone-dev:armel \
> + libcmocka-dev:armel \
> libcurl4-gnutls-dev:armel \
> libdaxctl-dev:armel \
> libdrm-dev:armel \
> @@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:armel \
> libjemalloc-dev:armel \
> libjpeg62-turbo-dev:armel \
> + libjson-c-dev:armel \
> liblttng-ust-dev:armel \
> liblzo2-dev:armel \
> libncursesw5-dev:armel \
> diff --git a/tests/docker/dockerfiles/debian-armhf-cross.docker b/tests/docker/dockerfiles/debian-armhf-cross.docker
> index a2ebce96f8..addbc9a793 100644
> --- a/tests/docker/dockerfiles/debian-armhf-cross.docker
> +++ b/tests/docker/dockerfiles/debian-armhf-cross.docker
> @@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:armhf \
> libcap-ng-dev:armhf \
> libcapstone-dev:armhf \
> + libcmocka-dev:armhf \
> libcurl4-gnutls-dev:armhf \
> libdaxctl-dev:armhf \
> libdrm-dev:armhf \
> @@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:armhf \
> libjemalloc-dev:armhf \
> libjpeg62-turbo-dev:armhf \
> + libjson-c-dev:armhf \
> liblttng-ust-dev:armhf \
> liblzo2-dev:armhf \
> libncursesw5-dev:armhf \
> diff --git a/tests/docker/dockerfiles/debian-mips64el-cross.docker b/tests/docker/dockerfiles/debian-mips64el-cross.docker
> index b02dcb7fd9..1bb7d8e184 100644
> --- a/tests/docker/dockerfiles/debian-mips64el-cross.docker
> +++ b/tests/docker/dockerfiles/debian-mips64el-cross.docker
> @@ -83,6 +83,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:mips64el \
> libcap-ng-dev:mips64el \
> libcapstone-dev:mips64el \
> + libcmocka-dev:mips64el \
> libcurl4-gnutls-dev:mips64el \
> libdaxctl-dev:mips64el \
> libdrm-dev:mips64el \
> @@ -101,6 +102,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:mips64el \
> libjemalloc-dev:mips64el \
> libjpeg62-turbo-dev:mips64el \
> + libjson-c-dev:mips64el \
> liblttng-ust-dev:mips64el \
> liblzo2-dev:mips64el \
> libncursesw5-dev:mips64el \
> diff --git a/tests/docker/dockerfiles/debian-mipsel-cross.docker b/tests/docker/dockerfiles/debian-mipsel-cross.docker
> index b6d99ae324..a94b459b23 100644
> --- a/tests/docker/dockerfiles/debian-mipsel-cross.docker
> +++ b/tests/docker/dockerfiles/debian-mipsel-cross.docker
> @@ -83,6 +83,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:mipsel \
> libcap-ng-dev:mipsel \
> libcapstone-dev:mipsel \
> + libcmocka-dev:mipsel \
> libcurl4-gnutls-dev:mipsel \
> libdaxctl-dev:mipsel \
> libdrm-dev:mipsel \
> @@ -101,6 +102,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:mipsel \
> libjemalloc-dev:mipsel \
> libjpeg62-turbo-dev:mipsel \
> + libjson-c-dev:mipsel \
> liblttng-ust-dev:mipsel \
> liblzo2-dev:mipsel \
> libncursesw5-dev:mipsel \
> diff --git a/tests/docker/dockerfiles/debian-ppc64el-cross.docker b/tests/docker/dockerfiles/debian-ppc64el-cross.docker
> index bcf04bc90b..c641fd5d0e 100644
> --- a/tests/docker/dockerfiles/debian-ppc64el-cross.docker
> +++ b/tests/docker/dockerfiles/debian-ppc64el-cross.docker
> @@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:ppc64el \
> libcap-ng-dev:ppc64el \
> libcapstone-dev:ppc64el \
> + libcmocka-dev:ppc64el \
> libcurl4-gnutls-dev:ppc64el \
> libdaxctl-dev:ppc64el \
> libdrm-dev:ppc64el \
> @@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:ppc64el \
> libjemalloc-dev:ppc64el \
> libjpeg62-turbo-dev:ppc64el \
> + libjson-c-dev:ppc64el \
> liblttng-ust-dev:ppc64el \
> liblzo2-dev:ppc64el \
> libncursesw5-dev:ppc64el \
> diff --git a/tests/docker/dockerfiles/debian-s390x-cross.docker b/tests/docker/dockerfiles/debian-s390x-cross.docker
> index ff79a2cc4f..c0cbe09e66 100644
> --- a/tests/docker/dockerfiles/debian-s390x-cross.docker
> +++ b/tests/docker/dockerfiles/debian-s390x-cross.docker
> @@ -84,6 +84,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev:s390x \
> libcap-ng-dev:s390x \
> libcapstone-dev:s390x \
> + libcmocka-dev:s390x \
> libcurl4-gnutls-dev:s390x \
> libdaxctl-dev:s390x \
> libdrm-dev:s390x \
> @@ -102,6 +103,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev:s390x \
> libjemalloc-dev:s390x \
> libjpeg62-turbo-dev:s390x \
> + libjson-c-dev:s390x \
> liblttng-ust-dev:s390x \
> liblzo2-dev:s390x \
> libncursesw5-dev:s390x \
> diff --git a/tests/docker/dockerfiles/fedora.docker b/tests/docker/dockerfiles/fedora.docker
> index 2e6a84abfd..b39d311bbc 100644
> --- a/tests/docker/dockerfiles/fedora.docker
> +++ b/tests/docker/dockerfiles/fedora.docker
> @@ -53,12 +53,14 @@ exec "$@"' > /usr/bin/nosync && \
> gtk3-devel \
> hostname \
> jemalloc-devel \
> + json-c-devel \
> libaio-devel \
> libasan \
> libattr-devel \
> libbpf-devel \
> libcacard-devel \
> libcap-ng-devel \
> + libcmocka-devel \
> libcurl-devel \
> libdrm-devel \
> libepoxy-devel \
> @@ -106,7 +108,6 @@ exec "$@"' > /usr/bin/nosync && \
> python3-pip \
> python3-sphinx \
> python3-sphinx_rtd_theme \
> - python3-virtualenv \
> rdma-core-devel \
> rpm \
> sed \
> diff --git a/tests/docker/dockerfiles/opensuse-leap.docker b/tests/docker/dockerfiles/opensuse-leap.docker
> index 6f5993d602..047a435ab5 100644
> --- a/tests/docker/dockerfiles/opensuse-leap.docker
> +++ b/tests/docker/dockerfiles/opensuse-leap.docker
> @@ -1,10 +1,10 @@
> # THIS FILE WAS AUTO-GENERATED
> #
> -# $ lcitool dockerfile --layers all opensuse-leap-152 qemu
> +# $ lcitool dockerfile --layers all opensuse-leap-153 qemu
> #
> # https://gitlab.com/libvirt/libvirt-ci
>
> -FROM registry.opensuse.org/opensuse/leap:15.2
> +FROM registry.opensuse.org/opensuse/leap:15.3
>
> RUN zypper update -y && \
> zypper install -y \
> @@ -44,6 +44,7 @@ RUN zypper update -y && \
> libbz2-devel \
> libcacard-devel \
> libcap-ng-devel \
> + libcmocka-devel \
> libcurl-devel \
> libdrm-devel \
> libepoxy-devel \
> @@ -53,6 +54,7 @@ RUN zypper update -y && \
> libgnutls-devel \
> libiscsi-devel \
> libjpeg8-devel \
> + libjson-c-devel \
> libndctl-devel \
> libnettle-devel \
> libnfs-devel \
> @@ -94,7 +96,6 @@ RUN zypper update -y && \
> python3-pip \
> python3-setuptools \
> python3-sphinx_rtd_theme \
> - python3-virtualenv \
> python3-wheel \
> rdma-core-devel \
> rpm \
> diff --git a/tests/docker/dockerfiles/ubuntu2004.docker b/tests/docker/dockerfiles/ubuntu2004.docker
> index a3b38884e3..99803b343b 100644
> --- a/tests/docker/dockerfiles/ubuntu2004.docker
> +++ b/tests/docker/dockerfiles/ubuntu2004.docker
> @@ -40,6 +40,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libcacard-dev \
> libcap-ng-dev \
> libcapstone-dev \
> + libcmocka-dev \
> libcurl4-gnutls-dev \
> libdaxctl-dev \
> libdrm-dev \
> @@ -58,6 +59,7 @@ RUN export DEBIAN_FRONTEND=noninteractive && \
> libiscsi-dev \
> libjemalloc-dev \
> libjpeg-turbo8-dev \
> + libjson-c-dev \
> liblttng-ust-dev \
> liblzo2-dev \
> libncursesw5-dev \
> diff --git a/tests/lcitool/libvirt-ci b/tests/lcitool/libvirt-ci
> index f83b916d5e..324355cf62 160000
> --- a/tests/lcitool/libvirt-ci
> +++ b/tests/lcitool/libvirt-ci
> @@ -1 +1 @@
> -Subproject commit f83b916d5efa4bd33fbf4b7ea41bf6d535cc63fb
> +Subproject commit 324355cf62e86fb551408575afb123bac989ac37
> diff --git a/tests/lcitool/projects/qemu.yml b/tests/lcitool/projects/qemu.yml
> index d068a7a8de..d9f826f8eb 100644
> --- a/tests/lcitool/projects/qemu.yml
> +++ b/tests/lcitool/projects/qemu.yml
> @@ -9,6 +9,7 @@ packages:
> - capstone
> - ccache
> - clang
> + - cmocka
> - column
> - ctags
> - cyrus-sasl
> @@ -26,15 +27,16 @@ packages:
> - genisoimage
> - glib2
> - glib2-static
> - - glibc-static
> - glusterfs
> - gnutls
> - gtk3
> - hostname
> + - json-c
> - libaio
> - libattr
> - libasan
> - libbpf
> + - libc-static
> - libcacard
> - libcap-ng
> - libcurl
> @@ -90,7 +92,7 @@ packages:
> - python3-pip
> - python3-sphinx
> - python3-sphinx-rtd-theme
> - - python3-virtualenv
> + - python3-venv
> - rpm2cpio
> - sdl2
> - sdl2-image
> diff --git a/tests/lcitool/refresh b/tests/lcitool/refresh
> index 5e260f8cd6..2a59150322 100755
> --- a/tests/lcitool/refresh
> +++ b/tests/lcitool/refresh
> @@ -25,7 +25,7 @@ self_dir = Path(__file__).parent
> src_dir = self_dir.parent.parent
> dockerfiles_dir = Path(src_dir, "tests", "docker", "dockerfiles")
>
> -lcitool_path = Path(self_dir, "libvirt-ci", "lcitool")
> +lcitool_path = Path(self_dir, "libvirt-ci", "bin", "lcitool")
>
> lcitool_cmd = [lcitool_path, "--data-dir", self_dir]
>
> @@ -112,7 +112,7 @@ try:
> generate_dockerfile("debian-amd64", "debian-11",
> trailer="".join(debian11_extras))
> generate_dockerfile("fedora", "fedora-35")
> - generate_dockerfile("opensuse-leap", "opensuse-leap-152")
> + generate_dockerfile("opensuse-leap", "opensuse-leap-153")
> generate_dockerfile("ubuntu2004", "ubuntu-2004",
> trailer="".join(ubuntu2004_tsanhack))
>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v1 03/13] gitlab: drop 'containers-layer2' stage
2022-07-25 14:05 ` [PATCH v1 03/13] gitlab: drop 'containers-layer2' stage Alex Bennée
@ 2022-07-26 14:30 ` Philippe Mathieu-Daudé via
0 siblings, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé via @ 2022-07-26 14:30 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Thomas Huth, Wainer dos Santos Moschetta,
Beraldo Leal
On 25/7/22 16:05, Alex Bennée wrote:
> From: Daniel P. Berrangé <berrange@redhat.com>
>
> Since we express dependancies via a 'needs' clause, we don't need to
Typo "dependencies".
> split container builds into separate stages. GitLab happily lets jobs
> depend on other jobs in the same stage and will run them when possible.
Nice!
> Acked-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> Message-Id: <20220722130431.2319019-4-berrange@redhat.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .gitlab-ci.d/container-cross.yml | 24 ++++++++++++------------
> .gitlab-ci.d/stages.yml | 1 -
> 2 files changed, 12 insertions(+), 13 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v1 06/13] semihosting: Don't return negative values on qemu_semihosting_console_write() failure
2022-07-25 14:05 ` [PATCH v1 06/13] semihosting: Don't return negative values on qemu_semihosting_console_write() failure Alex Bennée
@ 2022-07-26 14:31 ` Philippe Mathieu-Daudé via
0 siblings, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé via @ 2022-07-26 14:31 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Richard Henderson
On 25/7/22 16:05, Alex Bennée wrote:
> From: Peter Maydell <peter.maydell@linaro.org>
>
> The documentation comment for qemu_semihosting_console_write() says
> * Returns: number of bytes written -- this should only ever be short
> * on some sort of i/o error.
>
> and the callsites rely on this. However, the implementation code
> path which sends console output to a chardev doesn't honour this,
> and will return negative values on error. Bring it into line with
> the other implementation codepaths and the documentation, so that
> it returns 0 on error.
>
> Spotted by Coverity, because console_write() passes the return value
> to unlock_user(), which doesn't accept a negative length.
>
> Resolves: Coverity CID 1490288
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> Message-Id: <20220719121110.225657-2-peter.maydell@linaro.org>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> semihosting/console.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH v1 01/13] tests: refresh to latest libvirt-ci module
2022-07-25 14:05 ` [PATCH v1 01/13] tests: refresh to latest libvirt-ci module Alex Bennée
2022-07-25 18:56 ` Richard Henderson
@ 2022-07-26 14:34 ` Philippe Mathieu-Daudé via
1 sibling, 0 replies; 19+ messages in thread
From: Philippe Mathieu-Daudé via @ 2022-07-26 14:34 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, aurelien, pbonzini, stefanha, crosa,
peter.maydell, Thomas Huth, Ed Maste, Li-Wen Hsu,
Wainer dos Santos Moschetta, Beraldo Leal
On 25/7/22 16:05, Alex Bennée wrote:
> From: Daniel P. Berrangé <berrange@redhat.com>
>
> Notable changes:
>
> - libvirt-ci source tree was re-arranged, so the script we
> run now lives in a bin/ sub-dir
>
> - opensuse 15.2 is replaced by opensuse 15.3
>
> - libslirp is temporarily dropped on opensuse as the
> libslirp-version.h is broken
>
> https://bugzilla.opensuse.org/show_bug.cgi?id=1201551
>
> - The incorrectly named python3-virtualenv module was
> changed to python3-venv, but most distros don't need
> any package as 'venv' is a standard part of python
>
> - glibc-static was renamed to libc-static, to reflect
> fact that it isn't going to be glibc on all distros
>
> - The cmocka/json-c deps that were manually added to
> the centos dockerfile and are now consistently added
> to all targets
>
> Acked-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> Message-Id: <20220722130431.2319019-2-berrange@redhat.com>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .gitlab-ci.d/cirrus/freebsd-12.vars | 3 +--
> .gitlab-ci.d/cirrus/freebsd-13.vars | 3 +--
> .gitlab-ci.d/cirrus/macos-11.vars | 4 ++--
> tests/docker/dockerfiles/alpine.docker | 4 +++-
> tests/docker/dockerfiles/centos8.docker | 6 +++---
> tests/docker/dockerfiles/debian-amd64.docker | 2 ++
> tests/docker/dockerfiles/debian-arm64-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-armel-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-armhf-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-mips64el-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-mipsel-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-ppc64el-cross.docker | 2 ++
> tests/docker/dockerfiles/debian-s390x-cross.docker | 2 ++
> tests/docker/dockerfiles/fedora.docker | 3 ++-
> tests/docker/dockerfiles/opensuse-leap.docker | 7 ++++---
> tests/docker/dockerfiles/ubuntu2004.docker | 2 ++
> tests/lcitool/libvirt-ci | 2 +-
> tests/lcitool/projects/qemu.yml | 6 ++++--
> tests/lcitool/refresh | 4 ++--
> 19 files changed, 41 insertions(+), 19 deletions(-)
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2022-07-26 14:42 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-25 14:05 [PATCH v1 00/13] fixes for 7.1 (testing, docs, semihosting) Alex Bennée
2022-07-25 14:05 ` [PATCH v1 01/13] tests: refresh to latest libvirt-ci module Alex Bennée
2022-07-25 18:56 ` Richard Henderson
2022-07-26 14:34 ` Philippe Mathieu-Daudé via
2022-07-25 14:05 ` [PATCH v1 02/13] gitlab: show testlog.txt contents when cirrus/custom-runner jobs fail Alex Bennée
2022-07-25 14:05 ` [PATCH v1 03/13] gitlab: drop 'containers-layer2' stage Alex Bennée
2022-07-26 14:30 ` Philippe Mathieu-Daudé via
2022-07-25 14:05 ` [PATCH v1 04/13] .cirrus.yml: Change winsymlinks to 'native' Alex Bennée
2022-07-25 14:05 ` [PATCH v1 05/13] .gitlab-ci.d/windows.yml: Enable native Windows symlink Alex Bennée
2022-07-25 14:05 ` [PATCH v1 06/13] semihosting: Don't return negative values on qemu_semihosting_console_write() failure Alex Bennée
2022-07-26 14:31 ` Philippe Mathieu-Daudé via
2022-07-25 14:05 ` [PATCH v1 07/13] semihosting: Don't copy buffer after console_write() Alex Bennée
2022-07-25 14:05 ` [PATCH v1 08/13] semihosting: Check for errors on SET_ARG() Alex Bennée
2022-07-25 14:05 ` [PATCH v1 09/13] semihosting: Fix handling of buffer in TARGET_SYS_TMPNAM Alex Bennée
2022-07-25 14:05 ` [PATCH v1 10/13] qapi: Add exit-failure PanicAction Alex Bennée
2022-07-25 14:05 ` [PATCH v1 11/13] tests/tcg/s390x: Test unaligned accesses to lowcore Alex Bennée
2022-07-25 15:17 ` Thomas Huth
2022-07-25 14:05 ` [PATCH v1 12/13] docs/devel: fix description of OBJECT_DECLARE_SIMPLE_TYPE Alex Bennée
2022-07-25 14:05 ` [PATCH v1 13/13] qemu-options: bring the kernel and image options together Alex Bennée
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.