(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? Thank you, -- Masami HIRAMATSU Software Platform Research Dept. Linux Technology Research Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu...@hitachi.com -- 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/
