On 10/18/19 12:56 AM, Prabhakar Kushwaha wrote:
On Fri, Oct 18, 2019 at 12:12 PM Liu Yiding <liuyd.fnst@xxxxxxxxxxxxxx> wrote:
CC LKP mainter
On 10/18/19 2:32 PM, Liu Yiding wrote:
Hi All.
The patch 5c069b6dedef "selftests: Move test output to diagnostic
lines" from Apr 24, 2019,
leads to `make run_tests -C bpf` hanging forever.
Bpf includes many subtest, when cmd `make run_tests -C bpf` runs to
test_lwt_seg6local.sh, task will hang and runner.sh never run next
task. I checked ps aux, prefix.pl will never exit.
```
91058 [ 811.451584] # [25] VAR __license type_id=24 linkage=1
91059 [ 811.451586]-
91060 [ 811.455365] # [26] DATASEC license size=0 vlen=1 size == 0
91061 [ 811.455367]-
91062 [ 811.457424] #-
91063 [ 811.457425]-
91064 [ 811.460912] # selftests: test_lwt_seg6local [PASS]
91065 [ 811.460914]-
91066 [ 3620.461986] Thu Oct 17 14:54:05 CST 2019 detected soft_timeout
```
Ignore test_lwt_seg6local and run `make run_tests -C bpf` again, task
will hang on test_tc_tunnel.sh.
Kushwaha also meet this issue, `make run_tests -C bpf` hang on
test_lwt_ip_encap.sh (This test failed on my localhost).
Yes, i also faced this issue on Linux-5.3. if i kill prefix.pl or add
timeout in prefix.pl, test move ahead.
somehow this issue is not present on Linux 5.4-rc3+. It could be by
chance and may show on any other version.
Kees added timeout in this commit and this went into 5.4-rc3, if recall
correctly.
commit 852c8cbf34d3b3130a05c38064dd98614f97d3a8
Author: Kees Cook <keescook@xxxxxxxxxxxx>
selftests/kselftest/runner.sh: Add 45 second timeout per test
Do you see a timed out message from this test though. The test needs
to fixed probably, but this timeout helps bail out.
thanks,
-- Shuah
