[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 02/17] deprecation: don't enable TCG plugins by default with TCI
From: |
Alex Bennée |
Subject: |
[PATCH 02/17] deprecation: don't enable TCG plugins by default with TCI |
Date: |
Fri, 13 Sep 2024 18:26:40 +0100 |
The softmmu memory instrumentation test sees so many more accesses
than a normal translated host and its really not worth fixing up. Lets
deprecate this odd configuration and save on the CI cycles.
Message-Id: <20240910140733.4007719-13-alex.bennee@linaro.org>
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
docs/about/deprecated.rst | 8 ++++++++
configure | 11 +++++++++--
2 files changed, 17 insertions(+), 2 deletions(-)
diff --git a/docs/about/deprecated.rst b/docs/about/deprecated.rst
index f7c7c33d39..5aa2e35314 100644
--- a/docs/about/deprecated.rst
+++ b/docs/about/deprecated.rst
@@ -196,6 +196,14 @@ benefits from having plenty of host memory it seems
reasonable to
encourage users to use 64 bit builds of QEMU for analysis work
whatever targets they are instrumenting.
+TCG Plugin support not enabled by default with TCI (since 9.2)
+''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
+
+While the TCG interpreter can interpret the TCG ops used by plugins it
+is going to be so much slower it wouldn't make sense for any serious
+instrumentation. Due to implementation differences there will also be
+anomalies in things like memory instrumentation.
+
System emulator CPUs
--------------------
diff --git a/configure b/configure
index f376fa24b3..3778b61c40 100755
--- a/configure
+++ b/configure
@@ -629,6 +629,9 @@ meson_option_parse() {
exit 1
fi
}
+has_meson_option() {
+ test "${meson_options#*"$1"}" != "$meson_options"
+}
meson_add_machine_file() {
if test "$cross_compile" = "yes"; then
@@ -1048,8 +1051,12 @@ if test "$static" = "yes" ; then
plugins="no"
fi
if test "$plugins" != "no" && test $host_bits -eq 64; then
- plugins=yes
- subdirs="$subdirs contrib/plugins"
+ if has_meson_option "-Dtcg_interpreter=true"; then
+ plugins="no"
+ else
+ plugins=yes
+ subdirs="$subdirs contrib/plugins"
+ fi
fi
cat > $TMPC << EOF
--
2.39.2
- [PATCH 00/17] tcg plugins pre-PR (deprecations, mem apis, contrib plugins), Alex Bennée, 2024/09/13
- [PATCH 02/17] deprecation: don't enable TCG plugins by default with TCI,
Alex Bennée <=
- [PATCH 01/17] deprecation: don't enable TCG plugins by default on 32 bit hosts, Alex Bennée, 2024/09/13
- [PATCH 04/17] plugins: save value during memory accesses, Alex Bennée, 2024/09/13
- [PATCH 03/17] contrib/plugins: control flow plugin, Alex Bennée, 2024/09/13
- [PATCH 07/17] tests/tcg: allow to check output of plugins, Alex Bennée, 2024/09/13
- [PATCH 05/17] plugins: extend API to get latest memory value accessed, Alex Bennée, 2024/09/13
- [PATCH 06/17] tests/tcg: add mechanism to run specific tests with plugins, Alex Bennée, 2024/09/13
- [PATCH 08/17] tests/tcg/plugins/mem: add option to print memory accesses, Alex Bennée, 2024/09/13
- [PATCH 10/17] tests/tcg: clean up output of memory system test, Alex Bennée, 2024/09/13
- [PATCH 09/17] tests/tcg/multiarch: add test for plugin memory access, Alex Bennée, 2024/09/13
- [PATCH 14/17] util/timer: avoid deadlock when shutting down, Alex Bennée, 2024/09/13