Re: [PATCH v3 05/20] selftests/ftrace: add install target to enable test install
flat view
From: Steven Rostedt <rostedt@goodmis.org>
Date: 2015-01-02 15:45:35
Also in:
linux-kbuild, lkml, netdev
On Wed, 24 Dec 2014 09:27:41 -0700 Shuah Khan [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Add a new make target to enable installing test. This target installs test in the kselftest install location and add to the kselftest script to run the test. Install target can be run only from top level kernel source directory. Signed-off-by: Shuah Khan <redacted> --- tools/testing/selftests/ftrace/Makefile | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)diff --git a/tools/testing/selftests/ftrace/Makefile b/tools/testing/selftests/ftrace/Makefile index 76cc9f1..7c7cf42 100644 --- a/tools/testing/selftests/ftrace/Makefile +++ b/tools/testing/selftests/ftrace/Makefile@@ -1,7 +1,16 @@ +TEST_STR = /bin/sh ./ftracetest || echo ftrace selftests: [FAIL]
Is it ok that this removes the quotes around the echo string? I don't see anything wrong about it, but I don't know if there's a shell out there that will fail due to it. Other than than, Acked-by: Steven Rostedt <rostedt@goodmis.org> -- Steve
quoted hunk ↗ jump to hunk
+ all: +install: +ifdef INSTALL_KSFT_PATH + install ./ftracetest $(INSTALL_KSFT_PATH) + @cp -r test.d $(INSTALL_KSFT_PATH) + echo "$(TEST_STR)" >> $(KSELFTEST) +endif + run_tests: - @/bin/sh ./ftracetest || echo "ftrace selftests: [FAIL]" + @$(TEST_STR) clean: rm -rf logs/*