On Fri, 2015-03-27 at 15:17 -0700, Darren Hart wrote: > Adapt the futextest Makefiles to use lib.mk macros for RUN_TESTS and > EMIT_TESTS. For now, we reuse the run.sh mechanism provided by > futextest. This doesn't provide the standard selftests: [PASS|FAIL] > format, but the tests provide very similar output already. > > This results in the run_kselftest.sh script for futexes including a > single line: ./run.sh > > Cc: Shuah Khan <shua...@osg.samsung.com> > Cc: linux-...@vger.kernel.org > Cc: Ingo Molnar <mi...@elte.hu> > Cc: Peter Zijlstra <pet...@infradead.org> > Cc: Thomas Gleixner <t...@linutronix.de> > Cc: Davidlohr Bueso <d...@stgolabs.net> > Cc: KOSAKI Motohiro <kosaki.motoh...@jp.fujitsu.com> > Signed-off-by: Darren Hart <dvh...@linux.intel.com> > --- > tools/testing/selftests/futex/Makefile | 21 +++++++++++++++++++++ > tools/testing/selftests/futex/functional/Makefile | 4 ++++ > 2 files changed, 25 insertions(+) > > diff --git a/tools/testing/selftests/futex/Makefile > b/tools/testing/selftests/futex/Makefile > index 8629187..6a17529 100644 > --- a/tools/testing/selftests/futex/Makefile > +++ b/tools/testing/selftests/futex/Makefile > @@ -1,8 +1,29 @@ > SUBDIRS := functional > > +TEST_PROGS := run.sh > + > .PHONY: all clean
lib.mk defines all & clean as PHONY for you. > all: > for DIR in $(SUBDIRS); do $(MAKE) -C $$DIR $@ ; done > > +include ../lib.mk > + > +override define RUN_TESTS > + ./run.sh > +endef Do you need to do this override? The standard logic should work AFAICS, or do you not want the echo logic? > +override define INSTALL_RULE > + mkdir -p $(INSTALL_PATH) > + install -t $(INSTALL_PATH) $(TEST_PROGS) $(TEST_PROGS_EXTENDED) > $(TEST_FILES) > + > + @for SUBDIR in $(SUBDIRS); do \ > + $(MAKE) -C $$SUBDIR INSTALL_PATH=$(INSTALL_PATH)/$$SUBDIR > install; \ > + done; > +endef > + > +override define EMIT_TESTS > + echo "./run.sh" > +endef Ditto. cheers -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
