MPTCP Linux Development
 help / color / mirror / Atom feed
From: Matthieu Baerts <matttbe@kernel.org>
To: Mat Martineau <martineau@kernel.org>
Cc: MPTCP Upstream <mptcp@lists.linux.dev>
Subject: Re: [PATCH mptcp-net v5 17/20] selftests: mptcp: join: validate 8x8 subflows
Date: Mon, 20 Apr 2026 19:30:15 +0200	[thread overview]
Message-ID: <d3b5d201-4e4c-4857-bbb2-5ce3ea265e63@kernel.org> (raw)
In-Reply-To: <54880733-bd55-a589-20bd-3806585c6e14@kernel.org>

Hi Mat,

On 18/04/2026 20:22, Mat Martineau wrote:
> On Wed, 15 Apr 2026, Matthieu Baerts (NGI0) wrote:
> 
>> The limits have been recently increased, it is required to validate that
>> having 64 subflows is allowed.
>>
>> Here, both the client and the server have 8 network interfaces. The
>> server has 8 endpoints marked as 'signal' to announce all its v4
>> addresses. The client also has 8 endpoints, but marked as 'subflow' and
>> 'fullmesh' in order to create 8 subflows to each address announced by
>> the server. This means 63 additional subflows will be created after the
>> initial one.
>>
>> If it is not possible to increase the limits to 64, it means an older
>> kernel version is being used, and the test is skipped.
>>
> 
> Does this have much of an impact on total time for the tests? Would it
> be worthwhile to test the larger limits on kernels that can handle it,
> and run the existing tests on older kernels?
> 
> In other words, the larger limit test seems to have redundant coverage
> with some older tests, so maybe skip the redundant tests on new kernels.

Good point. In terms of time, like most tests there, it waits for the
whole transfer, so the same time as other 'speed=slow' tests.

Surprisingly, we didn't have similar tests before setting multiple
'signal' endpoints on the server side, and multiple 'subflow + fullmesh'
on the client side. It was either one or the other. Plus we didn't check
with subflows up the previous limit (8), it was max 5. So no redundancy
here, I think. Or we disable the simple ones focussing on the client and
server side separately? (but then it no longer validates that on older
kernels)

Note that this test helped me find the issue that led to "mptcp: pm:
ADD_ADDR rtx: resched blocked ADD_ADDR quicker", which led to most of
the other Fixes patches :)

Cheers,
Matt
-- 
Sponsored by the NGI0 Core fund.


  reply	other threads:[~2026-04-20 17:30 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-15  9:56 [PATCH mptcp-net v5 00/20] mptcp: pm: increase limits, and related fixes and cleanup Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 01/20] mptcp: pm: kernel: correctly retransmit ADD_ADDR ID 0 Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 02/20] mptcp: pm: ADD_ADDR rtx: fix potential data-race Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 03/20] mptcp: pm: ADD_ADDR rtx: allow ID 0 Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 04/20] mptcp: pm: ADD_ADDR rtx: always decrease sk refcount Matthieu Baerts (NGI0)
2026-04-15 17:09   ` Matthieu Baerts
2026-04-15  9:56 ` [PATCH mptcp-net v5 05/20] mptcp: pm: ADD_ADDR rtx: free sk if last Matthieu Baerts (NGI0)
2026-04-18 18:00   ` Mat Martineau
2026-04-20 17:12     ` Matthieu Baerts
2026-04-20 21:07       ` Mat Martineau
2026-04-15  9:56 ` [PATCH mptcp-net v5 06/20] mptcp: pm: ADD_ADDR rtx: resched blocked ADD_ADDR quicker Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 07/20] mptcp: pm: ADD_ADDR rtx: skip inactive subflows Matthieu Baerts (NGI0)
2026-04-18 18:09   ` Mat Martineau
2026-04-20 17:18     ` Matthieu Baerts
2026-04-20 21:07       ` Mat Martineau
2026-04-15  9:56 ` [PATCH mptcp-net v5 08/20] mptcp: pm: retrans ADD_ADDR: return early if no retrans Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 09/20] mptcp: pm: prio: skip closed subflows Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 10/20] selftests: mptcp: check output: catch cmd errors Matthieu Baerts (NGI0)
2026-04-15  9:56 ` [PATCH mptcp-net v5 11/20] selftests: mptcp: pm: restrict 'unknown' check to pm_nl_ctl Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 12/20] mptcp: pm: in-kernel: explicitly limit batches to array size Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 13/20] mptcp: pm: in-kernel: increase all limits to 64 Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 14/20] mptcp: pm: kernel: allow flushing more than 8 endpoints Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 15/20] mptcp: pm: in-kernel: increase endpoints limit Matthieu Baerts (NGI0)
2026-04-15 17:10   ` Matthieu Baerts
2026-04-15  9:57 ` [PATCH mptcp-net v5 16/20] selftests: mptcp: join: allow changing ifaces nr per test Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 17/20] selftests: mptcp: join: validate 8x8 subflows Matthieu Baerts (NGI0)
2026-04-18 18:22   ` Mat Martineau
2026-04-20 17:30     ` Matthieu Baerts [this message]
2026-04-20 21:11       ` Mat Martineau
2026-04-15  9:57 ` [PATCH mptcp-net v5 18/20] selftests: mptcp: pm: validate new limits Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 19/20] selftests: mptcp: pm: use simpler send/recv forms Matthieu Baerts (NGI0)
2026-04-15  9:57 ` [PATCH mptcp-net v5 20/20] mptcp: pm: clearer ADD_ADDR related helpers names Matthieu Baerts (NGI0)
2026-04-18 18:27   ` Mat Martineau
2026-04-20 17:50     ` Matthieu Baerts
2026-04-15 11:32 ` [PATCH mptcp-net v5 00/20] mptcp: pm: increase limits, and related fixes and cleanup MPTCP CI

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d3b5d201-4e4c-4857-bbb2-5ce3ea265e63@kernel.org \
    --to=matttbe@kernel.org \
    --cc=martineau@kernel.org \
    --cc=mptcp@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox