Hello, On 12/08/2026 16:55, Matthieu Baerts (NGI0) wrote: > This series contains a few independent new features, and small fixes for > net-next:
I checked Sashiko [1] and Clashiko [2] reviews, and here below is a brief summary. TL;DR: I don't think a v2 is needed, and follow-up patches fixing pre-existing issues will be sent separately. [1] https://sashiko.dev/#/patchset/20260812-net-next-mptcp-misc-feat-7-3-v1-0-1905a818f...@kernel.org [2] https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260812-net-next-mptcp-misc-feat-7-3-v1-0-1905a818f...@kernel.org > - Patch 1: Add WARN_ON_ONCE guards around extra_subflows to catch issues > with this counter, similar to what is done with other PM counters. Pre-existing issues: fixes will be sent to net. > - Patches 2-3: Follow-up patches to remove data_ack field from struct > mptcp_ext -- now unused after recent fixes -- and makes a userspace PM > helper static. Pre-existing issue: a fix will be sent to net. > - Patch 4: Honour tcp_rto_{min_us,max_ms} sysctls for MPTCP-level > retransmit timers like with DATA_FIN's and fallback timeout. Not needed: suggesting adding extra checks if the TCP RTO max and min sysctl are misconfigured (min > max). There is already a check in the code to avoid a division by 0. The rest is similar to TCP. > - Patches 5-6: Add per-event MIB counters for MPTCP_RST_EMPTCP resets to > help to spot such situations in production. Patch 5: some imprecisions in the commit message, not worth a respin I think. Plus a suggestion of adding an extra increment, but it is for a different case, so no. Patch 6: no need to handle the only test where the server initiates the subflow → the goal is to check that globally these counters are not unexpetedly incremented. Maybe MPJoinNotEstablished could be checked on the other side as well, but this shouldn't happen with the current selftests, and this addition can come in a follow-up patch. > - Patches 7-9: Small pcap-related improvements in the selftests. Patch 7: no need to differenciate the mptcp_connect_*.sh which are only there for the CI. Someone doing the debugging and asking for pcaps will use mptcp_connect.sh > - Patch 10: Fix compiler warning in the selftests. pre-existing, and not really an issue: the selftests don't try to trick mptcp_connect > - Patch 11: Avoid a buffer overflow when misusing the mptcp_diag tool > from the selftests. Small typo in the commit message, not worth a respin I think. Plus yes it is a fix, but not worth being backported: can be a bug when this tool designed for the selftest is used manually with a wrong address (which is written in the commit message). Plus pre-existing issues that are not worth fixing in a follow-up patch for this selftest tool. Cheers, Matt -- Sponsored by the NGI0 Core fund.
