diff options
author | Geliang Tang <geliang.tang@suse.com> | 2023-10-26 02:37:03 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-11-20 13:59:38 +0300 |
commit | e8b45937c82d51dc133e015e5e9423f8bd86a668 (patch) | |
tree | 4989797660088b71785aaca1f7b859501f404afb /net/wireless/util.c | |
parent | 1c6d07dd699801b327d2e2db541d22cc690987f1 (diff) | |
download | linux-e8b45937c82d51dc133e015e5e9423f8bd86a668.tar.xz |
selftests: mptcp: fix wait_rm_addr/sf parameters
commit 9168ea02b898d3dde98b51e4bd3fb082bd438dab upstream.
The second input parameter of 'wait_rm_addr/sf $1 1' is misused. If it's
1, wait_rm_addr/sf will never break, and will loop ten times, then
'wait_rm_addr/sf' equals to 'sleep 1'. This delay time is too long,
which can sometimes make the tests fail.
A better way to use wait_rm_addr/sf is to use rm_addr/sf_count to obtain
the current value, and then pass into wait_rm_addr/sf.
Fixes: 4369c198e599 ("selftests: mptcp: test userspace pm out of transfer")
Cc: stable@vger.kernel.org
Suggested-by: Matthieu Baerts <matttbe@kernel.org>
Reviewed-by: Matthieu Baerts <matttbe@kernel.org>
Signed-off-by: Geliang Tang <geliang.tang@suse.com>
Signed-off-by: Mat Martineau <martineau@kernel.org>
Link: https://lore.kernel.org/r/20231025-send-net-next-20231025-v1-2-db8f25f798eb@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/wireless/util.c')
0 files changed, 0 insertions, 0 deletions