* [PATCH] selftests: memory-hotplug: fix emit_tests regression
@ 2018-03-01 20:16 Shuah Khan
2018-03-02 8:31 ` Anders Roxell
0 siblings, 1 reply; 3+ messages in thread
From: Shuah Khan @ 2018-03-01 20:16 UTC (permalink / raw)
To: shuah, naresh.kamboju
Cc: Shuah Khan, tglx, gregkh, pombredanne, lei.yang, linux-kselftest,
linux-kernel
Commit 16c513b13477
("selftests: memory-hotplug: silence test command echo")
introduced regression in emit_tests and results in the following
failure when selftests are installed and run. Fix it.
Running tests in memory-hotplug
========================================
./run_kselftest.sh: line 121: @./mem-on-off-test.sh: No such file or
directory
selftests: memory-hotplug [FAIL]
Fixes: 16c513b13477 (selftests: memory-hotplug: silence test command echo")
Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
---
tools/testing/selftests/memory-hotplug/Makefile | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tools/testing/selftests/memory-hotplug/Makefile b/tools/testing/selftests/memory-hotplug/Makefile
index 183b46883875..686da510f989 100644
--- a/tools/testing/selftests/memory-hotplug/Makefile
+++ b/tools/testing/selftests/memory-hotplug/Makefile
@@ -5,7 +5,8 @@ include ../lib.mk
TEST_PROGS := mem-on-off-test.sh
override RUN_TESTS := @./mem-on-off-test.sh -r 2 && echo "selftests: memory-hotplug [PASS]" || echo "selftests: memory-hotplug [FAIL]"
-override EMIT_TESTS := echo "$(RUN_TESTS)"
+
+override EMIT_TESTS := echo "$(subst @,,$(RUN_TESTS))"
run_full_test:
@/bin/bash ./mem-on-off-test.sh && echo "memory-hotplug selftests: [PASS]" || echo "memory-hotplug selftests: [FAIL]"
--
2.14.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] selftests: memory-hotplug: fix emit_tests regression
2018-03-01 20:16 [PATCH] selftests: memory-hotplug: fix emit_tests regression Shuah Khan
@ 2018-03-02 8:31 ` Anders Roxell
2018-03-02 17:17 ` Shuah Khan
0 siblings, 1 reply; 3+ messages in thread
From: Anders Roxell @ 2018-03-02 8:31 UTC (permalink / raw)
To: Shuah Khan
Cc: shuah, Naresh Kamboju, tglx, gregkh, Philippe Ombredanne,
lei.yang, linux-kselftest, Linux Kernel Mailing List
On 1 March 2018 at 21:16, Shuah Khan <shuahkh@osg.samsung.com> wrote:
> Commit 16c513b13477
> ("selftests: memory-hotplug: silence test command echo")
>
> introduced regression in emit_tests and results in the following
> failure when selftests are installed and run. Fix it.
>
> Running tests in memory-hotplug
> ========================================
> ./run_kselftest.sh: line 121: @./mem-on-off-test.sh: No such file or
> directory
> selftests: memory-hotplug [FAIL]
>
> Fixes: 16c513b13477 (selftests: memory-hotplug: silence test command echo")
> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
Tested-by: Anders Roxell <anders.roxell@linaro.org>
> ---
> tools/testing/selftests/memory-hotplug/Makefile | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/memory-hotplug/Makefile b/tools/testing/selftests/memory-hotplug/Makefile
> index 183b46883875..686da510f989 100644
> --- a/tools/testing/selftests/memory-hotplug/Makefile
> +++ b/tools/testing/selftests/memory-hotplug/Makefile
> @@ -5,7 +5,8 @@ include ../lib.mk
>
> TEST_PROGS := mem-on-off-test.sh
> override RUN_TESTS := @./mem-on-off-test.sh -r 2 && echo "selftests: memory-hotplug [PASS]" || echo "selftests: memory-hotplug [FAIL]"
> -override EMIT_TESTS := echo "$(RUN_TESTS)"
> +
> +override EMIT_TESTS := echo "$(subst @,,$(RUN_TESTS))"
>
> run_full_test:
> @/bin/bash ./mem-on-off-test.sh && echo "memory-hotplug selftests: [PASS]" || echo "memory-hotplug selftests: [FAIL]"
> --
> 2.14.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] selftests: memory-hotplug: fix emit_tests regression
2018-03-02 8:31 ` Anders Roxell
@ 2018-03-02 17:17 ` Shuah Khan
0 siblings, 0 replies; 3+ messages in thread
From: Shuah Khan @ 2018-03-02 17:17 UTC (permalink / raw)
To: Anders Roxell, Naresh Kamboju
Cc: shuah, tglx, gregkh, Philippe Ombredanne, lei.yang,
linux-kselftest, Linux Kernel Mailing List, Shuah Khan
On 03/02/2018 01:31 AM, Anders Roxell wrote:
> On 1 March 2018 at 21:16, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>> Commit 16c513b13477
>> ("selftests: memory-hotplug: silence test command echo")
>>
>> introduced regression in emit_tests and results in the following
>> failure when selftests are installed and run. Fix it.
>>
>> Running tests in memory-hotplug
>> ========================================
>> ./run_kselftest.sh: line 121: @./mem-on-off-test.sh: No such file or
>> directory
>> selftests: memory-hotplug [FAIL]
>>
>> Fixes: 16c513b13477 (selftests: memory-hotplug: silence test command echo")
>> Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
>> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
>
> Tested-by: Anders Roxell <anders.roxell@linaro.org>
>
Thanks for testing. It is now in linux-kselftest fixes for the next
4.16-rc
thanks,
-- Shuah
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-03-02 17:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-01 20:16 [PATCH] selftests: memory-hotplug: fix emit_tests regression Shuah Khan
2018-03-02 8:31 ` Anders Roxell
2018-03-02 17:17 ` Shuah Khan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).