2024-01-02 14:16:48

by Laura Nao

Subject: [PATCH] selftests: Move KTAP bash helpers to selftests common folder

Move bash helpers for outputting in KTAP format to the common selftests
folder. This allows kselftests other than the dt one to source the file
and make use of the helper functions.
Define pass, fail and skip codes in the same file too.

Signed-off-by: Laura Nao <[email protected]>
---
tools/testing/selftests/Makefile | 1 +
tools/testing/selftests/dt/Makefile | 2 +-
tools/testing/selftests/dt/test_unprobed_devices.sh | 6 +-----
tools/testing/selftests/{dt => kselftest}/ktap_helpers.sh | 6 ++++++
4 files changed, 9 insertions(+), 6 deletions(-)
rename tools/testing/selftests/{dt => kselftest}/ktap_helpers.sh (94%)

diff --git a/tools/testing/selftests/Makefile b/tools/testing/selftests/Makefile
index 3b2061d1c1a5..976e96013c91 100644
--- a/tools/testing/selftests/Makefile
+++ b/tools/testing/selftests/Makefile
@@ -251,6 +251,7 @@ ifdef INSTALL_PATH
install -m 744 kselftest/module.sh $(INSTALL_PATH)/kselftest/
install -m 744 kselftest/runner.sh $(INSTALL_PATH)/kselftest/
install -m 744 kselftest/prefix.pl $(INSTALL_PATH)/kselftest/
+ install -m 744 kselftest/ktap_helpers.sh $(INSTALL_PATH)/kselftest/
install -m 744 run_kselftest.sh $(INSTALL_PATH)/
rm -f $(TEST_LIST)
@ret=1; \
diff --git a/tools/testing/selftests/dt/Makefile b/tools/testing/selftests/dt/Makefile
index 62dc00ee4978..2d33ee9e9b71 100644
--- a/tools/testing/selftests/dt/Makefile
+++ b/tools/testing/selftests/dt/Makefile
@@ -4,7 +4,7 @@ ifneq ($(PY3),)

TEST_PROGS := test_unprobed_devices.sh
TEST_GEN_FILES := compatible_list
-TEST_FILES := compatible_ignore_list ktap_helpers.sh
+TEST_FILES := compatible_ignore_list

include ../lib.mk

diff --git a/tools/testing/selftests/dt/test_unprobed_devices.sh b/tools/testing/selftests/dt/test_unprobed_devices.sh
index b07af2a4c4de..f2307ee443a6 100755
--- a/tools/testing/selftests/dt/test_unprobed_devices.sh
+++ b/tools/testing/selftests/dt/test_unprobed_devices.sh
@@ -15,16 +15,12 @@

DIR="$(dirname $(readlink -f "$0"))"

-source "${DIR}"/ktap_helpers.sh
+source "${DIR}"/../kselftest/ktap_helpers.sh

PDT=/proc/device-tree/
COMPAT_LIST="${DIR}"/compatible_list
IGNORE_LIST="${DIR}"/compatible_ignore_list

-KSFT_PASS=0
-KSFT_FAIL=1
-KSFT_SKIP=4
-
ktap_print_header

if [[ ! -d "${PDT}" ]]; then
diff --git a/tools/testing/selftests/dt/ktap_helpers.sh b/tools/testing/selftests/kselftest/ktap_helpers.sh
similarity index 94%
rename from tools/testing/selftests/dt/ktap_helpers.sh
rename to tools/testing/selftests/kselftest/ktap_helpers.sh
index 8dfae51bb4e2..dd79d96f3b5a 100644
--- a/tools/testing/selftests/dt/ktap_helpers.sh
+++ b/tools/testing/selftests/kselftest/ktap_helpers.sh
@@ -9,6 +9,12 @@ KTAP_CNT_PASS=0
KTAP_CNT_FAIL=0
KTAP_CNT_SKIP=0

+KSFT_PASS=0
+KSFT_FAIL=1
+KSFT_XFAIL=2
+KSFT_XPASS=3
+KSFT_SKIP=4
+
ktap_print_header() {
echo "TAP version 13"
}
--
2.30.2

2024-01-02 20:33:05

by Nícolas F. R. A. Prado

Subject: Re: [PATCH] selftests: Move KTAP bash helpers to selftests common folder

On Tue, Jan 02, 2024 at 03:15:28PM +0100, Laura Nao wrote:
> Move bash helpers for outputting in KTAP format to the common selftests
> folder. This allows kselftests other than the dt one to source the file
> and make use of the helper functions.
> Define pass, fail and skip codes in the same file too.
>
> Signed-off-by: Laura Nao <[email protected]>

Reviewed-by: N?colas F. R. A. Prado <[email protected]>
Tested-by: N?colas F. R. A. Prado <[email protected]>

Thanks,
N?colas

2024-01-22 17:17:11

by Rob Herring (Arm)

Subject: Re: [PATCH] selftests: Move KTAP bash helpers to selftests common folder

On Tue, Jan 02, 2024 at 03:15:28PM +0100, Laura Nao wrote:
> Move bash helpers for outputting in KTAP format to the common selftests
> folder. This allows kselftests other than the dt one to source the file
> and make use of the helper functions.
> Define pass, fail and skip codes in the same file too.
>
> Signed-off-by: Laura Nao <[email protected]>
> ---
> tools/testing/selftests/Makefile | 1 +
> tools/testing/selftests/dt/Makefile | 2 +-
> tools/testing/selftests/dt/test_unprobed_devices.sh | 6 +-----
> tools/testing/selftests/{dt => kselftest}/ktap_helpers.sh | 6 ++++++
> 4 files changed, 9 insertions(+), 6 deletions(-)
> rename tools/testing/selftests/{dt => kselftest}/ktap_helpers.sh (94%)

Acked-by: Rob Herring <[email protected]>

2024-02-12 15:01:50

by Nícolas F. R. A. Prado

Subject: Re: [PATCH] selftests: Move KTAP bash helpers to selftests common folder

On Tue, Jan 02, 2024 at 03:15:28PM +0100, Laura Nao wrote:
> Move bash helpers for outputting in KTAP format to the common selftests
> folder. This allows kselftests other than the dt one to source the file
> and make use of the helper functions.
> Define pass, fail and skip codes in the same file too.
>
> Signed-off-by: Laura Nao <[email protected]>
> ---

Hi Shuah, any thoughts on this patch?

It's gotten Rob's ack, so I take this will be merged through your tree.

Thanks,
N?colas

2024-02-20 22:59:49

by Shuah Khan

Subject: Re: [PATCH] selftests: Move KTAP bash helpers to selftests common folder

On 2/12/24 08:01, Nícolas F. R. A. Prado wrote:
> On Tue, Jan 02, 2024 at 03:15:28PM +0100, Laura Nao wrote:
>> Move bash helpers for outputting in KTAP format to the common selftests
>> folder. This allows kselftests other than the dt one to source the file
>> and make use of the helper functions.
>> Define pass, fail and skip codes in the same file too.
>>
>> Signed-off-by: Laura Nao <[email protected]>
>> ---
>
> Hi Shuah, any thoughts on this patch?
>
> It's gotten Rob's ack, so I take this will be merged through your tree.
>
> Thanks,
> Nícolas

Applied to next for Linux 6.9-rc1

thanks,
-- Shuah