]> git.baikalelectronics.ru Git - kernel.git/commit
selftests: fix prepending $(OUTPUT) to $(TEST_PROGS)
authorIlya Leoshkevich <iii@linux.ibm.com>
Wed, 3 Mar 2021 00:44:20 +0000 (01:44 +0100)
committerShuah Khan <skhan@linuxfoundation.org>
Fri, 26 Mar 2021 17:29:37 +0000 (11:29 -0600)
commit375cf3252e64b9547ddb2bef31a8af2b00b8fec9
treee109aa5c6d8142db4ae91e967c1708c8ff687d15
parent219c9076286fd92026e818ab1375558e5b20649f
selftests: fix prepending $(OUTPUT) to $(TEST_PROGS)

Currently the following command produces an error message:

    linux# make kselftest TARGETS=bpf O=/mnt/linux-build
    # selftests: bpf: test_libbpf.sh
    # ./test_libbpf.sh: line 23: ./test_libbpf_open: No such file or directory
    # test_libbpf: failed at file test_l4lb.o
    # selftests: test_libbpf [FAILED]

The error message might not affect the return code of make, therefore
one needs to grep make output in order to detect it.

This is not the only instance of the same underlying problem; any test
with more than one element in $(TEST_PROGS) fails the same way. Another
example:

    linux# make O=/mnt/linux-build TARGETS=splice kselftest
    [...]
    # ./short_splice_read.sh: 15: ./splice_read: not found
    # FAIL: /sys/module/test_module/sections/.init.text 2
    not ok 2 selftests: splice: short_splice_read.sh # exit=1

The current logic prepends $(OUTPUT) only to the first member of
$(TEST_PROGS). After that, run_one() does

   cd `dirname $TEST`

For all tests except the first one, `dirname $TEST` is ., which means
they cannot access the files generated in $(OUTPUT).

Fix by using $(addprefix) to prepend $(OUTPUT)/ to each member of
$(TEST_PROGS).

Fixes: cb1bcd83d17d ("selftests: lib.mk: copy test scripts and test files for make O=dir run")
Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
tools/testing/selftests/lib.mk