On 11/26/2014 10:49 PM, Masami Hiramatsu wrote: > (2014/11/12 5:27), Shuah Khan wrote: >> 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 source dir. >> >> Signed-off-by: Shuah Khan <shua...@osg.samsung.com> >> --- >> tools/testing/selftests/memory-hotplug/Makefile | 17 +- >> .../selftests/memory-hotplug/mem-on-off-test.sh | 238 >> +++++++++++++++++++++ >> .../selftests/memory-hotplug/on-off-test.sh | 238 >> --------------------- >> 3 files changed, 253 insertions(+), 240 deletions(-) >> create mode 100644 tools/testing/selftests/memory-hotplug/mem-on-off-test.sh >> delete mode 100644 tools/testing/selftests/memory-hotplug/on-off-test.sh >> >> diff --git a/tools/testing/selftests/memory-hotplug/Makefile >> b/tools/testing/selftests/memory-hotplug/Makefile >> index d46b8d4..8921631 100644 >> --- a/tools/testing/selftests/memory-hotplug/Makefile >> +++ b/tools/testing/selftests/memory-hotplug/Makefile >> @@ -1,9 +1,22 @@ >> +TEST_STR=/bin/bash ./mem-on-off-test.sh -r 2 || echo memory-hotplug >> selftests: [FAIL] >> + >> all: >> >> +install: >> +ifdef INSTALL_KSFT_PATH >> + install ./mem-on-off-test.sh $(INSTALL_KSFT_PATH)/mem-on-off-test.sh >> + @echo echo Start memory hotplug test .... >> $(KSELFTEST) >> + @echo "$(TEST_STR)" >> $(KSELFTEST) >> $(KSELFTEST) >> + @echo echo End memory hotplug test .... >> $(KSELFTEST) >> + @echo echo ============================== >> $(KSELFTEST) >> +else >> + @echo Run make kselftest_install in top level source directory >> +endif > > I saw this pattern repeated many times in this series. > Can we make it a macro and include it instead of repeating this code? >
I might be able to move this to the selftest Makfile and use target as the string. I will play with it and see if I can reduce the duplication further. thanks, -- Shuah -- Shuah Khan Sr. Linux Kernel Developer Samsung Research America (Silicon Valley) shua...@osg.samsung.com | (970) 217-8978 -- 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/
