summaryrefslogtreecommitdiff
path: root/net
diff options
context:
space:
mode:
authorGuillaume Nault <gnault@redhat.com>2022-03-09 01:15:03 +0300
committerJakub Kicinski <kuba@kernel.org>2022-03-10 07:23:32 +0300
commit94a4a4fe4c696413932eed8bdec46574de9576b8 (patch)
tree7b851b74e05a66adbadc2e0426975a5e0f1a0e61 /net
parent18dfc667550fe9c032a6dcc3402b50e691e18029 (diff)
downloadlinux-94a4a4fe4c696413932eed8bdec46574de9576b8.tar.xz
selftests: pmtu.sh: Kill nettest processes launched in subshell.
When using "run_cmd <command> &", then "$!" refers to the PID of the subshell used to run <command>, not the command itself. Therefore nettest_pids actually doesn't contain the list of the nettest commands running in the background. So cleanup() can't kill them and the nettest processes run until completion (fortunately they have a 5s timeout). Fix this by defining a new command for running processes in the background, for which "$!" really refers to the PID of the command run. Also, double quote variables on the modified lines, to avoid shellcheck warnings. Fixes: ece1278a9b81 ("selftests: net: add ESP-in-UDP PMTU test") Signed-off-by: Guillaume Nault <gnault@redhat.com> Reviewed-by: Shuah Khan <skhan@linuxfoundation.org> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net')
0 files changed, 0 insertions, 0 deletions