]> www.infradead.org Git - users/willy/xarray.git/commitdiff
selftests: mptcp: fix fastclose with csum failure
authorPaolo Abeni <pabeni@redhat.com>
Mon, 13 Nov 2023 23:16:17 +0000 (00:16 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 15 Nov 2023 04:10:21 +0000 (20:10 -0800)
Running the mp_join selftest manually with the following command line:

  ./mptcp_join.sh -z -C

leads to some failures:

  002 fastclose server test
  # ...
  rtx                                 [fail] got 1 MP_RST[s] TX expected 0
  # ...
  rstrx                               [fail] got 1 MP_RST[s] RX expected 0

The problem is really in the wrong expectations for the RST checks
implied by the csum validation. Note that the same check is repeated
explicitly in the same test-case, with the correct expectation and
pass successfully.

Address the issue explicitly setting the correct expectation for
the failing checks.

Reported-by: Xiumei Mu <xmu@redhat.com>
Fixes: 6bf41020b72b ("selftests: mptcp: update and extend fastclose test-cases")
Cc: stable@vger.kernel.org
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Reviewed-by: Matthieu Baerts <matttbe@kernel.org>
Signed-off-by: Matthieu Baerts <matttbe@kernel.org>
Link: https://lore.kernel.org/r/20231114-upstream-net-20231113-mptcp-misc-fixes-6-7-rc2-v1-5-7b9cd6a7b7f4@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/mptcp/mptcp_join.sh

index 75a2438efdf3737c038f947075f3705ee6042d42..3c94f2f194d68188d8a0002779ba40c2ab0ef0eb 100755 (executable)
@@ -3240,7 +3240,7 @@ fastclose_tests()
        if reset_check_counter "fastclose server test" "MPTcpExtMPFastcloseRx"; then
                test_linkfail=1024 fastclose=server \
                        run_tests $ns1 $ns2 10.0.1.1
-               chk_join_nr 0 0 0
+               chk_join_nr 0 0 0 0 0 0 1
                chk_fclose_nr 1 1 invert
                chk_rst_nr 1 1
        fi