| Message ID | 20200616211106.78251-1-luc.vanoostenryck@gmail.com (mailing list archive) |
|---|---|
| State | Mainlined, archived |
| Headers | show |
| Series | testsuite: be less paranoid with timeout | expand |
diff --git a/validation/test-suite b/validation/test-suite index 64a3e08fb4dd..a22f70135fb3 100755 --- a/validation/test-suite +++ b/validation/test-suite @@ -338,7 +338,7 @@ do_test() # do we want a timeout? pre_cmd="" if [ $check_timeout -ne 0 ]; then - pre_cmd="timeout -k 1s $check_timeout" + pre_cmd="timeout $check_timeout" fi shift
For some testcases, the testsuite use the command 'timeout' to ensure that the test finish after a reasonable amount of time. This is mainly used for some testcases which, in the past, were stuck in an infinite loop. This the command 'timeout' is used with an extra option (-k 1s) to issue a second kill signal in case the first one would have been ignored. However, this extra option is not supported on all implementations (Alpine) and its use seems a bit paranoid for sparse. So, remove this extra option. Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@gmail.com> --- validation/test-suite | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)