On Wed, 21 Feb 2024 14:17:06 -0300 Marcos Paulo de Souza <mpdeso...@suse.com> wrote:
Shuah, please skip this one. The simplification is not worthy if it causes a new warning. This reply[1] explains what's going on. Thanks in advance, Marcos [1]: https://lore.kernel.org/linux-kselftest/20240223161244.17709-1-mpdeso...@suse.com/ > We don't need new targets only to run two make modules and make clean. > We can test if TEST_GEN_MODS_DIR is specified, and then run the > commands. > > Signed-off-by: Marcos Paulo de Souza <mpdeso...@suse.com> > --- > tools/testing/selftests/lib.mk | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/tools/testing/selftests/lib.mk b/tools/testing/selftests/lib.mk > index 286ce0ee102b..eddcd4a849dc 100644 > --- a/tools/testing/selftests/lib.mk > +++ b/tools/testing/selftests/lib.mk > @@ -58,8 +58,9 @@ TEST_GEN_PROGS := $(patsubst > %,$(OUTPUT)/%,$(TEST_GEN_PROGS)) > TEST_GEN_PROGS_EXTENDED := $(patsubst > %,$(OUTPUT)/%,$(TEST_GEN_PROGS_EXTENDED)) > TEST_GEN_FILES := $(patsubst %,$(OUTPUT)/%,$(TEST_GEN_FILES)) > > -all: $(TEST_GEN_PROGS) $(TEST_GEN_PROGS_EXTENDED) $(TEST_GEN_FILES) \ > - $(if $(TEST_GEN_MODS_DIR),gen_mods_dir) > +all: $(TEST_GEN_PROGS) $(TEST_GEN_PROGS_EXTENDED) $(TEST_GEN_FILES) > + $(if $(TEST_GEN_MODS_DIR), \ > + $(Q)$(MAKE) -C $(TEST_GEN_MODS_DIR)) > > define RUN_TESTS > BASE_DIR="$(selfdir)"; \ > @@ -85,11 +86,6 @@ else > @$(call RUN_TESTS, $(TEST_GEN_PROGS) $(TEST_CUSTOM_PROGS) $(TEST_PROGS)) > endif > > -gen_mods_dir: > - $(Q)$(MAKE) -C $(TEST_GEN_MODS_DIR) > - > -clean_mods_dir: > - $(Q)$(MAKE) -C $(TEST_GEN_MODS_DIR) clean > > define INSTALL_SINGLE_RULE > $(if $(INSTALL_LIST),@mkdir -p $(INSTALL_PATH)) > @@ -133,9 +129,11 @@ endif > > define CLEAN > $(RM) -r $(TEST_GEN_PROGS) $(TEST_GEN_PROGS_EXTENDED) $(TEST_GEN_FILES) > $(EXTRA_CLEAN) > + $(if $(TEST_GEN_MODS_DIR), \ > + $(Q)$(MAKE) -C $(TEST_GEN_MODS_DIR) clean) > endef > > -clean: $(if $(TEST_GEN_MODS_DIR),clean_mods_dir) > +clean: > $(CLEAN) > > # Enables to extend CFLAGS and LDFLAGS from command line, e.g. > @@ -166,4 +164,4 @@ $(OUTPUT)/%:%.S > $(LINK.S) $^ $(LDLIBS) -o $@ > endif > > -.PHONY: run_tests all clean install emit_tests gen_mods_dir clean_mods_dir > +.PHONY: run_tests all clean install emit_tests > > -- > 2.42.1
