Skip to content

Commit af8361b

Browse files
committed
selftests: mptcp: join: validate C-flag + def limit
JIRA: https://issues.redhat.com/browse/RHEL-115576 Upstream Status: net-next.git commit 008385e commit 008385e Author: Matthieu Baerts (NGI0) <matttbe@kernel.org> Date: Thu Sep 25 12:32:37 2025 +0200 selftests: mptcp: join: validate C-flag + def limit The previous commit adds an exception for the C-flag case. The 'mptcp_join.sh' selftest is extended to validate this case. In this subtest, there is a typical CDN deployment with a client where MPTCP endpoints have been 'automatically' configured: - the server set net.mptcp.allow_join_initial_addr_port=0 - the client has multiple 'subflow' endpoints, and the default limits: not accepting ADD_ADDRs. Without the parent patch, the client is not able to establish new subflows using its 'subflow' endpoints. The parent commit fixes that. The 'Fixes' tag here below is the same as the one from the previous commit: this patch here is not fixing anything wrong in the selftests, but it validates the previous fix for an issue introduced by this commit ID. Fixes: df377be ("mptcp: add deny_join_id0 in mptcp_options_received") Cc: stable@vger.kernel.org Reviewed-by: Geliang Tang <geliang@kernel.org> Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org> Link: https://patch.msgid.link/20250925-net-next-mptcp-c-flag-laminar-v1-2-ad126cc47c6b@kernel.org Signed-off-by: Jakub Kicinski <kuba@kernel.org> Signed-off-by: Davide Caratti <dcaratti@redhat.com>
1 parent 2fb703b commit af8361b

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

tools/testing/selftests/net/mptcp/mptcp_join.sh

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3178,6 +3178,17 @@ deny_join_id0_tests()
31783178
run_tests $ns1 $ns2 10.0.1.1
31793179
chk_join_nr 1 1 1
31803180
fi
3181+
3182+
# default limits, server deny join id 0 + signal
3183+
if reset_with_allow_join_id0 "default limits, server deny join id 0" 0 1; then
3184+
pm_nl_set_limits $ns1 0 2
3185+
pm_nl_set_limits $ns2 0 2
3186+
pm_nl_add_endpoint $ns1 10.0.2.1 flags signal
3187+
pm_nl_add_endpoint $ns2 10.0.3.2 flags subflow
3188+
pm_nl_add_endpoint $ns2 10.0.4.2 flags subflow
3189+
run_tests $ns1 $ns2 10.0.1.1
3190+
chk_join_nr 2 2 2
3191+
fi
31813192
}
31823193

31833194
fullmesh_tests()

0 commit comments

Comments
 (0)