* [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes
@ 2019-11-13 11:59 Alex Bennée
2019-11-13 11:59 ` [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional Alex Bennée
` (4 more replies)
0 siblings, 5 replies; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 11:59 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, stefanb, Alex Bennée, richard.henderson,
f4bug, cota, stefanha, marcandre.lureau, pbonzini, aurelien
Hi,
As we approach release I'm just bundling up my random fixes for 4.2
into one tree for each release candidate. This has a minor fix for
debugging vm builds and a number of minor documentation fixes
including moving the MTTCG docs across to rst format. The final patch
is a tweak to Travis to drop the 32 bit targets from
MAIN_SOFTMMU_TARGETS. This may be too controversial although the 32
bit targets still get built and tested under GCC.
Please review.
Alex Bennée (5):
tests/vm: make --interactive (and therefore DEBUG=1) unconditional
docs/devel: rename plugins.rst to tcg-plugins.rst
docs/devel: update tcg-plugins.rst with API versioning details
docs/devel: convert multi-thread-tcg to a .rst document
.travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
docs/devel/index.rst | 3 +-
...ti-thread-tcg.txt => multi-thread-tcg.rst} | 28 ++++++++++++-------
docs/devel/{plugins.rst => tcg-plugins.rst} | 27 ++++++++++++++----
.travis.yml | 2 +-
MAINTAINERS | 1 +
tests/vm/basevm.py | 2 +-
6 files changed, 44 insertions(+), 19 deletions(-)
rename docs/devel/{multi-thread-tcg.txt => multi-thread-tcg.rst} (96%)
rename docs/devel/{plugins.rst => tcg-plugins.rst} (83%)
--
2.20.1
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional
2019-11-13 11:59 [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes Alex Bennée
@ 2019-11-13 11:59 ` Alex Bennée
2019-11-13 13:31 ` Thomas Huth
2019-11-13 11:59 ` [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst Alex Bennée
` (3 subsequent siblings)
4 siblings, 1 reply; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 11:59 UTC (permalink / raw)
To: qemu-devel
Cc: fam, Peter Maydell, berrange, stefanb, Alex Bennée,
richard.henderson, f4bug, Philippe Mathieu-Daudé,
cota, stefanha, marcandre.lureau, pbonzini, aurelien
While the concept of only dropping to ssh if a test fails is nice it
is more useful for this to be unconditional. You usually just want to
get the build up and running and then noodle around debugging or
attempting to replicate.
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
v2
- fix spelling
---
tests/vm/basevm.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py
index 91a9226026d..0b8c1b26576 100755
--- a/tests/vm/basevm.py
+++ b/tests/vm/basevm.py
@@ -403,7 +403,7 @@ def main(vmcls):
exitcode = 0
if vm.ssh(*cmd) != 0:
exitcode = 3
- if exitcode != 0 and args.interactive:
+ if args.interactive:
vm.ssh()
if not args.snapshot:
--
2.20.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst
2019-11-13 11:59 [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes Alex Bennée
2019-11-13 11:59 ` [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional Alex Bennée
@ 2019-11-13 11:59 ` Alex Bennée
2019-11-13 16:23 ` Philippe Mathieu-Daudé
2019-11-13 11:59 ` [PATCH v1 3/5] docs/devel: update tcg-plugins.rst with API versioning details Alex Bennée
` (2 subsequent siblings)
4 siblings, 1 reply; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 11:59 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, stefanb, Alex Bennée, richard.henderson,
f4bug, cota, stefanha, marcandre.lureau, pbonzini, aurelien
This makes it a bit clearer what this is about.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
docs/devel/index.rst | 2 +-
docs/devel/{plugins.rst => tcg-plugins.rst} | 0
MAINTAINERS | 1 +
3 files changed, 2 insertions(+), 1 deletion(-)
rename docs/devel/{plugins.rst => tcg-plugins.rst} (100%)
diff --git a/docs/devel/index.rst b/docs/devel/index.rst
index 2ff058bae38..c86a3cdff2f 100644
--- a/docs/devel/index.rst
+++ b/docs/devel/index.rst
@@ -22,4 +22,4 @@ Contents:
decodetree
secure-coding-practices
tcg
- plugins
+ tcg-plugins
diff --git a/docs/devel/plugins.rst b/docs/devel/tcg-plugins.rst
similarity index 100%
rename from docs/devel/plugins.rst
rename to docs/devel/tcg-plugins.rst
diff --git a/MAINTAINERS b/MAINTAINERS
index ff8d0d29f4b..b160d817208 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2369,6 +2369,7 @@ F: tcg/
TCG Plugins
M: Alex Bennée <alex.bennee@linaro.org>
S: Maintained
+F: docs/devel/tcg-plugins.rst
F: plugins/
F: tests/plugin
--
2.20.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v1 3/5] docs/devel: update tcg-plugins.rst with API versioning details
2019-11-13 11:59 [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes Alex Bennée
2019-11-13 11:59 ` [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional Alex Bennée
2019-11-13 11:59 ` [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst Alex Bennée
@ 2019-11-13 11:59 ` Alex Bennée
[not found] ` <SN6PR13MB2272B60021982C8C7F680CAA80760@SN6PR13MB2272.namprd13.prod.outlook.com>
2019-11-13 11:59 ` [PATCH v1 4/5] docs/devel: convert multi-thread-tcg to a .rst document Alex Bennée
2019-11-13 11:59 ` [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS Alex Bennée
4 siblings, 1 reply; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 11:59 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, stefanb, Alex Bennée, richard.henderson,
f4bug, cota, stefanha, marcandre.lureau, pbonzini, aurelien
While we are at it fix up the quoted code sections with the inline ::
approach.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
v2
- fix grammar
- mention we also will fail to load outside the range
- clean-up code sections
---
docs/devel/tcg-plugins.rst | 27 +++++++++++++++++++++------
1 file changed, 21 insertions(+), 6 deletions(-)
diff --git a/docs/devel/tcg-plugins.rst b/docs/devel/tcg-plugins.rst
index b18fb6729e3..718eef00f22 100644
--- a/docs/devel/tcg-plugins.rst
+++ b/docs/devel/tcg-plugins.rst
@@ -25,6 +25,23 @@ process. However the project reserves the right to change or break the
API should it need to do so. The best way to avoid this is to submit
your plugin upstream so they can be updated if/when the API changes.
+API versioning
+--------------
+
+All plugins need to declare a symbol which exports the plugin API
+version they were built against. This can be done simply by::
+
+ QEMU_PLUGIN_EXPORT int qemu_plugin_version = QEMU_PLUGIN_VERSION;
+
+The core code will refuse to load a plugin that doesn't export a
+`qemu_plugin_version` symbol or if plugin version is outside of QEMU's
+supported range of API versions.
+
+Additionally the `qemu_info_t` structure which is passed to the
+`qemu_plugin_install` method of a plugin will detail the minimum and
+current API versions supported by QEMU. The API version will be
+incremented if new APIs are added. The minimum API version will be
+incremented if existing APIs are changed or removed.
Exposure of QEMU internals
--------------------------
@@ -40,16 +57,14 @@ instructions and events are opaque to the plugins themselves.
Usage
=====
-The QEMU binary needs to be compiled for plugin support:
+The QEMU binary needs to be compiled for plugin support::
-::
- configure --enable-plugins
+ configure --enable-plugins
Once built a program can be run with multiple plugins loaded each with
-their own arguments:
+their own arguments::
-::
- $QEMU $OTHER_QEMU_ARGS \
+ $QEMU $OTHER_QEMU_ARGS \
-plugin tests/plugin/libhowvec.so,arg=inline,arg=hint \
-plugin tests/plugin/libhotblocks.so
--
2.20.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v1 4/5] docs/devel: convert multi-thread-tcg to a .rst document
2019-11-13 11:59 [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes Alex Bennée
` (2 preceding siblings ...)
2019-11-13 11:59 ` [PATCH v1 3/5] docs/devel: update tcg-plugins.rst with API versioning details Alex Bennée
@ 2019-11-13 11:59 ` Alex Bennée
2019-11-13 11:59 ` [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS Alex Bennée
4 siblings, 0 replies; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 11:59 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, stefanb, Alex Bennée, richard.henderson,
f4bug, cota, stefanha, marcandre.lureau, pbonzini, aurelien
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
docs/devel/index.rst | 1 +
...ti-thread-tcg.txt => multi-thread-tcg.rst} | 28 ++++++++++++-------
2 files changed, 19 insertions(+), 10 deletions(-)
rename docs/devel/{multi-thread-tcg.txt => multi-thread-tcg.rst} (96%)
diff --git a/docs/devel/index.rst b/docs/devel/index.rst
index c86a3cdff2f..3e6624ec604 100644
--- a/docs/devel/index.rst
+++ b/docs/devel/index.rst
@@ -22,4 +22,5 @@ Contents:
decodetree
secure-coding-practices
tcg
+ multi-thread-tcg
tcg-plugins
diff --git a/docs/devel/multi-thread-tcg.txt b/docs/devel/multi-thread-tcg.rst
similarity index 96%
rename from docs/devel/multi-thread-tcg.txt
rename to docs/devel/multi-thread-tcg.rst
index 782bebc28b4..4e914bacc0c 100644
--- a/docs/devel/multi-thread-tcg.txt
+++ b/docs/devel/multi-thread-tcg.rst
@@ -1,7 +1,10 @@
-Copyright (c) 2015-2016 Linaro Ltd.
+.. Copyright (c) 2015-2016 Linaro Ltd.
+.. This work is licensed under the terms of the GNU GPL, version 2 or
+.. later. See the COPYING file in the top-level directory.
-This work is licensed under the terms of the GNU GPL, version 2 or
-later. See the COPYING file in the top-level directory.
+==================
+Multi-threaded TCG
+==================
Introduction
============
@@ -40,7 +43,7 @@ Main Run Loop
Even when there is no code being generated there are a number of
structures associated with the hot-path through the main run-loop.
These are associated with looking up the next translation block to
-execute. These include:
+execute. These include::
tb_jmp_cache (per-vCPU, cache of recent jumps)
tb_ctx.htable (global hash table, phys address->tb lookup)
@@ -61,7 +64,9 @@ have their block-to-block jumps patched.
Global TCG State
----------------
-### User-mode emulation
+User-mode emulation
+~~~~~~~~~~~~~~~~~~~
+
We need to protect the entire code generation cycle including any post
generation patching of the translated code. This also implies a shared
translation buffer which contains code running on all cores. Any
@@ -78,7 +83,9 @@ patching.
Code generation is serialised with mmap_lock().
-### !User-mode emulation
+System emulation
+~~~~~~~~~~~~~~~~
+
Each vCPU has its own TCG context and associated TCG region, thereby
requiring no locking.
@@ -125,10 +132,11 @@ linked list of all Translation Blocks in that page (see page_next).
Both the jump patching and the page cache involve linked lists that
the invalidated TranslationBlock needs to be removed from.
-DESIGN REQUIREMENT: Safely handle invalidation of TBs
- - safely patch/revert direct jumps
- - remove central PageDesc lookup entries
- - ensure lookup caches/hashes are safely updated
+DESIGN REQUIREMENTS:
+ - Safely handle invalidation of TBs
+ - safely patch/revert direct jumps
+ - remove central PageDesc lookup entries
+ - ensure lookup caches/hashes are safely updated
(Current solution)
--
2.20.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 11:59 [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes Alex Bennée
` (3 preceding siblings ...)
2019-11-13 11:59 ` [PATCH v1 4/5] docs/devel: convert multi-thread-tcg to a .rst document Alex Bennée
@ 2019-11-13 11:59 ` Alex Bennée
2019-11-13 13:30 ` Thomas Huth
2019-11-13 16:29 ` Philippe Mathieu-Daudé
4 siblings, 2 replies; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 11:59 UTC (permalink / raw)
To: qemu-devel
Cc: fam, berrange, stefanb, Alex Bennée, richard.henderson,
f4bug, Philippe Mathieu-Daudé,
cota, stefanha, marcandre.lureau, pbonzini, aurelien
The older clangs are still struggling to build and run everything
withing the 50 minute timeout so lets lighten the load a bit more. We
still have coverage for GCC and hopefully no obscure 32 bit guest only
breakages slip through the cracks.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.travis.yml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index b9a026c8eeb..c09b6a00143 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -79,7 +79,7 @@ env:
- BASE_CONFIG="--disable-docs --disable-tools"
- TEST_CMD="make check V=1"
# This is broadly a list of "mainline" softmmu targets which have support across the major distros
- - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
+ - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
- CCACHE_SLOPPINESS="include_file_ctime,include_file_mtime"
- CCACHE_MAXSIZE=1G
--
2.20.1
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 11:59 ` [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS Alex Bennée
@ 2019-11-13 13:30 ` Thomas Huth
2019-11-19 12:13 ` Thomas Huth
2019-11-13 16:29 ` Philippe Mathieu-Daudé
1 sibling, 1 reply; 16+ messages in thread
From: Thomas Huth @ 2019-11-13 13:30 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
pbonzini, marcandre.lureau, Philippe Mathieu-Daudé,
aurelien
On 13/11/2019 12.59, Alex Bennée wrote:
> The older clangs are still struggling to build and run everything
> withing the 50 minute timeout so lets lighten the load a bit more. We
> still have coverage for GCC and hopefully no obscure 32 bit guest only
> breakages slip through the cracks.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .travis.yml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index b9a026c8eeb..c09b6a00143 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -79,7 +79,7 @@ env:
> - BASE_CONFIG="--disable-docs --disable-tools"
> - TEST_CMD="make check V=1"
> # This is broadly a list of "mainline" softmmu targets which have support across the major distros
> - - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
> + - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
> - CCACHE_SLOPPINESS="include_file_ctime,include_file_mtime"
> - CCACHE_MAXSIZE=1G
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional
2019-11-13 11:59 ` [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional Alex Bennée
@ 2019-11-13 13:31 ` Thomas Huth
0 siblings, 0 replies; 16+ messages in thread
From: Thomas Huth @ 2019-11-13 13:31 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, Peter Maydell, berrange, stefanb, richard.henderson, f4bug,
cota, stefanha, pbonzini, marcandre.lureau,
Philippe Mathieu-Daudé,
aurelien
On 13/11/2019 12.59, Alex Bennée wrote:
> While the concept of only dropping to ssh if a test fails is nice it
> is more useful for this to be unconditional. You usually just want to
> get the build up and running and then noodle around debugging or
> attempting to replicate.
>
> Cc: Peter Maydell <peter.maydell@linaro.org>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>
> ---
> v2
> - fix spelling
> ---
> tests/vm/basevm.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py
> index 91a9226026d..0b8c1b26576 100755
> --- a/tests/vm/basevm.py
> +++ b/tests/vm/basevm.py
> @@ -403,7 +403,7 @@ def main(vmcls):
> exitcode = 0
> if vm.ssh(*cmd) != 0:
> exitcode = 3
> - if exitcode != 0 and args.interactive:
> + if args.interactive:
> vm.ssh()
>
> if not args.snapshot:
>
Reviewed-by: Thomas Huth <thuth@redhat.com>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: FW: [PATCH v1 3/5] docs/devel: update tcg-plugins.rst with API versioning details
[not found] ` <SN6PR13MB2272B60021982C8C7F680CAA80760@SN6PR13MB2272.namprd13.prod.outlook.com>
@ 2019-11-13 14:40 ` Robert Foley
0 siblings, 0 replies; 16+ messages in thread
From: Robert Foley @ 2019-11-13 14:40 UTC (permalink / raw)
To: qemu-devel, Alex Bennée
Cc: fam, berrange, stefanb, Richard Henderson, f4bug, cota, stefanha,
pbonzini, marcandre.lureau, aurelien
On Wed, 13 Nov 2019 at 07:00, Alex Bennée <alex.bennee@linaro.org> wrote:
>
> While we are at it fix up the quoted code sections with the inline ::
> approach.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>
> ---
> v2
> - fix grammar
> - mention we also will fail to load outside the range
> - clean-up code sections
> ---
> docs/devel/tcg-plugins.rst | 27 +++++++++++++++++++++------
> 1 file changed, 21 insertions(+), 6 deletions(-)
>
> diff --git a/docs/devel/tcg-plugins.rst b/docs/devel/tcg-plugins.rst
> index b18fb6729e3..718eef00f22 100644
> --- a/docs/devel/tcg-plugins.rst
> +++ b/docs/devel/tcg-plugins.rst
> @@ -25,6 +25,23 @@ process. However the project reserves the right to change or break the
> API should it need to do so. The best way to avoid this is to submit
> your plugin upstream so they can be updated if/when the API changes.
>
> +API versioning
> +--------------
> +
> +All plugins need to declare a symbol which exports the plugin API
> +version they were built against. This can be done simply by::
> +
> + QEMU_PLUGIN_EXPORT int qemu_plugin_version = QEMU_PLUGIN_VERSION;
> +
> +The core code will refuse to load a plugin that doesn't export a
> +`qemu_plugin_version` symbol or if plugin version is outside of QEMU's
> +supported range of API versions.
> +
> +Additionally the `qemu_info_t` structure which is passed to the
> +`qemu_plugin_install` method of a plugin will detail the minimum and
> +current API versions supported by QEMU. The API version will be
> +incremented if new APIs are added. The minimum API version will be
> +incremented if existing APIs are changed or removed.
>
> Exposure of QEMU internals
> --------------------------
> @@ -40,16 +57,14 @@ instructions and events are opaque to the plugins themselves.
> Usage
> =====
>
> -The QEMU binary needs to be compiled for plugin support:
> +The QEMU binary needs to be compiled for plugin support::
>
> -::
> - configure --enable-plugins
> + configure --enable-plugins
>
> Once built a program can be run with multiple plugins loaded each with
> -their own arguments:
> +their own arguments::
>
> -::
> - $QEMU $OTHER_QEMU_ARGS \
> + $QEMU $OTHER_QEMU_ARGS \
> -plugin tests/plugin/libhowvec.so,arg=inline,arg=hint \
> -plugin tests/plugin/libhotblocks.so
>
> --
> 2.20.1
>
>
Reviewed-by: Robert Foley <robert.foley@linaro.org>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst
2019-11-13 11:59 ` [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst Alex Bennée
@ 2019-11-13 16:23 ` Philippe Mathieu-Daudé
2019-11-13 16:39 ` Paolo Bonzini
0 siblings, 1 reply; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-11-13 16:23 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
pbonzini, marcandre.lureau, aurelien
On 11/13/19 12:59 PM, Alex Bennée wrote:
> This makes it a bit clearer what this is about.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> docs/devel/index.rst | 2 +-
> docs/devel/{plugins.rst => tcg-plugins.rst} | 0
> MAINTAINERS | 1 +
> 3 files changed, 2 insertions(+), 1 deletion(-)
> rename docs/devel/{plugins.rst => tcg-plugins.rst} (100%)
>
> diff --git a/docs/devel/index.rst b/docs/devel/index.rst
> index 2ff058bae38..c86a3cdff2f 100644
> --- a/docs/devel/index.rst
> +++ b/docs/devel/index.rst
> @@ -22,4 +22,4 @@ Contents:
> decodetree
> secure-coding-practices
> tcg
> - plugins
> + tcg-plugins
> diff --git a/docs/devel/plugins.rst b/docs/devel/tcg-plugins.rst
> similarity index 100%
> rename from docs/devel/plugins.rst
> rename to docs/devel/tcg-plugins.rst
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ff8d0d29f4b..b160d817208 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2369,6 +2369,7 @@ F: tcg/
> TCG Plugins
> M: Alex Bennée <alex.bennee@linaro.org>
> S: Maintained
> +F: docs/devel/tcg-plugins.rst
> F: plugins/
What about (other patch):
plugins/ -> tcg/plugins/
> F: tests/plugin
tests/plugin/ -> tests/tcg/plugins/
Anyway for this patch:
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 11:59 ` [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS Alex Bennée
2019-11-13 13:30 ` Thomas Huth
@ 2019-11-13 16:29 ` Philippe Mathieu-Daudé
2019-11-13 17:38 ` [EXTERNAL]Re: " Aleksandar Markovic
1 sibling, 1 reply; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-11-13 16:29 UTC (permalink / raw)
To: Alex Bennée, qemu-devel, Aleksandar Markovic
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
marcandre.lureau, pbonzini, aurelien
On 11/13/19 12:59 PM, Alex Bennée wrote:
> The older clangs are still struggling to build and run everything
> withing the 50 minute timeout so lets lighten the load a bit more. We
> still have coverage for GCC and hopefully no obscure 32 bit guest only
> breakages slip through the cracks.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .travis.yml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index b9a026c8eeb..c09b6a00143 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -79,7 +79,7 @@ env:
> - BASE_CONFIG="--disable-docs --disable-tools"
> - TEST_CMD="make check V=1"
> # This is broadly a list of "mainline" softmmu targets which have support across the major distros
> - - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
> + - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
Aleksandar, since you mostly test 32-bit MIPS, are you OK we keep
mips-softmmu and drop mips64-softmmu here? Another job (acceptance-test)
builds the mips64el-softmmu.
> - CCACHE_SLOPPINESS="include_file_ctime,include_file_mtime"
> - CCACHE_MAXSIZE=1G
>
>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst
2019-11-13 16:23 ` Philippe Mathieu-Daudé
@ 2019-11-13 16:39 ` Paolo Bonzini
0 siblings, 0 replies; 16+ messages in thread
From: Paolo Bonzini @ 2019-11-13 16:39 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Alex Bennée, qemu-devel
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
marcandre.lureau, aurelien
> What about (other patch):
>
> plugins/ -> tcg/plugins/
>
>> F: tests/plugin
>
> tests/plugin/ -> tests/tcg/plugins/
tests/tcg is for code that runs on the target in general, but perhaps
tests/tcg-plugins would make sense.
Paolo
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [EXTERNAL]Re: [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 16:29 ` Philippe Mathieu-Daudé
@ 2019-11-13 17:38 ` Aleksandar Markovic
2019-11-13 18:44 ` Philippe Mathieu-Daudé
2019-11-13 19:01 ` Alex Bennée
0 siblings, 2 replies; 16+ messages in thread
From: Aleksandar Markovic @ 2019-11-13 17:38 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Alex Bennée, qemu-devel
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
marcandre.lureau, pbonzini, aurelien
> From: Philippe Mathieu-Daudé <philmd@redhat.com>
> > - - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
> > + - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>
> Aleksandar, since you mostly test 32-bit MIPS, are you OK we keep
> mips-softmmu and drop mips64-softmmu here? Another job (acceptance-test)
> builds the mips64el-softmmu.
Philippe, thanks for bringing this to my attention. Yes, 32-bit mips targets are important to us, but, what can we do, time constraints are time constraints, so I agree with Alex change, please go ahead, Alex. We can test 32-bit mips targets via other acceptance tests (those that can run longer, so-called "slow" group), and perhaps we can extend them to test more 32-bit mips systems.
Thanks to everybody,
Aleksandar
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [EXTERNAL]Re: [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 17:38 ` [EXTERNAL]Re: " Aleksandar Markovic
@ 2019-11-13 18:44 ` Philippe Mathieu-Daudé
2019-11-13 19:01 ` Alex Bennée
1 sibling, 0 replies; 16+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-11-13 18:44 UTC (permalink / raw)
To: Aleksandar Markovic, Alex Bennée, qemu-devel
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
marcandre.lureau, pbonzini, aurelien
On 11/13/19 6:38 PM, Aleksandar Markovic wrote:
>> From: Philippe Mathieu-Daudé <philmd@redhat.com>
>>> - - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>>> + - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>>
>> Aleksandar, since you mostly test 32-bit MIPS, are you OK we keep
>> mips-softmmu and drop mips64-softmmu here? Another job (acceptance-test)
>> builds the mips64el-softmmu.
>
> Philippe, thanks for bringing this to my attention. Yes, 32-bit mips targets are important to us, but, what can we do, time constraints are time constraints, so I agree with Alex change, please go ahead, Alex. We can test 32-bit mips targets via other acceptance tests (those that can run longer, so-called "slow" group), and perhaps we can extend them to test more 32-bit mips systems.
OK, let's keep mips64 as suggested Alex then.
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [EXTERNAL]Re: [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 17:38 ` [EXTERNAL]Re: " Aleksandar Markovic
2019-11-13 18:44 ` Philippe Mathieu-Daudé
@ 2019-11-13 19:01 ` Alex Bennée
1 sibling, 0 replies; 16+ messages in thread
From: Alex Bennée @ 2019-11-13 19:01 UTC (permalink / raw)
To: Aleksandar Markovic
Cc: fam, berrange, stefanb, richard.henderson, qemu-devel, f4bug,
cota, stefanha, marcandre.lureau, pbonzini,
Philippe Mathieu-Daudé,
aurelien
Aleksandar Markovic <amarkovic@wavecomp.com> writes:
>> From: Philippe Mathieu-Daudé <philmd@redhat.com>
>> > - - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>> > + - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>>
>> Aleksandar, since you mostly test 32-bit MIPS, are you OK we keep
>> mips-softmmu and drop mips64-softmmu here? Another job (acceptance-test)
>> builds the mips64el-softmmu.
>
> Philippe, thanks for bringing this to my attention. Yes, 32-bit mips
> targets are important to us, but, what can we do, time constraints are
> time constraints, so I agree with Alex change, please go ahead, Alex.
> We can test 32-bit mips targets via other acceptance tests (those that
> can run longer, so-called "slow" group), and perhaps we can extend
> them to test more 32-bit mips systems.
To be clear both gcc and clang have rules that test:
- CONFIG="--disable-user --target-list-exclude=${MAIN_SOFTMMU_TARGETS}"
So the main targets which are reducing their coverage are:
- CONFIG="--enable-debug --target-list=${MAIN_SOFTMMU_TARGETS}"
- CONFIG="--enable-modules --target-list=${MAIN_SOFTMMU_TARGETS}"
- CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS} "
- CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-sanitize"
compiler: clang
before_script:
- ./configure ${CONFIG} --extra-cflags="-fsanitize=undefined -Werror" || { cat config.log && exit 1; }
- CONFIG="--enable-gprof --enable-gcov --disable-pie --target-list=${MAIN_SOFTMMU_TARGETS}"
and the MacOSX 9.4 build:
# MacOSX builds
- env:
- CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS}"
os: osx
osx_image: xcode9.4
compiler: clang
The Xcode 10.3 build is already a reduced list:
- CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
>
> Thanks to everybody,
> Aleksandar
--
Alex Bennée
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS
2019-11-13 13:30 ` Thomas Huth
@ 2019-11-19 12:13 ` Thomas Huth
0 siblings, 0 replies; 16+ messages in thread
From: Thomas Huth @ 2019-11-19 12:13 UTC (permalink / raw)
To: Alex Bennée, qemu-devel
Cc: fam, berrange, stefanb, richard.henderson, f4bug, cota, stefanha,
marcandre.lureau, pbonzini, Philippe Mathieu-Daudé,
aurelien
On 13/11/2019 14.30, Thomas Huth wrote:
> On 13/11/2019 12.59, Alex Bennée wrote:
>> The older clangs are still struggling to build and run everything
>> withing the 50 minute timeout so lets lighten the load a bit more. We
>> still have coverage for GCC and hopefully no obscure 32 bit guest only
>> breakages slip through the cracks.
>>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>> .travis.yml | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index b9a026c8eeb..c09b6a00143 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -79,7 +79,7 @@ env:
>> - BASE_CONFIG="--disable-docs --disable-tools"
>> - TEST_CMD="make check V=1"
>> # This is broadly a list of "mainline" softmmu targets which have support across the major distros
>> - - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,arm-softmmu,i386-softmmu,mips-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>> + - MAIN_SOFTMMU_TARGETS="aarch64-softmmu,mips64-softmmu,ppc64-softmmu,riscv64-softmmu,s390x-softmmu,x86_64-softmmu"
>> - CCACHE_SLOPPINESS="include_file_ctime,include_file_mtime"
>> - CCACHE_MAXSIZE=1G
>
> Reviewed-by: Thomas Huth <thuth@redhat.com>
On a second glance, we also have this entry with --target-list-exclude
in our test matrix:
- env:
- CONFIG="--disable-user
--target-list-exclude=${MAIN_SOFTMMU_TARGETS}"
- CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-default"
compiler: clang
So while you've speed up one target, this one might get actually slower
instead. That's a little bit unfortunate. Is there maybe a better way to
tackle this?
Thomas
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2019-11-19 12:15 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-13 11:59 [PATCH for 4.2-rc2 v1 0/5] misc doc and testing fixes Alex Bennée
2019-11-13 11:59 ` [PATCH v1 1/5] tests/vm: make --interactive (and therefore DEBUG=1) unconditional Alex Bennée
2019-11-13 13:31 ` Thomas Huth
2019-11-13 11:59 ` [PATCH v1 2/5] docs/devel: rename plugins.rst to tcg-plugins.rst Alex Bennée
2019-11-13 16:23 ` Philippe Mathieu-Daudé
2019-11-13 16:39 ` Paolo Bonzini
2019-11-13 11:59 ` [PATCH v1 3/5] docs/devel: update tcg-plugins.rst with API versioning details Alex Bennée
[not found] ` <SN6PR13MB2272B60021982C8C7F680CAA80760@SN6PR13MB2272.namprd13.prod.outlook.com>
2019-11-13 14:40 ` FW: " Robert Foley
2019-11-13 11:59 ` [PATCH v1 4/5] docs/devel: convert multi-thread-tcg to a .rst document Alex Bennée
2019-11-13 11:59 ` [PATCH v1 5/5] .travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS Alex Bennée
2019-11-13 13:30 ` Thomas Huth
2019-11-19 12:13 ` Thomas Huth
2019-11-13 16:29 ` Philippe Mathieu-Daudé
2019-11-13 17:38 ` [EXTERNAL]Re: " Aleksandar Markovic
2019-11-13 18:44 ` Philippe Mathieu-Daudé
2019-11-13 19:01 ` Alex Bennée
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).