* [PULL 00/12] target-arm queue
@ 2019-12-20 14:26 Peter Maydell
2020-01-06 10:32 ` Peter Maydell
0 siblings, 1 reply; 27+ messages in thread
From: Peter Maydell @ 2019-12-20 14:26 UTC (permalink / raw)
To: qemu-devel
One last arm pullreq before I stop work for the end of the year...
-- PMM
The following changes since commit 8e5943260a8f765216674ee87ce8588cc4e7463e:
Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-pull-request' into staging (2019-12-20 12:46:10 +0000)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20191220
for you to fetch changes up to c8fa6079eb35888587f1be27c1590da4edcc5098:
arm/arm-powerctl: rebuild hflags after setting CP15 bits in arm_set_cpu_on() (2019-12-20 14:03:00 +0000)
----------------------------------------------------------------
target-arm queue:
* Support emulating the generic timers at frequencies other than 62.5MHz
* Various fixes for SMMUv3 emulation bugs
* Improve assert error message for hflags mismatches
* arm-powerctl: rebuild hflags after setting CP15 bits in arm_set_cpu_on()
----------------------------------------------------------------
Andrew Jeffery (4):
target/arm: Remove redundant scaling of nexttick
target/arm: Abstract the generic timer frequency
target/arm: Prepare generic timer for per-platform CNTFRQ
ast2600: Configure CNTFRQ at 1125MHz
Niek Linnenbank (1):
arm/arm-powerctl: rebuild hflags after setting CP15 bits in arm_set_cpu_on()
Philippe Mathieu-Daudé (1):
target/arm: Display helpful message when hflags mismatch
Simon Veith (6):
hw/arm/smmuv3: Apply address mask to linear strtab base address
hw/arm/smmuv3: Correct SMMU_BASE_ADDR_MASK value
hw/arm/smmuv3: Check stream IDs against actual table LOG2SIZE
hw/arm/smmuv3: Align stream table base address to table size
hw/arm/smmuv3: Use correct bit positions in EVT_SET_ADDR2 macro
hw/arm/smmuv3: Report F_STE_FETCH fault address in correct word position
hw/arm/smmuv3-internal.h | 6 ++---
target/arm/cpu.h | 5 ++++
hw/arm/aspeed_ast2600.c | 3 +++
hw/arm/smmuv3.c | 28 +++++++++++++++-----
target/arm/arm-powerctl.c | 3 +++
target/arm/cpu.c | 65 +++++++++++++++++++++++++++++++++++++++++------
target/arm/helper.c | 42 +++++++++++++++++++++++-------
7 files changed, 125 insertions(+), 27 deletions(-)
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2019-12-20 14:26 Peter Maydell
@ 2020-01-06 10:32 ` Peter Maydell
0 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-01-06 10:32 UTC (permalink / raw)
To: QEMU Developers
On Fri, 20 Dec 2019 at 14:26, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> One last arm pullreq before I stop work for the end of the year...
>
> -- PMM
>
> The following changes since commit 8e5943260a8f765216674ee87ce8588cc4e7463e:
>
> Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-pull-request' into staging (2019-12-20 12:46:10 +0000)
>
> are available in the Git repository at:
>
> https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20191220
>
> for you to fetch changes up to c8fa6079eb35888587f1be27c1590da4edcc5098:
>
> arm/arm-powerctl: rebuild hflags after setting CP15 bits in arm_set_cpu_on() (2019-12-20 14:03:00 +0000)
>
> ----------------------------------------------------------------
> target-arm queue:
> * Support emulating the generic timers at frequencies other than 62.5MHz
> * Various fixes for SMMUv3 emulation bugs
> * Improve assert error message for hflags mismatches
> * arm-powerctl: rebuild hflags after setting CP15 bits in arm_set_cpu_on()
>
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/5.0
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2020-04-14 16:26 Peter Maydell
2020-04-14 16:26 ` [PULL 01/12] osdep.h: Drop no-longer-needed Coverity workarounds Peter Maydell
` (12 more replies)
0 siblings, 13 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
Almost nothing in here is arm-related, but the target-arm
queue was convenient for these last minute bits and pieces
for 5.0...
thanks
-- PMM
The following changes since commit 14e5526b51910efd62cd31cd95b49baca975c83f:
Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2020-04-13 15:42:51 +0100)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20200414
for you to fetch changes up to 84f82ddcbb4ac4ed04c8675e85155329f23184f0:
Deprecate KVM support for AArch32 (2020-04-14 17:20:22 +0100)
----------------------------------------------------------------
patch queue:
* Fix some problems that trip up Coverity's scanner
* run-coverity-scan: New script automating the scan-and-upload process
* docs: Improve our gdbstub documentation
* configure: Honour --disable-werror for Sphinx
* docs: Fix errors produced when building with Sphinx 3.0
* docs: Require Sphinx 1.6 or better
* Add deprecation notice for KVM support on AArch32 hosts
----------------------------------------------------------------
Peter Maydell (12):
osdep.h: Drop no-longer-needed Coverity workarounds
thread.h: Fix Coverity version of qemu_cond_timedwait()
thread.h: Remove trailing semicolons from Coverity qemu_mutex_lock() etc
linux-user/flatload.c: Use "" for include of QEMU header target_flat.h
scripts/run-coverity-scan: Script to run Coverity Scan build
scripts/coverity-scan: Add Docker support
docs: Improve our gdbstub documentation
configure: Honour --disable-werror for Sphinx
scripts/kernel-doc: Add missing close-paren in c:function directives
kernel-doc: Use c:struct for Sphinx 3.0 and later
docs: Require Sphinx 1.6 or better
Deprecate KVM support for AArch32
configure | 9 +-
Makefile | 2 +-
include/qemu/osdep.h | 14 -
include/qemu/thread.h | 12 +-
linux-user/flatload.c | 2 +-
MAINTAINERS | 5 +
docs/conf.py | 6 +-
docs/sphinx/kerneldoc.py | 1 +
docs/system/deprecated.rst | 8 +
docs/system/gdb.rst | 22 +-
qemu-options.hx | 24 +-
scripts/coverity-scan/coverity-scan.docker | 131 ++++++++++
scripts/coverity-scan/run-coverity-scan | 401 +++++++++++++++++++++++++++++
scripts/kernel-doc | 18 +-
14 files changed, 615 insertions(+), 40 deletions(-)
create mode 100644 scripts/coverity-scan/coverity-scan.docker
create mode 100755 scripts/coverity-scan/run-coverity-scan
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 01/12] osdep.h: Drop no-longer-needed Coverity workarounds
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 02/12] thread.h: Fix Coverity version of qemu_cond_timedwait() Peter Maydell
` (11 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
In commit a1a98357e3fd in 2018 we added some workarounds for Coverity
not being able to handle the _Float* types introduced by recent
glibc. Newer versions of the Coverity scan tools have support for
these types, and will fail with errors about duplicate typedefs if we
have our workaround. Remove our copy of the typedefs.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20200319193323.2038-2-peter.maydell@linaro.org
---
include/qemu/osdep.h | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h
index 9bd3dcfd136..20f5c5f197d 100644
--- a/include/qemu/osdep.h
+++ b/include/qemu/osdep.h
@@ -33,20 +33,6 @@
#else
#include "exec/poison.h"
#endif
-#ifdef __COVERITY__
-/* Coverity does not like the new _Float* types that are used by
- * recent glibc, and croaks on every single file that includes
- * stdlib.h. These typedefs are enough to please it.
- *
- * Note that these fix parse errors so they cannot be placed in
- * scripts/coverity-model.c.
- */
-typedef float _Float32;
-typedef double _Float32x;
-typedef double _Float64;
-typedef __float80 _Float64x;
-typedef __float128 _Float128;
-#endif
#include "qemu/compiler.h"
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 02/12] thread.h: Fix Coverity version of qemu_cond_timedwait()
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
2020-04-14 16:26 ` [PULL 01/12] osdep.h: Drop no-longer-needed Coverity workarounds Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 03/12] thread.h: Remove trailing semicolons from Coverity qemu_mutex_lock() etc Peter Maydell
` (10 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
For Coverity's benefit, we provide simpler versions of functions like
qemu_mutex_lock(), qemu_cond_wait() and qemu_cond_timedwait(). When
we added qemu_cond_timedwait() in commit 3dcc9c6ec4ea, a cut and
paste error meant that the Coverity version of qemu_cond_timedwait()
was using the wrong _impl function, which makes the Coverity parser
complain:
"/qemu/include/qemu/thread.h", line 159: warning #140: too many arguments in
function call
return qemu_cond_timedwait(cond, mutex, ms);
^
"/qemu/include/qemu/thread.h", line 159: warning #120: return value type does
not match the function type
return qemu_cond_timedwait(cond, mutex, ms);
^
"/qemu/include/qemu/thread.h", line 156: warning #1563: function
"qemu_cond_timedwait" not emitted, consider modeling it or review
parse diagnostics to improve fidelity
static inline bool (qemu_cond_timedwait)(QemuCond *cond, QemuMutex *mutex,
^
These aren't fatal, but reduce the scope of the analysis. Fix the error.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20200319193323.2038-3-peter.maydell@linaro.org
---
include/qemu/thread.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/qemu/thread.h b/include/qemu/thread.h
index 047db0307e7..10262c63f58 100644
--- a/include/qemu/thread.h
+++ b/include/qemu/thread.h
@@ -67,7 +67,7 @@ extern QemuCondTimedWaitFunc qemu_cond_timedwait_func;
#define qemu_cond_wait(c, m) \
qemu_cond_wait_impl(c, m, __FILE__, __LINE__);
#define qemu_cond_timedwait(c, m, ms) \
- qemu_cond_wait_impl(c, m, ms, __FILE__, __LINE__);
+ qemu_cond_timedwait_impl(c, m, ms, __FILE__, __LINE__);
#else
#define qemu_mutex_lock(m) ({ \
QemuMutexLockFunc _f = atomic_read(&qemu_mutex_lock_func); \
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 03/12] thread.h: Remove trailing semicolons from Coverity qemu_mutex_lock() etc
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
2020-04-14 16:26 ` [PULL 01/12] osdep.h: Drop no-longer-needed Coverity workarounds Peter Maydell
2020-04-14 16:26 ` [PULL 02/12] thread.h: Fix Coverity version of qemu_cond_timedwait() Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 04/12] linux-user/flatload.c: Use "" for include of QEMU header target_flat.h Peter Maydell
` (9 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
All the Coverity-specific definitions of qemu_mutex_lock() and friends
have a trailing semicolon. This works fine almost everywhere because
of QEMU's mandatory-braces coding style and because most callsites are
simple, but target/s390x/sigp.c has a use of qemu_mutex_trylock() as
an if() statement, which makes the ';' a syntax error:
"../target/s390x/sigp.c", line 461: warning #18: expected a ")"
if (qemu_mutex_trylock(&qemu_sigp_mutex)) {
^
Remove the bogus semicolons from the macro definitions.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20200319193323.2038-4-peter.maydell@linaro.org
---
include/qemu/thread.h | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/include/qemu/thread.h b/include/qemu/thread.h
index 10262c63f58..d22848138ea 100644
--- a/include/qemu/thread.h
+++ b/include/qemu/thread.h
@@ -57,17 +57,17 @@ extern QemuCondTimedWaitFunc qemu_cond_timedwait_func;
* hide them.
*/
#define qemu_mutex_lock(m) \
- qemu_mutex_lock_impl(m, __FILE__, __LINE__);
+ qemu_mutex_lock_impl(m, __FILE__, __LINE__)
#define qemu_mutex_trylock(m) \
- qemu_mutex_trylock_impl(m, __FILE__, __LINE__);
+ qemu_mutex_trylock_impl(m, __FILE__, __LINE__)
#define qemu_rec_mutex_lock(m) \
- qemu_rec_mutex_lock_impl(m, __FILE__, __LINE__);
+ qemu_rec_mutex_lock_impl(m, __FILE__, __LINE__)
#define qemu_rec_mutex_trylock(m) \
- qemu_rec_mutex_trylock_impl(m, __FILE__, __LINE__);
+ qemu_rec_mutex_trylock_impl(m, __FILE__, __LINE__)
#define qemu_cond_wait(c, m) \
- qemu_cond_wait_impl(c, m, __FILE__, __LINE__);
+ qemu_cond_wait_impl(c, m, __FILE__, __LINE__)
#define qemu_cond_timedwait(c, m, ms) \
- qemu_cond_timedwait_impl(c, m, ms, __FILE__, __LINE__);
+ qemu_cond_timedwait_impl(c, m, ms, __FILE__, __LINE__)
#else
#define qemu_mutex_lock(m) ({ \
QemuMutexLockFunc _f = atomic_read(&qemu_mutex_lock_func); \
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 04/12] linux-user/flatload.c: Use "" for include of QEMU header target_flat.h
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (2 preceding siblings ...)
2020-04-14 16:26 ` [PULL 03/12] thread.h: Remove trailing semicolons from Coverity qemu_mutex_lock() etc Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 05/12] scripts/run-coverity-scan: Script to run Coverity Scan build Peter Maydell
` (8 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
The target_flat.h file is a QEMU header, so we should include it using
quotes, not angle brackets.
Coverity otherwise is unable to find the header:
"../linux-user/flatload.c", line 40: error #1712: cannot open source file
"target_flat.h"
#include <target_flat.h>
^
because the relevant directory is only on the -iquote path, not the -I path.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20200319193323.2038-5-peter.maydell@linaro.org
---
linux-user/flatload.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/linux-user/flatload.c b/linux-user/flatload.c
index 0122ab3afe6..66901f39cc5 100644
--- a/linux-user/flatload.c
+++ b/linux-user/flatload.c
@@ -37,7 +37,7 @@
#include "qemu.h"
#include "flat.h"
-#include <target_flat.h>
+#include "target_flat.h"
//#define DEBUG
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 05/12] scripts/run-coverity-scan: Script to run Coverity Scan build
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (3 preceding siblings ...)
2020-04-14 16:26 ` [PULL 04/12] linux-user/flatload.c: Use "" for include of QEMU header target_flat.h Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 06/12] scripts/coverity-scan: Add Docker support Peter Maydell
` (7 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
Add a new script to automate the process of running the Coverity
Scan build tools and uploading the resulting tarball to the
website.
This is intended eventually to be driven from Travis,
but it can be run locally, if you are a maintainer of the
QEMU project on the Coverity Scan website and have the secret
upload token.
The script must be run on a Fedora 30 system. Support for using a
Docker container is added in a following commit.
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20200319193323.2038-6-peter.maydell@linaro.org
---
MAINTAINERS | 5 +
scripts/coverity-scan/run-coverity-scan | 311 ++++++++++++++++++++++++
2 files changed, 316 insertions(+)
create mode 100755 scripts/coverity-scan/run-coverity-scan
diff --git a/MAINTAINERS b/MAINTAINERS
index 5f93e8c01d3..8cbc1fac2bf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2038,6 +2038,11 @@ M: Markus Armbruster <armbru@redhat.com>
S: Supported
F: scripts/coverity-model.c
+Coverity Scan integration
+M: Peter Maydell <peter.maydell@linaro.org>
+S: Maintained
+F: scripts/coverity-scan/
+
Device Tree
M: Alistair Francis <alistair.francis@wdc.com>
R: David Gibson <david@gibson.dropbear.id.au>
diff --git a/scripts/coverity-scan/run-coverity-scan b/scripts/coverity-scan/run-coverity-scan
new file mode 100755
index 00000000000..d40b51969fa
--- /dev/null
+++ b/scripts/coverity-scan/run-coverity-scan
@@ -0,0 +1,311 @@
+#!/bin/sh -e
+
+# Upload a created tarball to Coverity Scan, as per
+# https://scan.coverity.com/projects/qemu/builds/new
+
+# This work is licensed under the terms of the GNU GPL version 2,
+# or (at your option) any later version.
+# See the COPYING file in the top-level directory.
+#
+# Copyright (c) 2017-2020 Linaro Limited
+# Written by Peter Maydell
+
+# Note that this script will automatically download and
+# run the (closed-source) coverity build tools, so don't
+# use it if you don't trust them!
+
+# This script assumes that you're running it from a QEMU source
+# tree, and that tree is a fresh clean one, because we do an in-tree
+# build. (This is necessary so that the filenames that the Coverity
+# Scan server sees are relative paths that match up with the component
+# regular expressions it uses; an out-of-tree build won't work for this.)
+# The host machine should have as many of QEMU's dependencies
+# installed as possible, for maximum coverity coverage.
+
+# To do an upload you need to be a maintainer in the Coverity online
+# service, and you will need to know the "Coverity token", which is a
+# secret 8 digit hex string. You can find that from the web UI in the
+# project settings, if you have maintainer access there.
+
+# Command line options:
+# --dry-run : run the tools, but don't actually do the upload
+# --update-tools-only : update the cached copy of the tools, but don't run them
+# --tokenfile : file to read Coverity token from
+# --version ver : specify version being analyzed (default: ask git)
+# --description desc : specify description of this version (default: ask git)
+# --srcdir : QEMU source tree to analyze (default: current working dir)
+# --results-tarball : path to copy the results tarball to (default: don't
+# copy it anywhere, just upload it)
+#
+# User-specifiable environment variables:
+# COVERITY_TOKEN -- Coverity token
+# COVERITY_EMAIL -- the email address to use for uploads (default:
+# looks at your git user.email config)
+# COVERITY_BUILD_CMD -- make command (default: 'make -jN' where N is
+# number of CPUs as determined by 'nproc')
+# COVERITY_TOOL_BASE -- set to directory to put coverity tools
+# (default: /tmp/coverity-tools)
+#
+# You must specify the token, either by environment variable or by
+# putting it in a file and using --tokenfile. Everything else has
+# a reasonable default if this is run from a git tree.
+
+check_upload_permissions() {
+ # Check whether we can do an upload to the server; will exit the script
+ # with status 1 if the check failed (usually a bad token);
+ # will exit the script with status 0 if the check indicated that we
+ # can't upload yet (ie we are at quota)
+ # Assumes that PROJTOKEN, PROJNAME and DRYRUN have been initialized.
+
+ echo "Checking upload permissions..."
+
+ if ! up_perm="$(wget https://scan.coverity.com/api/upload_permitted --post-data "token=$PROJTOKEN&project=$PROJNAME" -q -O -)"; then
+ echo "Coverity Scan API access denied: bad token?"
+ exit 1
+ fi
+
+ # Really up_perm is a JSON response with either
+ # {upload_permitted:true} or {next_upload_permitted_at:<date>}
+ # We do some hacky string parsing instead of properly parsing it.
+ case "$up_perm" in
+ *upload_permitted*true*)
+ echo "Coverity Scan: upload permitted"
+ ;;
+ *next_upload_permitted_at*)
+ if [ "$DRYRUN" = yes ]; then
+ echo "Coverity Scan: upload quota reached, continuing dry run"
+ else
+ echo "Coverity Scan: upload quota reached; stopping here"
+ # Exit success as this isn't a build error.
+ exit 0
+ fi
+ ;;
+ *)
+ echo "Coverity Scan upload check: unexpected result $up_perm"
+ exit 1
+ ;;
+ esac
+}
+
+
+update_coverity_tools () {
+ # Check for whether we need to download the Coverity tools
+ # (either because we don't have a copy, or because it's out of date)
+ # Assumes that COVERITY_TOOL_BASE, PROJTOKEN and PROJNAME are set.
+
+ mkdir -p "$COVERITY_TOOL_BASE"
+ cd "$COVERITY_TOOL_BASE"
+
+ echo "Checking for new version of coverity build tools..."
+ wget https://scan.coverity.com/download/linux64 --post-data "token=$PROJTOKEN&project=$PROJNAME&md5=1" -O coverity_tool.md5.new
+
+ if ! cmp -s coverity_tool.md5 coverity_tool.md5.new; then
+ # out of date md5 or no md5: download new build tool
+ # blow away the old build tool
+ echo "Downloading coverity build tools..."
+ rm -rf coverity_tool coverity_tool.tgz
+ wget https://scan.coverity.com/download/linux64 --post-data "token=$PROJTOKEN&project=$PROJNAME" -O coverity_tool.tgz
+ if ! (cat coverity_tool.md5.new; echo " coverity_tool.tgz") | md5sum -c --status; then
+ echo "Downloaded tarball didn't match md5sum!"
+ exit 1
+ fi
+ # extract the new one, keeping it corralled in a 'coverity_tool' directory
+ echo "Unpacking coverity build tools..."
+ mkdir -p coverity_tool
+ cd coverity_tool
+ tar xf ../coverity_tool.tgz
+ cd ..
+ mv coverity_tool.md5.new coverity_tool.md5
+ fi
+
+ rm -f coverity_tool.md5.new
+}
+
+
+# Check user-provided environment variables and arguments
+DRYRUN=no
+UPDATE_ONLY=no
+
+while [ "$#" -ge 1 ]; do
+ case "$1" in
+ --dry-run)
+ shift
+ DRYRUN=yes
+ ;;
+ --update-tools-only)
+ shift
+ UPDATE_ONLY=yes
+ ;;
+ --version)
+ shift
+ if [ $# -eq 0 ]; then
+ echo "--version needs an argument"
+ exit 1
+ fi
+ VERSION="$1"
+ shift
+ ;;
+ --description)
+ shift
+ if [ $# -eq 0 ]; then
+ echo "--description needs an argument"
+ exit 1
+ fi
+ DESCRIPTION="$1"
+ shift
+ ;;
+ --tokenfile)
+ shift
+ if [ $# -eq 0 ]; then
+ echo "--tokenfile needs an argument"
+ exit 1
+ fi
+ COVERITY_TOKEN="$(cat "$1")"
+ shift
+ ;;
+ --srcdir)
+ shift
+ if [ $# -eq 0 ]; then
+ echo "--srcdir needs an argument"
+ exit 1
+ fi
+ SRCDIR="$1"
+ shift
+ ;;
+ --results-tarball)
+ shift
+ if [ $# -eq 0 ]; then
+ echo "--results-tarball needs an argument"
+ exit 1
+ fi
+ RESULTSTARBALL="$1"
+ shift
+ ;;
+ *)
+ echo "Unexpected argument '$1'"
+ exit 1
+ ;;
+ esac
+done
+
+if [ -z "$COVERITY_TOKEN" ]; then
+ echo "COVERITY_TOKEN environment variable not set"
+ exit 1
+fi
+
+if [ -z "$COVERITY_BUILD_CMD" ]; then
+ NPROC=$(nproc)
+ COVERITY_BUILD_CMD="make -j$NPROC"
+ echo "COVERITY_BUILD_CMD: using default '$COVERITY_BUILD_CMD'"
+fi
+
+if [ -z "$COVERITY_TOOL_BASE" ]; then
+ echo "COVERITY_TOOL_BASE: using default /tmp/coverity-tools"
+ COVERITY_TOOL_BASE=/tmp/coverity-tools
+fi
+
+if [ -z "$SRCDIR" ]; then
+ SRCDIR="$PWD"
+fi
+
+PROJTOKEN="$COVERITY_TOKEN"
+PROJNAME=QEMU
+TARBALL=cov-int.tar.xz
+
+
+if [ "$UPDATE_ONLY" = yes ]; then
+ # Just do the tools update; we don't need to check whether
+ # we are in a source tree or have upload rights for this,
+ # so do it before some of the command line and source tree checks.
+ update_coverity_tools
+ exit 0
+fi
+
+cd "$SRCDIR"
+
+echo "Checking this is a QEMU source tree..."
+if ! [ -e "$SRCDIR/VERSION" ]; then
+ echo "Not in a QEMU source tree?"
+ exit 1
+fi
+
+# Fill in defaults used by the non-update-only process
+if [ -z "$VERSION" ]; then
+ VERSION="$(git describe --always HEAD)"
+fi
+
+if [ -z "$DESCRIPTION" ]; then
+ DESCRIPTION="$(git rev-parse HEAD)"
+fi
+
+if [ -z "$COVERITY_EMAIL" ]; then
+ COVERITY_EMAIL="$(git config user.email)"
+fi
+
+check_upload_permissions
+
+update_coverity_tools
+
+TOOLBIN="$(cd "$COVERITY_TOOL_BASE" && echo $PWD/coverity_tool/cov-analysis-*/bin)"
+
+if ! test -x "$TOOLBIN/cov-build"; then
+ echo "Couldn't find cov-build in the coverity build-tool directory??"
+ exit 1
+fi
+
+export PATH="$TOOLBIN:$PATH"
+
+cd "$SRCDIR"
+
+echo "Doing make distclean..."
+make distclean
+
+echo "Configuring..."
+# We configure with a fixed set of enables here to ensure that we don't
+# accidentally reduce the scope of the analysis by doing the build on
+# the system that's missing a dependency that we need to build part of
+# the codebase.
+./configure --disable-modules --enable-sdl --enable-gtk \
+ --enable-opengl --enable-vte --enable-gnutls \
+ --enable-nettle --enable-curses --enable-curl \
+ --audio-drv-list=oss,alsa,sdl,pa --enable-virtfs \
+ --enable-vnc --enable-vnc-sasl --enable-vnc-jpeg --enable-vnc-png \
+ --enable-xen --enable-brlapi \
+ --enable-linux-aio --enable-attr \
+ --enable-cap-ng --enable-trace-backends=log --enable-spice --enable-rbd \
+ --enable-xfsctl --enable-libusb --enable-usb-redir \
+ --enable-libiscsi --enable-libnfs --enable-seccomp \
+ --enable-tpm --enable-libssh --enable-lzo --enable-snappy --enable-bzip2 \
+ --enable-numa --enable-rdma --enable-smartcard --enable-virglrenderer \
+ --enable-mpath --enable-libxml2 --enable-glusterfs \
+ --enable-virtfs --enable-zstd
+
+echo "Making libqemustub.a..."
+make libqemustub.a
+
+echo "Running cov-build..."
+rm -rf cov-int
+mkdir cov-int
+cov-build --dir cov-int $COVERITY_BUILD_CMD
+
+echo "Creating results tarball..."
+tar cvf - cov-int | xz > "$TARBALL"
+
+if [ ! -z "$RESULTSTARBALL" ]; then
+ echo "Copying results tarball to $RESULTSTARBALL..."
+ cp "$TARBALL" "$RESULTSTARBALL"
+fi
+
+echo "Uploading results tarball..."
+
+if [ "$DRYRUN" = yes ]; then
+ echo "Dry run only, not uploading $TARBALL"
+ exit 0
+fi
+
+curl --form token="$PROJTOKEN" --form email="$COVERITY_EMAIL" \
+ --form file=@"$TARBALL" --form version="$VERSION" \
+ --form description="$DESCRIPTION" \
+ https://scan.coverity.com/builds?project="$PROJNAME"
+
+echo "Done."
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 06/12] scripts/coverity-scan: Add Docker support
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (4 preceding siblings ...)
2020-04-14 16:26 ` [PULL 05/12] scripts/run-coverity-scan: Script to run Coverity Scan build Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 07/12] docs: Improve our gdbstub documentation Peter Maydell
` (6 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
Add support for running the Coverity Scan tools inside a Docker
container rather than directly on the host system.
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20200319193323.2038-7-peter.maydell@linaro.org
---
scripts/coverity-scan/coverity-scan.docker | 131 +++++++++++++++++++++
scripts/coverity-scan/run-coverity-scan | 90 ++++++++++++++
2 files changed, 221 insertions(+)
create mode 100644 scripts/coverity-scan/coverity-scan.docker
diff --git a/scripts/coverity-scan/coverity-scan.docker b/scripts/coverity-scan/coverity-scan.docker
new file mode 100644
index 00000000000..a4f64d12834
--- /dev/null
+++ b/scripts/coverity-scan/coverity-scan.docker
@@ -0,0 +1,131 @@
+# syntax=docker/dockerfile:1.0.0-experimental
+#
+# Docker setup for running the "Coverity Scan" tools over the source
+# tree and uploading them to the website, as per
+# https://scan.coverity.com/projects/qemu/builds/new
+# We do this on a fixed config (currently Fedora 30 with a known
+# set of dependencies and a configure command that enables a specific
+# set of options) so that random changes don't result in our accidentally
+# dropping some files from the scan.
+#
+# We don't build on top of the fedora.docker file because we don't
+# want to accidentally change or break the scan config when that
+# is updated.
+
+# The work of actually doing the build is handled by the
+# run-coverity-scan script.
+
+FROM fedora:30
+ENV PACKAGES \
+ alsa-lib-devel \
+ bc \
+ bison \
+ brlapi-devel \
+ bzip2 \
+ bzip2-devel \
+ ccache \
+ clang \
+ curl \
+ cyrus-sasl-devel \
+ dbus-daemon \
+ device-mapper-multipath-devel \
+ findutils \
+ flex \
+ gcc \
+ gcc-c++ \
+ gettext \
+ git \
+ glib2-devel \
+ glusterfs-api-devel \
+ gnutls-devel \
+ gtk3-devel \
+ hostname \
+ libaio-devel \
+ libasan \
+ libattr-devel \
+ libblockdev-mpath-devel \
+ libcap-devel \
+ libcap-ng-devel \
+ libcurl-devel \
+ libepoxy-devel \
+ libfdt-devel \
+ libgbm-devel \
+ libiscsi-devel \
+ libjpeg-devel \
+ libpmem-devel \
+ libnfs-devel \
+ libpng-devel \
+ librbd-devel \
+ libseccomp-devel \
+ libssh-devel \
+ libubsan \
+ libudev-devel \
+ libusbx-devel \
+ libxml2-devel \
+ libzstd-devel \
+ llvm \
+ lzo-devel \
+ make \
+ mingw32-bzip2 \
+ mingw32-curl \
+ mingw32-glib2 \
+ mingw32-gmp \
+ mingw32-gnutls \
+ mingw32-gtk3 \
+ mingw32-libjpeg-turbo \
+ mingw32-libpng \
+ mingw32-libtasn1 \
+ mingw32-nettle \
+ mingw32-nsis \
+ mingw32-pixman \
+ mingw32-pkg-config \
+ mingw32-SDL2 \
+ mingw64-bzip2 \
+ mingw64-curl \
+ mingw64-glib2 \
+ mingw64-gmp \
+ mingw64-gnutls \
+ mingw64-gtk3 \
+ mingw64-libjpeg-turbo \
+ mingw64-libpng \
+ mingw64-libtasn1 \
+ mingw64-nettle \
+ mingw64-pixman \
+ mingw64-pkg-config \
+ mingw64-SDL2 \
+ ncurses-devel \
+ nettle-devel \
+ nss-devel \
+ numactl-devel \
+ perl \
+ perl-Test-Harness \
+ pixman-devel \
+ pulseaudio-libs-devel \
+ python3 \
+ python3-sphinx \
+ PyYAML \
+ rdma-core-devel \
+ SDL2-devel \
+ snappy-devel \
+ sparse \
+ spice-server-devel \
+ systemd-devel \
+ systemtap-sdt-devel \
+ tar \
+ texinfo \
+ usbredir-devel \
+ virglrenderer-devel \
+ vte291-devel \
+ wget \
+ which \
+ xen-devel \
+ xfsprogs-devel \
+ zlib-devel
+ENV QEMU_CONFIGURE_OPTS --python=/usr/bin/python3
+
+RUN dnf install -y $PACKAGES
+RUN rpm -q $PACKAGES | sort > /packages.txt
+ENV PATH $PATH:/usr/libexec/python3-sphinx/
+ENV COVERITY_TOOL_BASE=/coverity-tools
+COPY run-coverity-scan run-coverity-scan
+RUN --mount=type=secret,id=coverity.token,required ./run-coverity-scan --update-tools-only --tokenfile /run/secrets/coverity.token
diff --git a/scripts/coverity-scan/run-coverity-scan b/scripts/coverity-scan/run-coverity-scan
index d40b51969fa..2e067ef5cfc 100755
--- a/scripts/coverity-scan/run-coverity-scan
+++ b/scripts/coverity-scan/run-coverity-scan
@@ -29,6 +29,7 @@
# Command line options:
# --dry-run : run the tools, but don't actually do the upload
+# --docker : create and work inside a docker container
# --update-tools-only : update the cached copy of the tools, but don't run them
# --tokenfile : file to read Coverity token from
# --version ver : specify version being analyzed (default: ask git)
@@ -36,6 +37,8 @@
# --srcdir : QEMU source tree to analyze (default: current working dir)
# --results-tarball : path to copy the results tarball to (default: don't
# copy it anywhere, just upload it)
+# --src-tarball : tarball to untar into src dir (default: none); this
+# is intended mainly for internal use by the Docker support
#
# User-specifiable environment variables:
# COVERITY_TOKEN -- Coverity token
@@ -125,6 +128,7 @@ update_coverity_tools () {
# Check user-provided environment variables and arguments
DRYRUN=no
UPDATE_ONLY=no
+DOCKER=no
while [ "$#" -ge 1 ]; do
case "$1" in
@@ -181,6 +185,19 @@ while [ "$#" -ge 1 ]; do
RESULTSTARBALL="$1"
shift
;;
+ --src-tarball)
+ shift
+ if [ $# -eq 0 ]; then
+ echo "--src-tarball needs an argument"
+ exit 1
+ fi
+ SRCTARBALL="$1"
+ shift
+ ;;
+ --docker)
+ DOCKER=yes
+ shift
+ ;;
*)
echo "Unexpected argument '$1'"
exit 1
@@ -212,6 +229,10 @@ PROJTOKEN="$COVERITY_TOKEN"
PROJNAME=QEMU
TARBALL=cov-int.tar.xz
+if [ "$UPDATE_ONLY" = yes ] && [ "$DOCKER" = yes ]; then
+ echo "Combining --docker and --update-only is not supported"
+ exit 1
+fi
if [ "$UPDATE_ONLY" = yes ]; then
# Just do the tools update; we don't need to check whether
@@ -221,8 +242,17 @@ if [ "$UPDATE_ONLY" = yes ]; then
exit 0
fi
+if [ ! -e "$SRCDIR" ]; then
+ mkdir "$SRCDIR"
+fi
+
cd "$SRCDIR"
+if [ ! -z "$SRCTARBALL" ]; then
+ echo "Untarring source tarball into $SRCDIR..."
+ tar xvf "$SRCTARBALL"
+fi
+
echo "Checking this is a QEMU source tree..."
if ! [ -e "$SRCDIR/VERSION" ]; then
echo "Not in a QEMU source tree?"
@@ -242,6 +272,66 @@ if [ -z "$COVERITY_EMAIL" ]; then
COVERITY_EMAIL="$(git config user.email)"
fi
+# Run ourselves inside docker if that's what the user wants
+if [ "$DOCKER" = yes ]; then
+ # build docker container including the coverity-scan tools
+ # Put the Coverity token into a temporary file that only
+ # we have read access to, and then pass it to docker build
+ # using --secret. This requires at least Docker 18.09.
+ # Mostly what we are trying to do here is ensure we don't leak
+ # the token into the Docker image.
+ umask 077
+ SECRETDIR=$(mktemp -d)
+ if [ -z "$SECRETDIR" ]; then
+ echo "Failed to create temporary directory"
+ exit 1
+ fi
+ trap 'rm -rf "$SECRETDIR"' INT TERM EXIT
+ echo "Created temporary directory $SECRETDIR"
+ SECRET="$SECRETDIR/token"
+ echo "$COVERITY_TOKEN" > "$SECRET"
+ echo "Building docker container..."
+ # TODO: This re-downloads the tools every time, rather than
+ # caching and reusing the image produced with the downloaded tools.
+ # Not sure why.
+ # TODO: how do you get 'docker build' to print the output of the
+ # commands it is running to its stdout? This would be useful for debug.
+ DOCKER_BUILDKIT=1 docker build -t coverity-scanner \
+ --secret id=coverity.token,src="$SECRET" \
+ -f scripts/coverity-scan/coverity-scan.docker \
+ scripts/coverity-scan
+ echo "Archiving sources to be analyzed..."
+ ./scripts/archive-source.sh "$SECRETDIR/qemu-sources.tgz"
+ if [ "$DRYRUN" = yes ]; then
+ DRYRUNARG=--dry-run
+ fi
+ echo "Running scanner..."
+ # If we need to capture the output tarball, get the inner run to
+ # save it to the secrets directory so we can copy it out before the
+ # directory is cleaned up.
+ if [ ! -z "$RESULTSTARBALL" ]; then
+ RTARGS="--results-tarball /work/cov-int.tar.xz"
+ else
+ RTARGS=""
+ fi
+ # Arrange for this docker run to get access to the sources with -v.
+ # We pass through all the configuration from the outer script to the inner.
+ export COVERITY_EMAIL COVERITY_BUILD_CMD
+ docker run -it --env COVERITY_EMAIL --env COVERITY_BUILD_CMD \
+ -v "$SECRETDIR:/work" coverity-scanner \
+ ./run-coverity-scan --version "$VERSION" \
+ --description "$DESCRIPTION" $DRYRUNARG --tokenfile /work/token \
+ --srcdir /qemu --src-tarball /work/qemu-sources.tgz $RTARGS
+ if [ ! -z "$RESULTSTARBALL" ]; then
+ echo "Copying results tarball to $RESULTSTARBALL..."
+ cp "$SECRETDIR/cov-int.tar.xz" "$RESULTSTARBALL"
+ fi
+ echo "Docker work complete."
+ exit 0
+fi
+
+# Otherwise, continue with the full build and upload process.
+
check_upload_permissions
update_coverity_tools
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 07/12] docs: Improve our gdbstub documentation
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (5 preceding siblings ...)
2020-04-14 16:26 ` [PULL 06/12] scripts/coverity-scan: Add Docker support Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 08/12] configure: Honour --disable-werror for Sphinx Peter Maydell
` (5 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
The documentation of our -s and -gdb options is quite old; in
particular it still claims that it will cause QEMU to stop and wait
for the gdb connection, when this has not been true for some time:
you also need to pass -S if you want to make QEMU not launch the
guest on startup.
Improve the documentation to mention this requirement in the
executable's --help output, the documentation of the -gdb option in
the manual, and in the "GDB usage" chapter.
Includes some minor tweaks to these paragraphs of documentation
since I was editing them anyway (such as dropping the description
of our gdb support as "primitive").
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Message-id: 20200403094014.9589-1-peter.maydell@linaro.org
---
docs/system/gdb.rst | 22 +++++++++++++++-------
qemu-options.hx | 24 ++++++++++++++++++------
2 files changed, 33 insertions(+), 13 deletions(-)
diff --git a/docs/system/gdb.rst b/docs/system/gdb.rst
index 639f814b32d..a40145fcf84 100644
--- a/docs/system/gdb.rst
+++ b/docs/system/gdb.rst
@@ -3,17 +3,25 @@
GDB usage
---------
-QEMU has a primitive support to work with gdb, so that you can do
-'Ctrl-C' while the virtual machine is running and inspect its state.
+QEMU supports working with gdb via gdb's remote-connection facility
+(the "gdbstub"). This allows you to debug guest code in the same
+way that you might with a low-level debug facility like JTAG
+on real hardware. You can stop and start the virtual machine,
+examine state like registers and memory, and set breakpoints and
+watchpoints.
-In order to use gdb, launch QEMU with the '-s' option. It will wait for
-a gdb connection:
+In order to use gdb, launch QEMU with the ``-s`` and ``-S`` options.
+The ``-s`` option will make QEMU listen for an incoming connection
+from gdb on TCP port 1234, and ``-S`` will make QEMU not start the
+guest until you tell it to from gdb. (If you want to specify which
+TCP port to use or to use something other than TCP for the gdbstub
+connection, use the ``-gdb dev`` option instead of ``-s``.)
.. parsed-literal::
- |qemu_system| -s -kernel bzImage -hda rootdisk.img -append "root=/dev/hda"
- Connected to host network interface: tun0
- Waiting gdb connection on port 1234
+ |qemu_system| -s -S -kernel bzImage -hda rootdisk.img -append "root=/dev/hda"
+
+QEMU will launch but will silently wait for gdb to connect.
Then launch gdb on the 'vmlinux' executable::
diff --git a/qemu-options.hx b/qemu-options.hx
index 16debd03cb6..292d4e7c0ce 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -3680,14 +3680,26 @@ SRST
ERST
DEF("gdb", HAS_ARG, QEMU_OPTION_gdb, \
- "-gdb dev wait for gdb connection on 'dev'\n", QEMU_ARCH_ALL)
+ "-gdb dev accept gdb connection on 'dev'. (QEMU defaults to starting\n"
+ " the guest without waiting for gdb to connect; use -S too\n"
+ " if you want it to not start execution.)\n",
+ QEMU_ARCH_ALL)
SRST
``-gdb dev``
- Wait for gdb connection on device dev (see
- :ref:`gdb_005fusage`). Typical connections will likely be
- TCP-based, but also UDP, pseudo TTY, or even stdio are reasonable
- use case. The latter is allowing to start QEMU from within gdb and
- establish the connection via a pipe:
+ Accept a gdb connection on device dev (see
+ :ref:`gdb_005fusage`). Note that this option does not pause QEMU
+ execution -- if you want QEMU to not start the guest until you
+ connect with gdb and issue a ``continue`` command, you will need to
+ also pass the ``-S`` option to QEMU.
+
+ The most usual configuration is to listen on a local TCP socket::
+
+ -gdb tcp::3117
+
+ but you can specify other backends; UDP, pseudo TTY, or even stdio
+ are all reasonable use cases. For example, a stdio connection
+ allows you to start QEMU from within gdb and establish the
+ connection via a pipe:
.. parsed-literal::
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 08/12] configure: Honour --disable-werror for Sphinx
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (6 preceding siblings ...)
2020-04-14 16:26 ` [PULL 07/12] docs: Improve our gdbstub documentation Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 09/12] scripts/kernel-doc: Add missing close-paren in c:function directives Peter Maydell
` (4 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
If we are not making warnings fatal for compilation, make them
non-fatal when building the Sphinx documentation also. (For instance
Sphinx 3.0 warns about some constructs that older versions were happy
with, which is a build failure if we use the warnings-as-errors
flag.)
This provides a workaround at least for LP:1872113.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20200411182934.28678-2-peter.maydell@linaro.org
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
---
configure | 9 ++++++++-
Makefile | 2 +-
2 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/configure b/configure
index 233c671aaa9..9b1f5b33e45 100755
--- a/configure
+++ b/configure
@@ -4928,6 +4928,12 @@ if check_include sys/kcov.h ; then
kcov=yes
fi
+# If we're making warnings fatal, apply this to Sphinx runs as well
+sphinx_werror=""
+if test "$werror" = "yes"; then
+ sphinx_werror="-W"
+fi
+
# Check we have a new enough version of sphinx-build
has_sphinx_build() {
# This is a bit awkward but works: create a trivial document and
@@ -4936,7 +4942,7 @@ has_sphinx_build() {
# sphinx-build doesn't exist at all or if it is too old.
mkdir -p "$TMPDIR1/sphinx"
touch "$TMPDIR1/sphinx/index.rst"
- "$sphinx_build" -c "$source_path/docs" -b html "$TMPDIR1/sphinx" "$TMPDIR1/sphinx/out" >/dev/null 2>&1
+ "$sphinx_build" $sphinx_werror -c "$source_path/docs" -b html "$TMPDIR1/sphinx" "$TMPDIR1/sphinx/out" >/dev/null 2>&1
}
# Check if tools are available to build documentation.
@@ -7631,6 +7637,7 @@ echo "INSTALL_PROG=$install -c -m 0755" >> $config_host_mak
echo "INSTALL_LIB=$install -c -m 0644" >> $config_host_mak
echo "PYTHON=$python" >> $config_host_mak
echo "SPHINX_BUILD=$sphinx_build" >> $config_host_mak
+echo "SPHINX_WERROR=$sphinx_werror" >> $config_host_mak
echo "GENISOIMAGE=$genisoimage" >> $config_host_mak
echo "CC=$cc" >> $config_host_mak
if $iasl -h > /dev/null 2>&1; then
diff --git a/Makefile b/Makefile
index 84ef8816000..8a9113e6663 100644
--- a/Makefile
+++ b/Makefile
@@ -1076,7 +1076,7 @@ sphinxdocs: $(MANUAL_BUILDDIR)/devel/index.html \
# Note the use of different doctree for each (manual, builder) tuple;
# this works around Sphinx not handling parallel invocation on
# a single doctree: https://github.com/sphinx-doc/sphinx/issues/2946
-build-manual = $(call quiet-command,CONFDIR="$(qemu_confdir)" $(SPHINX_BUILD) $(if $(V),,-q) -W -b $2 -D version=$(VERSION) -D release="$(FULL_VERSION)" -d .doctrees/$1-$2 $(SRC_PATH)/docs/$1 $(MANUAL_BUILDDIR)/$1 ,"SPHINX","$(MANUAL_BUILDDIR)/$1")
+build-manual = $(call quiet-command,CONFDIR="$(qemu_confdir)" $(SPHINX_BUILD) $(if $(V),,-q) $(SPHINX_WERROR) -b $2 -D version=$(VERSION) -D release="$(FULL_VERSION)" -d .doctrees/$1-$2 $(SRC_PATH)/docs/$1 $(MANUAL_BUILDDIR)/$1 ,"SPHINX","$(MANUAL_BUILDDIR)/$1")
# We assume all RST files in the manual's directory are used in it
manual-deps = $(wildcard $(SRC_PATH)/docs/$1/*.rst $(SRC_PATH)/docs/$1/*/*.rst) \
$(SRC_PATH)/docs/defs.rst.inc \
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 09/12] scripts/kernel-doc: Add missing close-paren in c:function directives
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (7 preceding siblings ...)
2020-04-14 16:26 ` [PULL 08/12] configure: Honour --disable-werror for Sphinx Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 10/12] kernel-doc: Use c:struct for Sphinx 3.0 and later Peter Maydell
` (3 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
When kernel-doc generates a 'c:function' directive for a function
one of whose arguments is a function pointer, it fails to print
the close-paren after the argument list of the function pointer
argument, for instance in the memory API documentation:
.. c:function:: void memory_region_init_resizeable_ram (MemoryRegion * mr, struct Object * owner, const char * name, uint64_t size, uint64_t max_size, void (*resized) (const char*, uint64_t length, void *host, Error ** errp)
which should have a ')' after the 'void *host' which is the
last argument to 'resized'.
Older versions of Sphinx don't try to parse the argumnet
to c:function, but Sphinx 3.0 does do this and will complain:
/home/petmay01/linaro/qemu-from-laptop/qemu/docs/../include/exec/memory.h:834: WARNING: Error in declarator or parameters
Invalid C declaration: Expecting "," or ")" in parameters, got "EOF". [error at 208]
void memory_region_init_resizeable_ram (MemoryRegion * mr, struct Object * owner, const char * name, uint64_t size, uint64_t max_size, void (*resized) (const char*, uint64_t length, void *host, Error ** errp)
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------^
Add the missing close-paren.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20200411182934.28678-3-peter.maydell@linaro.org
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
---
scripts/kernel-doc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/kernel-doc b/scripts/kernel-doc
index af470eb3211..8dc30e01e58 100755
--- a/scripts/kernel-doc
+++ b/scripts/kernel-doc
@@ -853,7 +853,7 @@ sub output_function_rst(%) {
if ($type =~ m/([^\(]*\(\*)\s*\)\s*\(([^\)]*)\)/) {
# pointer-to-function
- print $1 . $parameter . ") (" . $2;
+ print $1 . $parameter . ") (" . $2 . ")";
} else {
print $type . " " . $parameter;
}
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 10/12] kernel-doc: Use c:struct for Sphinx 3.0 and later
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (8 preceding siblings ...)
2020-04-14 16:26 ` [PULL 09/12] scripts/kernel-doc: Add missing close-paren in c:function directives Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 11/12] docs: Require Sphinx 1.6 or better Peter Maydell
` (2 subsequent siblings)
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
The kernel-doc Sphinx plugin and associated script currently emit
'c:type' directives for "struct foo" documentation.
Sphinx 3.0 warns about this:
/home/petmay01/linaro/qemu-from-laptop/qemu/docs/../include/exec/memory.h:3: WARNING: Type must be either just a name or a typedef-like declaration.
If just a name:
Error in declarator or parameters
Invalid C declaration: Expected identifier in nested name, got keyword: struct [error at 6]
struct MemoryListener
------^
If typedef-like declaration:
Error in declarator or parameters
Invalid C declaration: Expected identifier in nested name. [error at 21]
struct MemoryListener
---------------------^
because it wants us to use the new-in-3.0 'c:struct' instead.
Plumb the Sphinx version through to the kernel-doc script
and use it to select 'c:struct' for newer versions than 3.0.
Fixes: LP:1872113
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
---
docs/sphinx/kerneldoc.py | 1 +
scripts/kernel-doc | 16 +++++++++++++++-
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/docs/sphinx/kerneldoc.py b/docs/sphinx/kerneldoc.py
index 1159405cb92..3e879402064 100644
--- a/docs/sphinx/kerneldoc.py
+++ b/docs/sphinx/kerneldoc.py
@@ -99,6 +99,7 @@ class KernelDocDirective(Directive):
env.note_dependency(os.path.abspath(f))
cmd += ['-export-file', f]
+ cmd += ['-sphinx-version', sphinx.__version__]
cmd += [filename]
try:
diff --git a/scripts/kernel-doc b/scripts/kernel-doc
index 8dc30e01e58..030b5c8691f 100755
--- a/scripts/kernel-doc
+++ b/scripts/kernel-doc
@@ -71,6 +71,8 @@ Output selection (mutually exclusive):
DOC: sections. May be specified multiple times.
Output selection modifiers:
+ -sphinx-version VER Generate rST syntax for the specified Sphinx version.
+ Only works with reStructuredTextFormat.
-no-doc-sections Do not output DOC: sections.
-enable-lineno Enable output of #define LINENO lines. Only works with
reStructuredText format.
@@ -286,6 +288,7 @@ use constant {
};
my $output_selection = OUTPUT_ALL;
my $show_not_found = 0; # No longer used
+my $sphinx_version = "0.0"; # if not specified, assume old
my @export_file_list;
@@ -436,6 +439,8 @@ while ($ARGV[0] =~ m/^--?(.*)/) {
$enable_lineno = 1;
} elsif ($cmd eq 'show-not-found') {
$show_not_found = 1; # A no-op but don't fail
+ } elsif ($cmd eq 'sphinx-version') {
+ $sphinx_version = shift @ARGV;
} else {
# Unknown argument
usage();
@@ -963,7 +968,16 @@ sub output_struct_rst(%) {
my $oldprefix = $lineprefix;
my $name = $args{'type'} . " " . $args{'struct'};
- print "\n\n.. c:type:: " . $name . "\n\n";
+ # Sphinx 3.0 and up will emit warnings for "c:type:: struct Foo".
+ # It wants to see "c:struct:: Foo" (and will add the word 'struct' in
+ # the rendered output).
+ if ((split(/\./, $sphinx_version))[0] >= 3) {
+ my $sname = $name;
+ $sname =~ s/^struct //;
+ print "\n\n.. c:struct:: " . $sname . "\n\n";
+ } else {
+ print "\n\n.. c:type:: " . $name . "\n\n";
+ }
print_lineno($declaration_start_line);
$lineprefix = " ";
output_highlight_rst($args{'purpose'});
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 11/12] docs: Require Sphinx 1.6 or better
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (9 preceding siblings ...)
2020-04-14 16:26 ` [PULL 10/12] kernel-doc: Use c:struct for Sphinx 3.0 and later Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 16:26 ` [PULL 12/12] Deprecate KVM support for AArch32 Peter Maydell
2020-04-14 19:09 ` [PULL 00/12] target-arm queue Peter Maydell
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
Versions of Sphinx older than 1.6 can't build all of our documentation,
because they are too picky about the syntax of the argument to the
option:: directive; see Sphinx bugs #646, #3366:
https://github.com/sphinx-doc/sphinx/issues/646
https://github.com/sphinx-doc/sphinx/issues/3366
Trying to build with a 1.4.x Sphinx fails with
docs/system/images.rst:4: SEVERE: Duplicate ID: "cmdoption-qcow2-arg-encrypt"
and a 1.5.x Sphinx fails with
docs/system/invocation.rst:544: WARNING: Malformed option description '[enable=]PATTERN', should look like "opt", "-opt
args", "--opt args", "/opt args" or "+opt args"
Update our needs_sphinx setting to indicate that we require at least
1.6. This will allow configure to fall back to "don't build the
docs" rather than causing the build to fail entirely, which is
probably what most users building on a host old enough to have such
an old Sphinx would want; if they do want the docs then they'll have
a useful indication of what they need to do (upgrade Sphinx!) rather
than a confusing error message.
In theory our distro support policy would suggest that we should
support building on the Sphinx shipped in those distros, but:
* EPEL7 has Sphinx 1.2.3 (which we've never supported!)
* Debian Stretch has Sphinx 1.4.8
Trying to get our docs to work with Sphinx 1.4 is not tractable
for the 5.0 release and I'm not sure it's worthwhile effort anyway;
at least with this change the build as a whole now succeeds.
Thanks to John Snow for doing the investigation and testing to
confirm what Sphinx versions fail in what ways and what distros
shipped what.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
---
docs/conf.py | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/docs/conf.py b/docs/conf.py
index 7768611e89c..d6e173ef77b 100644
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -59,8 +59,10 @@ sys.path.insert(0, os.path.join(qemu_docdir, "sphinx"))
# If your documentation needs a minimal Sphinx version, state it here.
#
-# 1.3 is where the 'alabaster' theme was shipped with Sphinx.
-needs_sphinx = '1.3'
+# Sphinx 1.5 and earlier can't build our docs because they are too
+# picky about the syntax of the argument to the option:: directive
+# (see Sphinx bugs #646, #3366).
+needs_sphinx = '1.6'
# Add any Sphinx extension module names here, as strings. They can be
# extensions coming with Sphinx (named 'sphinx.ext.*') or your custom
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* [PULL 12/12] Deprecate KVM support for AArch32
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (10 preceding siblings ...)
2020-04-14 16:26 ` [PULL 11/12] docs: Require Sphinx 1.6 or better Peter Maydell
@ 2020-04-14 16:26 ` Peter Maydell
2020-04-14 19:09 ` [PULL 00/12] target-arm queue Peter Maydell
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 16:26 UTC (permalink / raw)
To: qemu-devel
The Linux kernel has dropped support for allowing 32-bit Arm systems
to host KVM guests (kernel commit 541ad0150ca4aa663a2, which just
landed upstream in the 5.7 merge window). Mark QEMU's support for
this configuration as deprecated, so that we can delete that support
code in 5.2.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Andrew Jones <drjones@redhat.com>
---
docs/system/deprecated.rst | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/docs/system/deprecated.rst b/docs/system/deprecated.rst
index c633fe2beff..3142fac3865 100644
--- a/docs/system/deprecated.rst
+++ b/docs/system/deprecated.rst
@@ -336,6 +336,14 @@ The ``compat`` property used to set backwards compatibility modes for
the processor has been deprecated. The ``max-cpu-compat`` property of
the ``pseries`` machine type should be used instead.
+KVM guest support on 32-bit Arm hosts (since 5.0)
+'''''''''''''''''''''''''''''''''''''''''''''''''
+
+The Linux kernel has dropped support for allowing 32-bit Arm systems
+to host KVM guests as of the 5.7 kernel. Accordingly, QEMU is deprecating
+its support for this configuration and will remove it in a future version.
+Running 32-bit guests on a 64-bit Arm host remains supported.
+
System emulator devices
-----------------------
--
2.20.1
^ permalink raw reply related [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
` (11 preceding siblings ...)
2020-04-14 16:26 ` [PULL 12/12] Deprecate KVM support for AArch32 Peter Maydell
@ 2020-04-14 19:09 ` Peter Maydell
12 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-04-14 19:09 UTC (permalink / raw)
To: QEMU Developers
On Tue, 14 Apr 2020 at 17:26, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> Almost nothing in here is arm-related, but the target-arm
> queue was convenient for these last minute bits and pieces
> for 5.0...
>
> thanks
> -- PMM
>
> The following changes since commit 14e5526b51910efd62cd31cd95b49baca975c83f:
>
> Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging (2020-04-13 15:42:51 +0100)
>
> are available in the Git repository at:
>
> https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20200414
>
> for you to fetch changes up to 84f82ddcbb4ac4ed04c8675e85155329f23184f0:
>
> Deprecate KVM support for AArch32 (2020-04-14 17:20:22 +0100)
>
> ----------------------------------------------------------------
> patch queue:
> * Fix some problems that trip up Coverity's scanner
> * run-coverity-scan: New script automating the scan-and-upload process
> * docs: Improve our gdbstub documentation
> * configure: Honour --disable-werror for Sphinx
> * docs: Fix errors produced when building with Sphinx 3.0
> * docs: Require Sphinx 1.6 or better
> * Add deprecation notice for KVM support on AArch32 hosts
>
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/5.0
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2020-07-20 12:56 Peter Maydell
2020-07-20 21:24 ` Peter Maydell
0 siblings, 1 reply; 27+ messages in thread
From: Peter Maydell @ 2020-07-20 12:56 UTC (permalink / raw)
To: qemu-devel
Not much here, mostly documentation, but a few bug fixes.
thanks
-- PMM
The following changes since commit 873ec69aeb12e24eec7fb317fd0cd8494e8489dd:
Merge remote-tracking branch 'remotes/cminyard/tags/for-qemu-i2c-5' into staging (2020-07-20 11:03:09 +0100)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20200720
for you to fetch changes up to 6a0b7505f1fd6769c3f1558fda76464d51e4118a:
docs/system: Document the arm virt board (2020-07-20 11:35:17 +0100)
----------------------------------------------------------------
target-arm queue:
* virt: Don't enable MTE emulation by default
* virt: Diagnose attempts to use MTE with memory-hotplug or KVM
(rather than silently not working correctly)
* util: Implement qemu_get_thread_id() for OpenBSD
* qdev: Add doc comments for qdev_unrealize and GPIO functions,
and standardize on doc-comments-in-header-file
* hw/arm/armsse: Assert info->num_cpus is in-bounds in armsse_realize()
* docs/system: Document canon-a1100, collie, gumstix, virt boards
----------------------------------------------------------------
David CARLIER (1):
util: Implement qemu_get_thread_id() for OpenBSD
Peter Maydell (8):
qdev: Move doc comments from qdev.c to qdev-core.h
qdev: Document qdev_unrealize()
qdev: Document GPIO related functions
hw/arm/armsse: Assert info->num_cpus is in-bounds in armsse_realize()
docs/system: Briefly document canon-a1100 board
docs/system: Briefly document collie board
docs/system: Briefly document gumstix boards
docs/system: Document the arm virt board
Richard Henderson (3):
hw/arm/virt: Enable MTE via a machine property
hw/arm/virt: Error for MTE enabled with KVM
hw/arm/virt: Disable memory hotplug when MTE is enabled
docs/system/arm/collie.rst | 16 +++
docs/system/arm/digic.rst | 11 ++
docs/system/arm/gumstix.rst | 21 ++++
docs/system/arm/virt.rst | 161 ++++++++++++++++++++++++++
docs/system/target-arm.rst | 4 +
include/hw/arm/virt.h | 1 +
include/hw/qdev-core.h | 267 ++++++++++++++++++++++++++++++++++++++++++-
include/hw/qdev-properties.h | 13 +++
hw/arm/armsse.c | 2 +
hw/arm/virt.c | 50 +++++++-
hw/core/qdev.c | 33 ------
target/arm/cpu.c | 19 +--
target/arm/cpu64.c | 5 +-
util/oslib-posix.c | 2 +
MAINTAINERS | 4 +
15 files changed, 559 insertions(+), 50 deletions(-)
create mode 100644 docs/system/arm/collie.rst
create mode 100644 docs/system/arm/digic.rst
create mode 100644 docs/system/arm/gumstix.rst
create mode 100644 docs/system/arm/virt.rst
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2020-07-20 12:56 Peter Maydell
@ 2020-07-20 21:24 ` Peter Maydell
0 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-07-20 21:24 UTC (permalink / raw)
To: QEMU Developers
On Mon, 20 Jul 2020 at 13:56, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> Not much here, mostly documentation, but a few bug fixes.
>
> thanks
> -- PMM
>
> The following changes since commit 873ec69aeb12e24eec7fb317fd0cd8494e8489dd:
>
> Merge remote-tracking branch 'remotes/cminyard/tags/for-qemu-i2c-5' into staging (2020-07-20 11:03:09 +0100)
>
> are available in the Git repository at:
>
> https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20200720
>
> for you to fetch changes up to 6a0b7505f1fd6769c3f1558fda76464d51e4118a:
>
> docs/system: Document the arm virt board (2020-07-20 11:35:17 +0100)
>
> ----------------------------------------------------------------
> target-arm queue:
> * virt: Don't enable MTE emulation by default
> * virt: Diagnose attempts to use MTE with memory-hotplug or KVM
> (rather than silently not working correctly)
> * util: Implement qemu_get_thread_id() for OpenBSD
> * qdev: Add doc comments for qdev_unrealize and GPIO functions,
> and standardize on doc-comments-in-header-file
> * hw/arm/armsse: Assert info->num_cpus is in-bounds in armsse_realize()
> * docs/system: Document canon-a1100, collie, gumstix, virt boards
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2020-10-08 14:25 Peter Maydell
0 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2020-10-08 14:25 UTC (permalink / raw)
To: qemu-devel
The following changes since commit 6eeea6725a70e6fcb5abba0764496bdab07ddfb3:
Merge remote-tracking branch 'remotes/huth-gitlab/tags/pull-request-2020-10-06' into staging (2020-10-06 21:13:34 +0100)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20201008
for you to fetch changes up to ba118c26e16a97e6ff6de8184057d3420ce16a23:
target/arm: Make '-cpu max' have a 48-bit PA (2020-10-08 15:24:32 +0100)
----------------------------------------------------------------
target-arm queue:
* hw/ssi/npcm7xx_fiu: Fix handling of unsigned integer
* hw/arm/fsl-imx25: Fix a typo
* hw/arm/sbsa-ref : Fix SMMUv3 Initialisation
* hw/arm/sbsa-ref : allocate IRQs for SMMUv3
* hw/char/bcm2835_aux: Allow less than 32-bit accesses
* hw/arm/virt: Implement kvm-steal-time
* target/arm: Make '-cpu max' have a 48-bit PA
----------------------------------------------------------------
Andrew Jones (6):
linux headers: sync to 5.9-rc7
target/arm/kvm: Make uncalled stubs explicitly unreachable
hw/arm/virt: Move post cpu realize check into its own function
hw/arm/virt: Move kvm pmu setup to virt_cpu_post_init
tests/qtest: Restore aarch64 arm-cpu-features test
hw/arm/virt: Implement kvm-steal-time
Graeme Gregory (2):
hw/arm/sbsa-ref : Fix SMMUv3 Initialisation
hw/arm/sbsa-ref : allocate IRQs for SMMUv3
Peter Maydell (1):
target/arm: Make '-cpu max' have a 48-bit PA
Philippe Mathieu-Daudé (3):
hw/ssi/npcm7xx_fiu: Fix handling of unsigned integer
hw/arm/fsl-imx25: Fix a typo
hw/char/bcm2835_aux: Allow less than 32-bit accesses
docs/system/arm/cpu-features.rst | 11 ++++
include/hw/arm/fsl-imx25.h | 2 +-
include/hw/arm/virt.h | 5 ++
linux-headers/linux/kvm.h | 6 ++-
target/arm/cpu.h | 4 ++
target/arm/kvm_arm.h | 94 ++++++++++++++++++++++++++-------
hw/arm/sbsa-ref.c | 3 +-
hw/arm/virt.c | 110 ++++++++++++++++++++++++++++-----------
hw/char/bcm2835_aux.c | 4 +-
hw/ssi/npcm7xx_fiu.c | 12 ++---
target/arm/cpu.c | 8 +++
target/arm/cpu64.c | 4 ++
target/arm/kvm.c | 16 ++++++
target/arm/kvm64.c | 64 +++++++++++++++++++++--
target/arm/monitor.c | 2 +-
tests/qtest/arm-cpu-features.c | 25 +++++++--
hw/ssi/trace-events | 2 +-
tests/qtest/meson.build | 3 +-
18 files changed, 303 insertions(+), 72 deletions(-)
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2021-07-18 12:46 Peter Maydell
2021-07-18 16:35 ` Peter Maydell
0 siblings, 1 reply; 27+ messages in thread
From: Peter Maydell @ 2021-07-18 12:46 UTC (permalink / raw)
To: qemu-devel
Last few changes before rc0: a few bug fixes, but mostly
docs stuff.
-- PMM
The following changes since commit a97fca4ceb9d9b10aa8b582e817a5ee6c42ffbaf:
Merge remote-tracking branch 'remotes/mst/tags/for_upstream3' into staging (2021-07-16 16:34:42 +0100)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20210718
for you to fetch changes up to 8fe612a183dec4c63afdc57537079bc742d024ca:
target/arm: Remove duplicate 'plus1' function from Neon and SVE decode (2021-07-18 10:59:47 +0100)
----------------------------------------------------------------
target-arm queue:
* Remove duplicate 'plus1' function from Neon and SVE decode
* Fix offsets for TTBCR for big-endian hosts
* docs: fix copyright date
* docs: add license/version info to HTML footers
* docs: add an About section
* docs: document some more arm boards
----------------------------------------------------------------
Peter Maydell (11):
docs: Fix documentation Copyright date
docs: Stop calling the top level subsections of our manual 'manuals'
docs: Remove "Contents:" lines from top-level subsections
docs: Move deprecation, build and license info out of system/
docs: Add some actual About text to about/index.rst
docs: Add license note to the HTML page footer
docs: Add QEMU version information to HTML footer
docs: Add skeletal documentation of cubieboard
docs: Add skeletal documentation of the emcraft-sf2
docs: Add skeletal documentation of highbank and midway
target/arm: Remove duplicate 'plus1' function from Neon and SVE decode
Richard Henderson (1):
target/arm: Fix offsets for TTBCR
docs/_templates/footer.html | 14 ++++++++++++++
docs/{system => about}/build-platforms.rst | 0
docs/{system => about}/deprecated.rst | 0
docs/about/index.rst | 27 +++++++++++++++++++++++++++
docs/{system => about}/license.rst | 0
docs/{system => about}/removed-features.rst | 0
docs/conf.py | 2 +-
docs/devel/index.rst | 7 +------
docs/index.rst | 1 +
docs/interop/index.rst | 9 ++-------
docs/meson.build | 3 ++-
docs/specs/index.rst | 7 ++-----
docs/system/arm/cubieboard.rst | 16 ++++++++++++++++
docs/system/arm/emcraft-sf2.rst | 15 +++++++++++++++
docs/system/arm/highbank.rst | 19 +++++++++++++++++++
docs/system/index.rst | 11 +----------
docs/system/target-arm.rst | 3 +++
docs/tools/index.rst | 7 ++-----
docs/user/index.rst | 7 +------
target/arm/neon-ls.decode | 4 ++--
target/arm/neon-shared.decode | 2 +-
target/arm/sve.decode | 2 +-
target/arm/helper.c | 11 +++++++----
target/arm/translate-neon.c | 5 -----
target/arm/translate-sve.c | 5 -----
MAINTAINERS | 4 ++++
26 files changed, 122 insertions(+), 59 deletions(-)
create mode 100644 docs/_templates/footer.html
rename docs/{system => about}/build-platforms.rst (100%)
rename docs/{system => about}/deprecated.rst (100%)
create mode 100644 docs/about/index.rst
rename docs/{system => about}/license.rst (100%)
rename docs/{system => about}/removed-features.rst (100%)
create mode 100644 docs/system/arm/cubieboard.rst
create mode 100644 docs/system/arm/emcraft-sf2.rst
create mode 100644 docs/system/arm/highbank.rst
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2021-07-18 12:46 Peter Maydell
@ 2021-07-18 16:35 ` Peter Maydell
0 siblings, 0 replies; 27+ messages in thread
From: Peter Maydell @ 2021-07-18 16:35 UTC (permalink / raw)
To: QEMU Developers
On Sun, 18 Jul 2021 at 13:46, Peter Maydell <peter.maydell@linaro.org> wrote:
>
> Last few changes before rc0: a few bug fixes, but mostly
> docs stuff.
>
> -- PMM
>
> The following changes since commit a97fca4ceb9d9b10aa8b582e817a5ee6c42ffbaf:
>
> Merge remote-tracking branch 'remotes/mst/tags/for_upstream3' into staging (2021-07-16 16:34:42 +0100)
>
> are available in the Git repository at:
>
> https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20210718
>
> for you to fetch changes up to 8fe612a183dec4c63afdc57537079bc742d024ca:
>
> target/arm: Remove duplicate 'plus1' function from Neon and SVE decode (2021-07-18 10:59:47 +0100)
>
> ----------------------------------------------------------------
> target-arm queue:
> * Remove duplicate 'plus1' function from Neon and SVE decode
> * Fix offsets for TTBCR for big-endian hosts
> * docs: fix copyright date
> * docs: add license/version info to HTML footers
> * docs: add an About section
> * docs: document some more arm boards
>
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/6.1
for any user-visible changes.
-- PMM
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2023-05-12 15:34 Peter Maydell
2023-05-13 8:36 ` Richard Henderson
0 siblings, 1 reply; 27+ messages in thread
From: Peter Maydell @ 2023-05-12 15:34 UTC (permalink / raw)
To: qemu-devel
Hi; here's a relatively small target-arm queue, pretty much all
bug fixes. (There are a few non-arm patches that I've thrown in
there too for my convenience :-))
thanks
-- PMM
The following changes since commit 278238505d28d292927bff7683f39fb4fbca7fd1:
Merge tag 'pull-tcg-20230511-2' of https://gitlab.com/rth7680/qemu into staging (2023-05-11 11:44:23 +0100)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20230512
for you to fetch changes up to 478dccbb99db0bf8f00537dd0b4d0de88d5cb537:
target/arm: Correct AArch64.S2MinTxSZ 32-bit EL1 input size check (2023-05-12 16:01:25 +0100)
----------------------------------------------------------------
target-arm queue:
* More refactoring of files into tcg/
* Don't allow stage 2 page table walks to downgrade to NS
* Fix handling of SW and NSW bits for stage 2 walks
* MAINTAINERS: Update Akihiko Odaki's email address
* ui: Fix pixel colour channel order for PNG screenshots
* docs: Remove unused weirdly-named cross-reference targets
* hw/mips/malta: Fix minor dead code issue
* Fixes for the "allow CONFIG_TCG=n" changes
* tests/qtest: Don't run cdrom boot tests if no accelerator is present
* target/arm: Correct AArch64.S2MinTxSZ 32-bit EL1 input size check
----------------------------------------------------------------
Akihiko Odaki (1):
MAINTAINERS: Update Akihiko Odaki's email address
Fabiano Rosas (3):
target/arm: Select SEMIHOSTING when using TCG
target/arm: Select CONFIG_ARM_V7M when TCG is enabled
tests/qtest: Don't run cdrom boot tests if no accelerator is present
Peter Maydell (6):
target/arm: Don't allow stage 2 page table walks to downgrade to NS
target/arm: Fix handling of SW and NSW bits for stage 2 walks
ui: Fix pixel colour channel order for PNG screenshots
docs: Remove unused weirdly-named cross-reference targets
hw/mips/malta: Fix minor dead code issue
target/arm: Correct AArch64.S2MinTxSZ 32-bit EL1 input size check
Richard Henderson (2):
target/arm: Move translate-a32.h, arm_ldst.h, sve_ldst_internal.h to tcg/
target/arm: Move helper-{a64,mve,sme,sve}.h to tcg/
MAINTAINERS | 4 +-
docs/system/devices/igb.rst | 2 +-
docs/system/devices/ivshmem.rst | 2 -
docs/system/devices/net.rst | 2 +-
docs/system/devices/usb.rst | 2 -
docs/system/keys.rst | 2 +-
docs/system/linuxboot.rst | 2 +-
docs/system/target-i386.rst | 4 --
target/arm/helper.h | 8 +--
target/arm/internals.h | 12 +++-
target/arm/{ => tcg}/arm_ldst.h | 0
target/arm/{ => tcg}/helper-a64.h | 0
target/arm/{ => tcg}/helper-mve.h | 0
target/arm/{ => tcg}/helper-sme.h | 0
target/arm/{ => tcg}/helper-sve.h | 0
target/arm/{ => tcg}/sve_ldst_internal.h | 0
target/arm/{ => tcg}/translate-a32.h | 0
hw/mips/malta.c | 5 +-
target/arm/gdbstub64.c | 2 +-
target/arm/helper.c | 15 ++++-
target/arm/ptw.c | 95 +++++++++++++++++++-------------
target/arm/tcg/pauth_helper.c | 6 +-
tests/qtest/cdrom-test.c | 10 ++++
ui/console.c | 4 +-
target/arm/Kconfig | 9 +--
25 files changed, 109 insertions(+), 77 deletions(-)
rename target/arm/{ => tcg}/arm_ldst.h (100%)
rename target/arm/{ => tcg}/helper-a64.h (100%)
rename target/arm/{ => tcg}/helper-mve.h (100%)
rename target/arm/{ => tcg}/helper-sme.h (100%)
rename target/arm/{ => tcg}/helper-sve.h (100%)
rename target/arm/{ => tcg}/sve_ldst_internal.h (100%)
rename target/arm/{ => tcg}/translate-a32.h (100%)
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2023-05-12 15:34 Peter Maydell
@ 2023-05-13 8:36 ` Richard Henderson
0 siblings, 0 replies; 27+ messages in thread
From: Richard Henderson @ 2023-05-13 8:36 UTC (permalink / raw)
To: Peter Maydell, qemu-devel
On 5/12/23 16:34, Peter Maydell wrote:
> Hi; here's a relatively small target-arm queue, pretty much all
> bug fixes. (There are a few non-arm patches that I've thrown in
> there too for my convenience :-))
>
> thanks
> -- PMM
>
> The following changes since commit 278238505d28d292927bff7683f39fb4fbca7fd1:
>
> Merge tag 'pull-tcg-20230511-2' ofhttps://gitlab.com/rth7680/qemu into staging (2023-05-11 11:44:23 +0100)
>
> are available in the Git repository at:
>
> https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20230512
>
> for you to fetch changes up to 478dccbb99db0bf8f00537dd0b4d0de88d5cb537:
>
> target/arm: Correct AArch64.S2MinTxSZ 32-bit EL1 input size check (2023-05-12 16:01:25 +0100)
>
> ----------------------------------------------------------------
> target-arm queue:
> * More refactoring of files into tcg/
> * Don't allow stage 2 page table walks to downgrade to NS
> * Fix handling of SW and NSW bits for stage 2 walks
> * MAINTAINERS: Update Akihiko Odaki's email address
> * ui: Fix pixel colour channel order for PNG screenshots
> * docs: Remove unused weirdly-named cross-reference targets
> * hw/mips/malta: Fix minor dead code issue
> * Fixes for the "allow CONFIG_TCG=n" changes
> * tests/qtest: Don't run cdrom boot tests if no accelerator is present
> * target/arm: Correct AArch64.S2MinTxSZ 32-bit EL1 input size check
Applied, thanks. Please update https://wiki.qemu.org/ChangeLog/8.1 as appropriate.
r~
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2023-11-06 15:32 Peter Maydell
2023-11-07 3:02 ` Stefan Hajnoczi
0 siblings, 1 reply; 27+ messages in thread
From: Peter Maydell @ 2023-11-06 15:32 UTC (permalink / raw)
To: qemu-devel
Hi; here's another arm pullreq. These changes are all bug fixes
(including some Coverity issue fixes), so are OK for applying
either before or after softfreeze.
thanks
-- PMM
The following changes since commit 3e01f1147a16ca566694b97eafc941d62fa1e8d8:
Merge tag 'pull-sp-20231105' of https://gitlab.com/rth7680/qemu into staging (2023-11-06 09:34:22 +0800)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20231106
for you to fetch changes up to 5722fc471296d5f042df4b005a851cc8008df0c9:
target/arm: Fix A64 LDRA immediate decode (2023-11-06 15:00:29 +0000)
----------------------------------------------------------------
target-arm queue:
* hw/arm/virt: fix PMU IRQ registration
* hw/arm/virt: Report correct register sizes in ACPI DBG2/SPCR tables
* hw/i386/intel_iommu: vtd_slpte_nonzero_rsvd(): assert no overflow
* util/filemonitor-inotify: qemu_file_monitor_watch(): assert no overflow
* mc146818rtc: rtc_set_time(): initialize tm to zeroes
* block/nvme: nvme_process_completion() fix bound for cid
* hw/core/loader: gunzip(): initialize z_stream
* io/channel-socket: qio_channel_socket_flush(): improve msg validation
* hw/arm/vexpress-a9: Remove useless mapping of RAM at address 0
* target/arm: Fix A64 LDRA immediate decode
----------------------------------------------------------------
Peter Maydell (4):
tests/qtest/bios-tables-test: Allow changes to virt SPCR and DBG2
tests/qtest/bios-tables-test: Update virt SPCR and DBG2 golden references
hw/arm/vexpress-a9: Remove useless mapping of RAM at address 0
target/arm: Fix A64 LDRA immediate decode
Sebastian Ott (1):
hw/arm/virt: fix PMU IRQ registration
Udo Steinberg (1):
hw/arm/virt: Report correct register sizes in ACPI DBG2/SPCR tables.
Vladimir Sementsov-Ogievskiy (6):
hw/i386/intel_iommu: vtd_slpte_nonzero_rsvd(): assert no overflow
util/filemonitor-inotify: qemu_file_monitor_watch(): assert no overflow
mc146818rtc: rtc_set_time(): initialize tm to zeroes
block/nvme: nvme_process_completion() fix bound for cid
hw/core/loader: gunzip(): initialize z_stream
io/channel-socket: qio_channel_socket_flush(): improve msg validation
docs/system/arm/vexpress.rst | 3 +++
target/arm/tcg/translate.h | 5 +++++
target/arm/tcg/a64.decode | 2 +-
block/nvme.c | 7 ++++---
hw/arm/vexpress.c | 14 +++-----------
hw/arm/virt-acpi-build.c | 4 ++--
hw/arm/virt.c | 3 ++-
hw/core/loader.c | 2 +-
hw/i386/intel_iommu.c | 23 ++++++++++++++++++++---
hw/rtc/mc146818rtc.c | 2 +-
io/channel-socket.c | 5 +++++
util/filemonitor-inotify.c | 25 +++++++++++++++++--------
tests/data/acpi/virt/DBG2 | Bin 87 -> 87 bytes
tests/data/acpi/virt/SPCR | Bin 80 -> 80 bytes
14 files changed, 64 insertions(+), 31 deletions(-)
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2023-11-06 15:32 Peter Maydell
@ 2023-11-07 3:02 ` Stefan Hajnoczi
0 siblings, 0 replies; 27+ messages in thread
From: Stefan Hajnoczi @ 2023-11-07 3:02 UTC (permalink / raw)
To: Peter Maydell; +Cc: qemu-devel
[-- Attachment #1: Type: text/plain, Size: 115 bytes --]
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/8.2 for any user-visible changes.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PULL 00/12] target-arm queue
@ 2025-02-10 15:49 Peter Maydell
2025-02-11 2:37 ` Stefan Hajnoczi
0 siblings, 1 reply; 27+ messages in thread
From: Peter Maydell @ 2025-02-10 15:49 UTC (permalink / raw)
To: qemu-devel
The following changes since commit 131c58469f6fb68c89b38fee6aba8bbb20c7f4bf:
rust: add --rust-target option for bindgen (2025-02-06 13:51:46 -0500)
are available in the Git repository at:
https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20250210
for you to fetch changes up to 27a8d899c7a100fd5aa040a8b993bb257687c393:
linux-user: Do not define struct sched_attr if libc headers do (2025-02-07 16:09:20 +0000)
----------------------------------------------------------------
target-arm queue:
* Deprecate pxa2xx CPUs, iwMMXt emulation, -old-param option
* Drop unused AArch64DecodeTable typedefs
* Minor code cleanups
* hw/net/cadence_gem: Fix the mask/compare/disable-mask logic
* linux-user: Do not define struct sched_attr if libc headers do
----------------------------------------------------------------
Andrew Yuan (1):
hw/net/cadence_gem: Fix the mask/compare/disable-mask logic
Khem Raj (1):
linux-user: Do not define struct sched_attr if libc headers do
Peter Maydell (4):
target/arm: deprecate the pxa2xx CPUs and iwMMXt emulation
tests/tcg/arm: Remove test-arm-iwmmxt test
target/arm: Drop unused AArch64DecodeTable typedefs
qemu-options: Deprecate -old-param command line option
Philippe Mathieu-Daudé (6):
hw/arm/boot: Propagate vCPU to arm_load_dtb()
hw/arm/fsl-imx6: Add local 'mpcore/gic' variables
hw/arm/fsl-imx6ul: Add local 'mpcore/gic' variables
hw/arm/fsl-imx7: Add local 'mpcore/gic' variables
hw/cpu/arm: Alias 'num-cpu' property on TYPE_REALVIEW_MPCORE
hw/cpu/arm: Declare CPU QOM types using DEFINE_TYPES() macro
docs/about/deprecated.rst | 34 ++++++++++++++++++++++
include/hw/arm/boot.h | 4 ++-
target/arm/cpu.h | 1 +
hw/arm/boot.c | 11 +++----
hw/arm/fsl-imx6.c | 52 ++++++++++++++-------------------
hw/arm/fsl-imx6ul.c | 64 +++++++++++++++++------------------------
hw/arm/fsl-imx7.c | 52 +++++++++++++++------------------
hw/arm/virt.c | 2 +-
hw/cpu/a15mpcore.c | 21 ++++++--------
hw/cpu/a9mpcore.c | 21 ++++++--------
hw/cpu/arm11mpcore.c | 21 ++++++--------
hw/cpu/realview_mpcore.c | 29 +++++++------------
hw/net/cadence_gem.c | 26 +++++++++++++----
linux-user/syscall.c | 4 ++-
system/vl.c | 1 +
target/arm/cpu.c | 3 ++
target/arm/tcg/cpu32.c | 36 +++++++++++++++--------
target/arm/tcg/translate-a64.c | 11 -------
tests/tcg/arm/Makefile.target | 7 -----
tests/tcg/arm/README | 5 ----
tests/tcg/arm/test-arm-iwmmxt.S | 49 -------------------------------
21 files changed, 205 insertions(+), 249 deletions(-)
delete mode 100644 tests/tcg/arm/test-arm-iwmmxt.S
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PULL 00/12] target-arm queue
2025-02-10 15:49 Peter Maydell
@ 2025-02-11 2:37 ` Stefan Hajnoczi
0 siblings, 0 replies; 27+ messages in thread
From: Stefan Hajnoczi @ 2025-02-11 2:37 UTC (permalink / raw)
To: Peter Maydell; +Cc: qemu-devel
[-- Attachment #1: Type: text/plain, Size: 116 bytes --]
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/10.0 for any user-visible changes.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 27+ messages in thread
end of thread, other threads:[~2025-02-11 2:38 UTC | newest]
Thread overview: 27+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-14 16:26 [PULL 00/12] target-arm queue Peter Maydell
2020-04-14 16:26 ` [PULL 01/12] osdep.h: Drop no-longer-needed Coverity workarounds Peter Maydell
2020-04-14 16:26 ` [PULL 02/12] thread.h: Fix Coverity version of qemu_cond_timedwait() Peter Maydell
2020-04-14 16:26 ` [PULL 03/12] thread.h: Remove trailing semicolons from Coverity qemu_mutex_lock() etc Peter Maydell
2020-04-14 16:26 ` [PULL 04/12] linux-user/flatload.c: Use "" for include of QEMU header target_flat.h Peter Maydell
2020-04-14 16:26 ` [PULL 05/12] scripts/run-coverity-scan: Script to run Coverity Scan build Peter Maydell
2020-04-14 16:26 ` [PULL 06/12] scripts/coverity-scan: Add Docker support Peter Maydell
2020-04-14 16:26 ` [PULL 07/12] docs: Improve our gdbstub documentation Peter Maydell
2020-04-14 16:26 ` [PULL 08/12] configure: Honour --disable-werror for Sphinx Peter Maydell
2020-04-14 16:26 ` [PULL 09/12] scripts/kernel-doc: Add missing close-paren in c:function directives Peter Maydell
2020-04-14 16:26 ` [PULL 10/12] kernel-doc: Use c:struct for Sphinx 3.0 and later Peter Maydell
2020-04-14 16:26 ` [PULL 11/12] docs: Require Sphinx 1.6 or better Peter Maydell
2020-04-14 16:26 ` [PULL 12/12] Deprecate KVM support for AArch32 Peter Maydell
2020-04-14 19:09 ` [PULL 00/12] target-arm queue Peter Maydell
-- strict thread matches above, loose matches on Subject: below --
2025-02-10 15:49 Peter Maydell
2025-02-11 2:37 ` Stefan Hajnoczi
2023-11-06 15:32 Peter Maydell
2023-11-07 3:02 ` Stefan Hajnoczi
2023-05-12 15:34 Peter Maydell
2023-05-13 8:36 ` Richard Henderson
2021-07-18 12:46 Peter Maydell
2021-07-18 16:35 ` Peter Maydell
2020-10-08 14:25 Peter Maydell
2020-07-20 12:56 Peter Maydell
2020-07-20 21:24 ` Peter Maydell
2019-12-20 14:26 Peter Maydell
2020-01-06 10:32 ` Peter Maydell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).