tests/tcg/Makefile.target | 21 ++++++++++++++++--- tests/tcg/multiarch/Makefile.target | 2 +- .../multiarch/system/Makefile.softmmu-target | 2 +- tests/tcg/x86_64/Makefile.softmmu-target | 2 +- 4 files changed, 21 insertions(+), 6 deletions(-)
Commit 25aaf0cb7f (“tests/tcg: reduce the number of plugin test
combinations”) added support for running tests with specific plugins
passed via the EXTRA_RUNS variable.
However, due to the optimization, the rules generated as a shuffled
combination of tests and plugins might not correctly generate the rules
for the tests passed via EXTRA_RUNS.
This commit fixes it by correctly generating the rules for the tests
that require a specific plugin to run.
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
---
tests/tcg/Makefile.target | 21 ++++++++++++++++---
tests/tcg/multiarch/Makefile.target | 2 +-
.../multiarch/system/Makefile.softmmu-target | 2 +-
tests/tcg/x86_64/Makefile.softmmu-target | 2 +-
4 files changed, 21 insertions(+), 6 deletions(-)
diff --git a/tests/tcg/Makefile.target b/tests/tcg/Makefile.target
index 18afd5be19..e7bc66b5a0 100644
--- a/tests/tcg/Makefile.target
+++ b/tests/tcg/Makefile.target
@@ -170,6 +170,10 @@ endif
PLUGINS=$(filter-out $(DISABLE_PLUGINS), \
$(patsubst %.c, lib%.so, $(notdir $(wildcard $(PLUGIN_SRC)/*.c))))
+strip-plugin = $(wordlist 1, 1, $(subst -with-, ,$1))
+extract-plugin = $(wordlist 2, 2, $(subst -with-, ,$1))
+extract-test = $(subst run-plugin-,,$(wordlist 1, 1, $(subst -with-, ,$1)))
+
# We need to ensure expand the run-plugin-TEST-with-PLUGIN
# pre-requistes manually here as we can't use stems to handle it. We
# only expand MULTIARCH_TESTS which are common on most of our targets
@@ -179,6 +183,13 @@ PLUGINS=$(filter-out $(DISABLE_PLUGINS), \
ifneq ($(MULTIARCH_TESTS),)
+# Extract extra tests from the extra test+plugin combination.
+EXTRA_TESTS_WITH_PLUGIN=$(foreach test, \
+ $(EXTRA_RUNS_WITH_PLUGIN),$(call extract-test,$(test)))
+# Exclude tests that were specified to run with specific plugins from the tests
+# which can run with any plugin combination.
+MULTIARCH_TESTS:=$(filter-out $(EXTRA_TESTS_WITH_PLUGIN), $(MULTIARCH_TESTS))
+
NUM_PLUGINS := $(words $(PLUGINS))
NUM_TESTS := $(words $(MULTIARCH_TESTS))
@@ -186,19 +197,23 @@ define mod_plus_one
$(shell $(PYTHON) -c "print( ($(1) % $(2)) + 1 )")
endef
+# Rules for running tests with any plugin combination, i.e., no specific plugin.
$(foreach _idx, $(shell seq 1 $(NUM_TESTS)), \
$(eval _test := $(word $(_idx), $(MULTIARCH_TESTS))) \
$(eval _plugin := $(word $(call mod_plus_one, $(_idx), $(NUM_PLUGINS)), $(PLUGINS))) \
$(eval run-plugin-$(_test)-with-$(_plugin): $(_test) $(_plugin)) \
$(eval RUN_TESTS+=run-plugin-$(_test)-with-$(_plugin)))
+# Rules for running extra tests with specific plugins.
+RULE_TEXT := $(foreach f,$(EXTRA_RUNS_WITH_PLUGIN), \
+ $(f): $(call extract-test,$(f)) $(call extract-plugin,$(f)))
+$(eval $(RULE_TEXT))
+
endif # MULTIARCH_TESTS
endif # CONFIG_PLUGIN
-strip-plugin = $(wordlist 1, 1, $(subst -with-, ,$1))
-extract-plugin = $(wordlist 2, 2, $(subst -with-, ,$1))
-
RUN_TESTS+=$(EXTRA_RUNS)
+RUN_TESTS+=$(EXTRA_RUNS_WITH_PLUGIN)
# Some plugins need additional arguments above the default to fully
# exercise things. We can define them on a per-test basis here.
diff --git a/tests/tcg/multiarch/Makefile.target b/tests/tcg/multiarch/Makefile.target
index 38345ff880..8dc65d7a06 100644
--- a/tests/tcg/multiarch/Makefile.target
+++ b/tests/tcg/multiarch/Makefile.target
@@ -201,7 +201,7 @@ run-plugin-test-plugin-mem-access-with-libmem.so: \
$(SRC_PATH)/tests/tcg/multiarch/check-plugin-output.sh \
$(QEMU) $<
-EXTRA_RUNS += run-plugin-test-plugin-mem-access-with-libmem.so
+EXTRA_RUNS_WITH_PLUGIN += run-plugin-test-plugin-mem-access-with-libmem.so
endif
# Update TESTS
diff --git a/tests/tcg/multiarch/system/Makefile.softmmu-target b/tests/tcg/multiarch/system/Makefile.softmmu-target
index 4171b4e6aa..98c4eda5e0 100644
--- a/tests/tcg/multiarch/system/Makefile.softmmu-target
+++ b/tests/tcg/multiarch/system/Makefile.softmmu-target
@@ -77,5 +77,5 @@ run-plugin-memory-with-libmem.so: memory libmem.so
run-plugin-memory-with-libmem.so: PLUGIN_ARGS=$(COMMA)region-summary=true
run-plugin-memory-with-libmem.so: CHECK_PLUGIN_OUTPUT_COMMAND=$(MULTIARCH_SYSTEM_SRC)/validate-memory-counts.py $@.out
-EXTRA_RUNS += run-plugin-memory-with-libmem.so
+EXTRA_RUNS_WITH_PLUGIN += run-plugin-memory-with-libmem.so
endif
diff --git a/tests/tcg/x86_64/Makefile.softmmu-target b/tests/tcg/x86_64/Makefile.softmmu-target
index 3e30ca9307..4e65f58b57 100644
--- a/tests/tcg/x86_64/Makefile.softmmu-target
+++ b/tests/tcg/x86_64/Makefile.softmmu-target
@@ -40,5 +40,5 @@ run-plugin-patch-target-with-libpatch.so: \
run-plugin-patch-target-with-libpatch.so: \
CHECK_PLUGIN_OUTPUT_COMMAND=$(X64_SYSTEM_SRC)/validate-patch.py $@.out
run-plugin-patch-target-with-libpatch.so: patch-target libpatch.so
-EXTRA_RUNS+=run-plugin-patch-target-with-libpatch.so
+EXTRA_RUNS_WITH_PLUGIN+=run-plugin-patch-target-with-libpatch.so
endif
--
2.34.1
Hi, On 7/31/25 15:52, Gustavo Romero wrote: > Commit 25aaf0cb7f (“tests/tcg: reduce the number of plugin test > combinations”) added support for running tests with specific plugins > passed via the EXTRA_RUNS variable. > > However, due to the optimization, the rules generated as a shuffled > combination of tests and plugins might not correctly generate the rules > for the tests passed via EXTRA_RUNS. > > This commit fixes it by correctly generating the rules for the tests > that require a specific plugin to run. > > Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org> > --- > tests/tcg/Makefile.target | 21 ++++++++++++++++--- > tests/tcg/multiarch/Makefile.target | 2 +- > .../multiarch/system/Makefile.softmmu-target | 2 +- > tests/tcg/x86_64/Makefile.softmmu-target | 2 +- > 4 files changed, 21 insertions(+), 6 deletions(-) > > diff --git a/tests/tcg/Makefile.target b/tests/tcg/Makefile.target > index 18afd5be19..e7bc66b5a0 100644 > --- a/tests/tcg/Makefile.target > +++ b/tests/tcg/Makefile.target > @@ -170,6 +170,10 @@ endif > PLUGINS=$(filter-out $(DISABLE_PLUGINS), \ > $(patsubst %.c, lib%.so, $(notdir $(wildcard $(PLUGIN_SRC)/*.c)))) > > +strip-plugin = $(wordlist 1, 1, $(subst -with-, ,$1)) > +extract-plugin = $(wordlist 2, 2, $(subst -with-, ,$1)) > +extract-test = $(subst run-plugin-,,$(wordlist 1, 1, $(subst -with-, ,$1))) > + > # We need to ensure expand the run-plugin-TEST-with-PLUGIN > # pre-requistes manually here as we can't use stems to handle it. We > # only expand MULTIARCH_TESTS which are common on most of our targets > @@ -179,6 +183,13 @@ PLUGINS=$(filter-out $(DISABLE_PLUGINS), \ > > ifneq ($(MULTIARCH_TESTS),) > > +# Extract extra tests from the extra test+plugin combination. > +EXTRA_TESTS_WITH_PLUGIN=$(foreach test, \ > + $(EXTRA_RUNS_WITH_PLUGIN),$(call extract-test,$(test))) > +# Exclude tests that were specified to run with specific plugins from the tests > +# which can run with any plugin combination. > +MULTIARCH_TESTS:=$(filter-out $(EXTRA_TESTS_WITH_PLUGIN), $(MULTIARCH_TESTS)) > + > NUM_PLUGINS := $(words $(PLUGINS)) > NUM_TESTS := $(words $(MULTIARCH_TESTS)) > > @@ -186,19 +197,23 @@ define mod_plus_one > $(shell $(PYTHON) -c "print( ($(1) % $(2)) + 1 )") > endef > > +# Rules for running tests with any plugin combination, i.e., no specific plugin. > $(foreach _idx, $(shell seq 1 $(NUM_TESTS)), \ > $(eval _test := $(word $(_idx), $(MULTIARCH_TESTS))) \ > $(eval _plugin := $(word $(call mod_plus_one, $(_idx), $(NUM_PLUGINS)), $(PLUGINS))) \ > $(eval run-plugin-$(_test)-with-$(_plugin): $(_test) $(_plugin)) \ > $(eval RUN_TESTS+=run-plugin-$(_test)-with-$(_plugin))) > > +# Rules for running extra tests with specific plugins. > +RULE_TEXT := $(foreach f,$(EXTRA_RUNS_WITH_PLUGIN), \ > + $(f): $(call extract-test,$(f)) $(call extract-plugin,$(f))) > +$(eval $(RULE_TEXT)) hmm this is busted. It doesn't work for when EXTRA_RUNS_WITH_PLUGIN has multiple tests. Please ignore this patch, I'm crafting a v2. Cheers, Gustavo > endif # MULTIARCH_TESTS > endif # CONFIG_PLUGIN > > -strip-plugin = $(wordlist 1, 1, $(subst -with-, ,$1)) > -extract-plugin = $(wordlist 2, 2, $(subst -with-, ,$1)) > - > RUN_TESTS+=$(EXTRA_RUNS) > +RUN_TESTS+=$(EXTRA_RUNS_WITH_PLUGIN) > > # Some plugins need additional arguments above the default to fully > # exercise things. We can define them on a per-test basis here. > diff --git a/tests/tcg/multiarch/Makefile.target b/tests/tcg/multiarch/Makefile.target > index 38345ff880..8dc65d7a06 100644 > --- a/tests/tcg/multiarch/Makefile.target > +++ b/tests/tcg/multiarch/Makefile.target > @@ -201,7 +201,7 @@ run-plugin-test-plugin-mem-access-with-libmem.so: \ > $(SRC_PATH)/tests/tcg/multiarch/check-plugin-output.sh \ > $(QEMU) $< > > -EXTRA_RUNS += run-plugin-test-plugin-mem-access-with-libmem.so > +EXTRA_RUNS_WITH_PLUGIN += run-plugin-test-plugin-mem-access-with-libmem.so > endif > > # Update TESTS > diff --git a/tests/tcg/multiarch/system/Makefile.softmmu-target b/tests/tcg/multiarch/system/Makefile.softmmu-target > index 4171b4e6aa..98c4eda5e0 100644 > --- a/tests/tcg/multiarch/system/Makefile.softmmu-target > +++ b/tests/tcg/multiarch/system/Makefile.softmmu-target > @@ -77,5 +77,5 @@ run-plugin-memory-with-libmem.so: memory libmem.so > run-plugin-memory-with-libmem.so: PLUGIN_ARGS=$(COMMA)region-summary=true > run-plugin-memory-with-libmem.so: CHECK_PLUGIN_OUTPUT_COMMAND=$(MULTIARCH_SYSTEM_SRC)/validate-memory-counts.py $@.out > > -EXTRA_RUNS += run-plugin-memory-with-libmem.so > +EXTRA_RUNS_WITH_PLUGIN += run-plugin-memory-with-libmem.so > endif > diff --git a/tests/tcg/x86_64/Makefile.softmmu-target b/tests/tcg/x86_64/Makefile.softmmu-target > index 3e30ca9307..4e65f58b57 100644 > --- a/tests/tcg/x86_64/Makefile.softmmu-target > +++ b/tests/tcg/x86_64/Makefile.softmmu-target > @@ -40,5 +40,5 @@ run-plugin-patch-target-with-libpatch.so: \ > run-plugin-patch-target-with-libpatch.so: \ > CHECK_PLUGIN_OUTPUT_COMMAND=$(X64_SYSTEM_SRC)/validate-patch.py $@.out > run-plugin-patch-target-with-libpatch.so: patch-target libpatch.so > -EXTRA_RUNS+=run-plugin-patch-target-with-libpatch.so > +EXTRA_RUNS_WITH_PLUGIN+=run-plugin-patch-target-with-libpatch.so > endif
© 2016 - 2025 Red Hat, Inc.