[PATCH for 10.1 07/13] tests/tcg: skip libsyscall.so on softmmu tests

Alex Bennée posted 13 patches 3 months, 3 weeks ago
Maintainers: Paolo Bonzini <pbonzini@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Thomas Huth <thuth@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>
[PATCH for 10.1 07/13] tests/tcg: skip libsyscall.so on softmmu tests
Posted by Alex Bennée 3 months, 3 weeks ago
It isn't testing anything and just expanding the runtime of testing.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 tests/tcg/Makefile.target | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/tests/tcg/Makefile.target b/tests/tcg/Makefile.target
index af68f11664f..3d96182a7b9 100644
--- a/tests/tcg/Makefile.target
+++ b/tests/tcg/Makefile.target
@@ -155,6 +155,12 @@ VPATH+=$(PLUGIN_LIB)
 # For example, libpatch.so only needs to run against the arch-specific patch
 # target test, so we explicitly run it in the arch-specific Makefile.
 DISABLE_PLUGINS=libpatch.so
+
+# Likewise don't bother with the syscall plugin for softmmu
+ifneq ($(filter %-softmmu, $(TARGET)),)
+DISABLE_PLUGINS += libsyscall.so
+endif
+
 PLUGINS=$(filter-out $(DISABLE_PLUGINS), \
 	$(patsubst %.c, lib%.so, $(notdir $(wildcard $(PLUGIN_SRC)/*.c))))
 
-- 
2.47.2


Re: [PATCH for 10.1 07/13] tests/tcg: skip libsyscall.so on softmmu tests
Posted by Manos Pitsidianakis 3 months, 3 weeks ago
On Thu, Jul 24, 2025 at 2:00 PM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> It isn't testing anything and just expanding the runtime of testing.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>  tests/tcg/Makefile.target | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/tests/tcg/Makefile.target b/tests/tcg/Makefile.target
> index af68f11664f..3d96182a7b9 100644
> --- a/tests/tcg/Makefile.target
> +++ b/tests/tcg/Makefile.target
> @@ -155,6 +155,12 @@ VPATH+=$(PLUGIN_LIB)
>  # For example, libpatch.so only needs to run against the arch-specific patch
>  # target test, so we explicitly run it in the arch-specific Makefile.
>  DISABLE_PLUGINS=libpatch.so
> +
> +# Likewise don't bother with the syscall plugin for softmmu
> +ifneq ($(filter %-softmmu, $(TARGET)),)
> +DISABLE_PLUGINS += libsyscall.so
> +endif
> +
>  PLUGINS=$(filter-out $(DISABLE_PLUGINS), \
>         $(patsubst %.c, lib%.so, $(notdir $(wildcard $(PLUGIN_SRC)/*.c))))
>
> --
> 2.47.2
>
>

Reviewed-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>