]> git.baikalelectronics.ru Git - kernel.git/commit
ktest.pl: Add RUN_TIMEOUT option with default unlimited
authorSteven Rostedt <rostedt@goodmis.org>
Wed, 18 Jan 2023 21:37:25 +0000 (16:37 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Mar 2023 08:34:27 +0000 (09:34 +0100)
commit62623103f80a9b77538d90d4d2a9bd9c8fb8877e
tree6fd6bc0cfbdd8c36fb7108ac5ec2c7ecb41d8ae3
parent0a85dc604ab48f1b44cb7225a26824e323f741f9
ktest.pl: Add RUN_TIMEOUT option with default unlimited

commit 4112b34bf3fdcc9f30be13cf985a8913029b73ab upstream.

There is a disconnect between the run_command function and the
wait_for_input. The wait_for_input has a default timeout of 2 minutes. But
if that happens, the run_command loop will exit out to the waitpid() of
the executing command. This fails in that it no longer monitors the
command, and also, the ssh to the test box can hang when its finished, as
it's waiting for the pipe it's writing to to flush, but the loop that
reads that pipe has already exited, leaving the command stuck, and the
test hangs.

Instead, make the default "wait_for_input" of the run_command infinite,
and allow the user to override it if they want with a default timeout
option "RUN_TIMEOUT".

But this fixes the hang that happens when the pipe is full and the ssh
session never exits.

Cc: stable@vger.kernel.org
Fixes: 943e4212751c5 ("ktest: Add timeout to ssh command")
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
tools/testing/ktest/ktest.pl
tools/testing/ktest/sample.conf