mbox series

[net-next,00/11] mptcp: MIB counters for MPJ TX + misc improvements

Message ID 20240902-net-next-mptcp-mib-mpjtx-misc-v1-0-d3e0f3773b90@kernel.org (mailing list archive)
Headers show
Series mptcp: MIB counters for MPJ TX + misc improvements | expand

Message

Matthieu Baerts (NGI0) Sept. 2, 2024, 10:45 a.m. UTC
Recently, a few issues have been discovered around the creation of
additional subflows. Without these counters, it was difficult to point
out the reason why some subflows were not created as expected.

In patch 3, all error paths from __mptcp_subflow_connect() are covered,
except the one related to the 'fully established mode', because it can
only happen with the userspace PM, which will propagate the error to the
userspace in this case (ENOTCONN).

These new counters are also verified in the MPTCP Join selftest in patch
6.

While at it, a few other patches are improving the MPTCP path-manager
code ...

 - Patch 1: 'flush' related helpers are renamed to avoid confusions
 - Patch 2: directly pass known ID and flags to create a new subflow,
            i/o getting them later by iterating over all endpoints again

... and the MPJoin selftests:

 - Patch 4: reduce the number of positional parameters
 - Patch 5: only one line for the 'join' checks, instead of 3
 - Patch 7: more explicit check names, instead of sometimes too cryptic
            ones: rtx, ptx, ftx, ctx, fclzrx, sum
 - Patch 8: specify client/server instead of 'invert' for some checks
            not suggesting one specific direction
 - Patch 9: mute errors of mptcp_connect when ran in the background
 - Patch 10: simplify checksum_tests by using a for-loop
 - Patch 11: remove 'define' re-definitions

Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
---
Geliang Tang (1):
      selftests: mptcp: join: simplify checksum_tests

Matthieu Baerts (NGI0) (10):
      mptcp: pm: rename helpers linked to 'flush'
      mptcp: pm: reduce entries iterations on connect
      mptcp: MIB counters for sent MP_JOIN
      selftests: mptcp: join: reduce join_nr params
      selftests: mptcp: join: one line for join check
      selftests: mptcp: join: validate MPJ SYN TX MIB counters
      selftests: mptcp: join: more explicit check name
      selftests: mptcp: join: specify host being checked
      selftests: mptcp: join: mute errors when ran in the background
      selftests: mptcp: pm_nl_ctl: remove re-definition

 net/mptcp/mib.c                                 |   4 +
 net/mptcp/mib.h                                 |   4 +
 net/mptcp/pm.c                                  |  11 -
 net/mptcp/pm_netlink.c                          |  78 ++----
 net/mptcp/pm_userspace.c                        |  40 +--
 net/mptcp/protocol.h                            |  16 +-
 net/mptcp/subflow.c                             |  50 +++-
 tools/testing/selftests/net/mptcp/mptcp_join.sh | 350 ++++++++++++++----------
 tools/testing/selftests/net/mptcp/pm_nl_ctl.c   |  10 +-
 9 files changed, 309 insertions(+), 254 deletions(-)
---
base-commit: 221f9cce949ac8042f65b71ed1fde13b99073256
change-id: 20240902-net-next-mptcp-mib-mpjtx-misc-d80298438016

Best regards,

Comments

patchwork-bot+netdevbpf@kernel.org Sept. 3, 2024, 10:30 p.m. UTC | #1
Hello:

This series was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon, 02 Sep 2024 12:45:51 +0200 you wrote:
> Recently, a few issues have been discovered around the creation of
> additional subflows. Without these counters, it was difficult to point
> out the reason why some subflows were not created as expected.
> 
> In patch 3, all error paths from __mptcp_subflow_connect() are covered,
> except the one related to the 'fully established mode', because it can
> only happen with the userspace PM, which will propagate the error to the
> userspace in this case (ENOTCONN).
> 
> [...]

Here is the summary with links:
  - [net-next,01/11] mptcp: pm: rename helpers linked to 'flush'
    https://git.kernel.org/netdev/net-next/c/7bcf4d8022f9
  - [net-next,02/11] mptcp: pm: reduce entries iterations on connect
    https://git.kernel.org/netdev/net-next/c/b83fbca1b4c9
  - [net-next,03/11] mptcp: MIB counters for sent MP_JOIN
    https://git.kernel.org/netdev/net-next/c/1bd1788b6cab
  - [net-next,04/11] selftests: mptcp: join: reduce join_nr params
    https://git.kernel.org/netdev/net-next/c/1b2965a8cd8d
  - [net-next,05/11] selftests: mptcp: join: one line for join check
    https://git.kernel.org/netdev/net-next/c/ba8a664004da
  - [net-next,06/11] selftests: mptcp: join: validate MPJ SYN TX MIB counters
    https://git.kernel.org/netdev/net-next/c/004125c251a6
  - [net-next,07/11] selftests: mptcp: join: more explicit check name
    https://git.kernel.org/netdev/net-next/c/6ed495345be8
  - [net-next,08/11] selftests: mptcp: join: specify host being checked
    https://git.kernel.org/netdev/net-next/c/8d328dbcf61b
  - [net-next,09/11] selftests: mptcp: join: mute errors when ran in the background
    https://git.kernel.org/netdev/net-next/c/08eecd7e7fe7
  - [net-next,10/11] selftests: mptcp: join: simplify checksum_tests
    https://git.kernel.org/netdev/net-next/c/0e2b4584d61a
  - [net-next,11/11] selftests: mptcp: pm_nl_ctl: remove re-definition
    https://git.kernel.org/netdev/net-next/c/38dc0708bcc8

You are awesome, thank you!