Re: [PATCH 2/5] selftest/futex: Update Makefile to use lib.mk

On Mon, 2015-03-30 at 09:31 -0700, Darren Hart wrote:

On 3/29/15, 4:49 PM, "Michael Ellerman" [off-list ref] wrote:

quoted

On Fri, 2015-03-27 at 15:17 -0700, Darren Hart wrote:

quoted

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

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.

OK, I can remove this (it was just part of the originals). Shall I resent
the whole series or just a follow-on patch?

Up to Shuah. It's a minor nit, so probably a follow-up patch is fine.

quoted

quoted

 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?

The standard logic wants to run each individual test and uses the echo
logic. My tests have their own pass/fail reporting mechanism. As I run
many options on the same tests and have pretty-printing logic, I prefer to
keep that in a run script rather than in a makefile.

I don't care for the echo logic as a mechanism, I think the reporting
should be part of the test. It also doesn't scale past pass/fail - like
xpass and xfail, for example.

Fare enuf.

cheers