[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 02/18] deprecation: don't enable TCG plugins by default with T
From: |
Alex Bennée |
Subject: |
[PATCH v2 02/18] deprecation: don't enable TCG plugins by default with TCI |
Date: |
Mon, 16 Sep 2024 09:53:44 +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.5
- [PATCH v2 00/18] tcg plugins pre-PR (deprecations, mem apis, contrib plugins), Alex Bennée, 2024/09/16
- [PATCH v2 04/18] plugins: save value during memory accesses, Alex Bennée, 2024/09/16
- [PATCH v2 02/18] deprecation: don't enable TCG plugins by default with TCI,
Alex Bennée <=
- [PATCH v2 05/18] plugins: extend API to get latest memory value accessed, Alex Bennée, 2024/09/16
- [PATCH v2 03/18] contrib/plugins: control flow plugin, Alex Bennée, 2024/09/16
- [PATCH v2 10/18] tests/tcg: clean up output of memory system test, Alex Bennée, 2024/09/16
- [PATCH v2 06/18] tests/tcg: add mechanism to run specific tests with plugins, Alex Bennée, 2024/09/16
- [PATCH v2 09/18] tests/tcg/multiarch: add test for plugin memory access, Alex Bennée, 2024/09/16
- [PATCH v2 11/18] tests/tcg: only read/write 64 bit words on 64 bit systems, Alex Bennée, 2024/09/16
- [PATCH v2 12/18] tests/tcg: ensure s390x-softmmu output redirected, Alex Bennée, 2024/09/16
- [PATCH v2 08/18] tests/tcg/plugins/mem: add option to print memory accesses, Alex Bennée, 2024/09/16