[PATCH net-next 3/3] selftests: mptcp: reset the last TS before the first test

2024-09-02 Thread Matthieu Baerts (NGI0)
-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_connect.sh | 2 ++ tools/testing/selftests/net/mptcp/mptcp_join.sh| 3 ++- tools/testing/selftests/net/mptcp/mptcp_sockopt.sh | 1 + tools/testing/selftests/net/mptcp/pm_netlink.sh| 2 ++ tools/testing/selftests/net

[PATCH net-next 2/3] sefltests: mptcp: connect: remote time in TAP output

2024-09-02 Thread Matthieu Baerts (NGI0)
It is now added by the MPTCP lib automatically, see the parent commit. The time in the TAP output might be sligly different from the one displayed before, but that's OK. Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_connect.s

[PATCH net-next 1/3] selftests: mptcp: lib: add time per subtests in TAP output

2024-09-02 Thread Matthieu Baerts (NGI0)
l.org/dev-tools/ktap.html [3] Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_lib.sh | 17 - 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_lib.sh b/tools/testing/self

[PATCH net-next 0/3] selftests: mptcp: add time per subtests in TAP output

2024-09-02 Thread Matthieu Baerts (NGI0)
proves the precision of the first subtests in all MPTCP subtests. Link: https://github.com/linux-netdev/nipa/pull/36 Signed-off-by: Matthieu Baerts (NGI0) --- Matthieu Baerts (NGI0) (3): selftests: mptcp: lib: add time per subtests in TAP output sefltests: mptcp: connect: remote time in

[PATCH net-next 11/11] selftests: mptcp: pm_nl_ctl: remove re-definition

2024-09-02 Thread Matthieu Baerts (NGI0)
, and drop the other one. Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/pm_nl_ctl.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/pm_nl_ctl.c b/tools/testing/selftests/n

[PATCH net-next 10/11] selftests: mptcp: join: simplify checksum_tests

2024-09-02 Thread Matthieu Baerts (NGI0)
From: Geliang Tang The four checksum tests are similar, only one line is different. So a for-loop can be used to simplify these tests. Signed-off-by: Geliang Tang Reviewed-by: Matthieu Baerts (NGI0) Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh

[PATCH net-next 09/11] selftests: mptcp: join: mute errors when ran in the background

2024-09-02 Thread Matthieu Baerts (NGI0)
wed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 36 - 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/selftests/net/mptcp/mptc

[PATCH net-next 08/11] selftests: mptcp: join: specify host being checked

2024-09-02 Thread Matthieu Baerts (NGI0)
sending it, which is more unusual. Also before, the 'invert' message was printed after a few checks, but it was not clear which ones exactly. Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 89 +

[PATCH net-next 07/11] selftests: mptcp: join: more explicit check name

2024-09-02 Thread Matthieu Baerts (NGI0)
Before, the check names had to be very short. It is no longer the case now that these checks are printed on a dedicated line. Then, it looks better to have more explicit names. Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh

[PATCH net-next 06/11] selftests: mptcp: join: validate MPJ SYN TX MIB counters

2024-09-02 Thread Matthieu Baerts (NGI0)
there are dropped, or there are errors. While at it, the "no MPC reuse with single endpoint" subtest has been modified to force a bind() error. Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 89 ++

[PATCH net-next 05/11] selftests: mptcp: join: one line for join check

2024-09-02 Thread Matthieu Baerts (NGI0)
reported like before. Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 45 - 1 file changed, 30 insertions(+), 15 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing

[PATCH net-next 04/11] selftests: mptcp: join: reduce join_nr params

2024-09-02 Thread Matthieu Baerts (NGI0)
\ join_corrupted_pkts=4 \ chk_join_nr 1 1 1 It will then be easier to add new optional parameters. Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 31 ++--- 1 file changed, 22 insertions(+), 9 deletions(-) diff --git a

[PATCH net-next 03/11] mptcp: MIB counters for sent MP_JOIN

2024-09-02 Thread Matthieu Baerts (NGI0)
Baerts (NGI0) --- net/mptcp/mib.c | 4 net/mptcp/mib.h | 4 net/mptcp/subflow.c | 21 ++--- 3 files changed, 26 insertions(+), 3 deletions(-) diff --git a/net/mptcp/mib.c b/net/mptcp/mib.c index 7884217f33eb..ec0d461cb921 100644 --- a/net/mptcp/mib.c +++ b/net/mptcp

[PATCH net-next 02/11] mptcp: pm: reduce entries iterations on connect

2024-09-02 Thread Matthieu Baerts (NGI0)
, to avoid having to set it in the 'rebuild_header' hook, which will cause a new iteration of the endpoint entries. Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm.c | 11 net/mptcp/pm_netlink.c | 66 ++---

[PATCH net-next 01/11] mptcp: pm: rename helpers linked to 'flush'

2024-09-02 Thread Matthieu Baerts (NGI0)
. Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index f891bc714668..275959581586 100644 --- a/net/mptcp/pm_netlink.c

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

2024-09-02 Thread Matthieu Baerts (NGI0)
me 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) --- Geliang Tang (1):

[PATCH net v2 15/15] selftests: mptcp: join: check re-re-adding ID 0 signal

2024-08-27 Thread Matthieu Baerts (NGI0)
e introduced by this commit ID. Fixes: d0876b2284cf ("mptcp: add the incoming RM_ADDR support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- v2: - Make the connection longer, to have time to finish all the endpoints manipulations wh

[PATCH net v2 14/15] mptcp: pm: ADD_ADDR 0 is not a new address

2024-08-27 Thread Matthieu Baerts (NGI0)
plicated subflows created by the client. Fixes: d0876b2284cf ("mptcp: add the incoming RM_ADDR support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm.c | 4 +++- net/mptcp/pm_netlink.c | 9 + net/mptcp/proto

[PATCH net v2 13/15] selftests: mptcp: join: validate event numbers

2024-08-27 Thread Matthieu Baerts (NGI0)
add netlink event support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 74 - tools/testing/selftests/net/mptcp/mptcp_lib.sh | 4 ++ 2 files changed, 75 insertions(+), 3

[PATCH net v2 12/15] mptcp: avoid duplicated SUB_CLOSED events

2024-08-27 Thread Matthieu Baerts (NGI0)
7dc7 ("mptcp: add netlink event support") Cc: sta...@vger.kernel.org Tested-by: Arınç ÜNAL Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/protocol.c | 6 ++ net/mptcp/protocol.h | 3 ++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --g

[PATCH net v2 11/15] selftests: mptcp: join: check re-re-adding ID 0 endp

2024-08-27 Thread Matthieu Baerts (NGI0)
ssue introduced by this commit ID. Fixes: 3ad14f54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- v2: - Make the connection longer, to have time to finish all the endpoints manipula

[PATCH net v2 10/15] mptcp: pm: fix ID 0 endp usage after multiple re-creations

2024-08-27 Thread Matthieu Baerts (NGI0)
vger.kernel.org Tested-by: Arınç ÜNAL Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 3ff273e219f2..a93450ded50a 100644 --

[PATCH net v2 09/15] mptcp: pm: do not remove already closed subflows

2024-08-27 Thread Matthieu Baerts (NGI0)
;) Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 5a84a55e37cc..3ff273e219f2 100644 --- a/net/mptcp/pm_netlink.c +++ b/net/mptcp/pm

[PATCH net v2 08/15] selftests: mptcp: join: no extra msg if no counter

2024-08-27 Thread Matthieu Baerts (NGI0)
;) Cc: sta...@vger.kernel.org Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/sel

[PATCH net v2 07/15] selftests: mptcp: join: check re-adding init endp with != id

2024-08-27 Thread Matthieu Baerts (NGI0)
54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git

[PATCH net v2 06/15] mptcp: pm: reset MPC endp ID when re-added

2024-08-27 Thread Matthieu Baerts (NGI0)
int, the same variable is updated if the address match the one of the initial subflow. Fixes: 3ad14f54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 13 +++

[PATCH net v2 05/15] mptcp: pm: skip connecting to already established sf

2024-08-27 Thread Matthieu Baerts (NGI0)
to be, but not the ones being closed. Fixes: d84ad04941c3 ("mptcp: skip connecting the connected address") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletion

[PATCH net v2 04/15] mptcp: pm: send ACK on an active subflow

2024-08-27 Thread Matthieu Baerts (NGI0)
Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 42d4e7b5f65d..ed2205ef7208 100644 --- a/net/mptcp/pm_netlink.c +++ b/net/mptcp/pm_netlink.c @@ -765,9

[PATCH net v2 03/15] selftests: mptcp: join: check removing ID 0 endpoint

2024-08-27 Thread Matthieu Baerts (NGI0)
is not fixing anything wrong in the selftests, but it validates the previous fix for an issue introduced by this commit ID. Fixes: 3ad14f54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (

[PATCH net v2 02/15] mptcp: pm: fix RM_ADDR ID for the initial subflow

2024-08-27 Thread Matthieu Baerts (NGI0)
") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 28 +++- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index ec45ab4c66ab..42d4

[PATCH net v2 01/15] mptcp: pm: reuse ID 0 after delete and re-add

2024-08-27 Thread Matthieu Baerts (NGI0)
cking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 8d2f97854c64..ec45ab4c66ab 100644 --- a

[PATCH net v2 00/15] mptcp: more fixes for the in-kernel PM

2024-08-27 Thread Matthieu Baerts (NGI0)
low -- has been reached. A fix for v5.10. Patch 15 validates the previous patch. Signed-off-by: Matthieu Baerts (NGI0) --- Changes in v2: - Patches 11,15/15: allow the connection to run for longer, should fix the issue seen on the Netdev CI, with a debug kconfig. - Link to v1: https://lore.ke

[PATCH net 4/4] mptcp: pr_debug: add missing \n at the end

2024-08-26 Thread Matthieu Baerts (NGI0)
ug(\".*[^n]\)\(\"[,)]\)/\1\\\n\2/g" So in case of conflicts, simply drop the modifications, and launch this command. Fixes: f870fa0b5768 ("mptcp: Add MPTCP socket stubs") Cc: sta...@vger.kernel.org Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- ne

[PATCH net 3/4] mptcp: sched: check both backup in retrans

2024-08-26 Thread Matthieu Baerts (NGI0)
ing: the 'backup' flag was set by accident instead of the 'request_bkup' one. Now that the latter has been fixed, get_retrans() needs to be adapted as well. Fixes: b6a66e521a20 ("mptcp: sched: check both directions for backup") Cc: sta...@vger.kernel.org Reviewed-by: Mat

[PATCH net 2/4] selftests: mptcp: join: cannot rm sf if closed

2024-08-26 Thread Matthieu Baerts (NGI0)
pm out of transfer") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b

[PATCH net 1/4] mptcp: close subflow when receiving TCP+FIN

2024-08-26 Thread Matthieu Baerts (NGI0)
ing a FIN+ACK in return. Fixes: 40947e13997a ("mptcp: schedule worker when subflow is closed") Cc: sta...@vger.kernel.org Link: https://github.com/multipath-tcp/packetdrill/pull/154 [1] Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/protocol.c | 5 +

[PATCH net 0/4] mptcp: close subflow when receiving TCP+FIN and misc.

2024-08-26 Thread Matthieu Baerts (NGI0)
can follow the 'Fixes' commit and be backported up to v5.7. Patch 4 adds a missing \n at the end of pr_debug(), causing debug messages to be displayed with a delay, which confuses the debugger. A fix for v5.6. Signed-off-by: Matthieu Baerts (NGI0) --- Note: Peter's email address h

[PATCH net 15/15] selftests: mptcp: join: check re-re-adding ID 0 signal

2024-08-26 Thread Matthieu Baerts (NGI0)
e introduced by this commit ID. Fixes: d0876b2284cf ("mptcp: add the incoming RM_ADDR support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 30 - 1 file changed,

[PATCH net 14/15] mptcp: pm: ADD_ADDR 0 is not a new address

2024-08-26 Thread Matthieu Baerts (NGI0)
plicated subflows created by the client. Fixes: d0876b2284cf ("mptcp: add the incoming RM_ADDR support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm.c | 4 +++- net/mptcp/pm_netlink.c | 9 + net/mptcp/proto

[PATCH net 13/15] selftests: mptcp: join: validate event numbers

2024-08-26 Thread Matthieu Baerts (NGI0)
add netlink event support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 74 - tools/testing/selftests/net/mptcp/mptcp_lib.sh | 4 ++ 2 files changed, 75 insertions(+), 3

[PATCH net 12/15] mptcp: avoid duplicated SUB_CLOSED events

2024-08-26 Thread Matthieu Baerts (NGI0)
7dc7 ("mptcp: add netlink event support") Cc: sta...@vger.kernel.org Tested-by: Arınç ÜNAL Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/protocol.c | 6 ++ net/mptcp/protocol.h | 3 ++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --g

[PATCH net 11/15] selftests: mptcp: join: check re-re-adding ID 0 endp

2024-08-26 Thread Matthieu Baerts (NGI0)
ssue introduced by this commit ID. Fixes: 3ad14f54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 23 +-- 1 file

[PATCH net 10/15] mptcp: pm: fix ID 0 endp usage after multiple re-creations

2024-08-26 Thread Matthieu Baerts (NGI0)
vger.kernel.org Tested-by: Arınç ÜNAL Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 90b7c8b45027..591ae2ffb4dd 100644 --

[PATCH net 09/15] mptcp: pm: do not remove already closed subflows

2024-08-26 Thread Matthieu Baerts (NGI0)
;) Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 75c8aede7026..90b7c8b45027 100644 --- a/net/mptcp/pm_netlink.c +++ b/net/mptcp/pm

[PATCH net 08/15] selftests: mptcp: join: no extra msg if no counter

2024-08-26 Thread Matthieu Baerts (NGI0)
;) Cc: sta...@vger.kernel.org Reviewed-by: Geliang Tang Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/sel

[PATCH net 07/15] selftests: mptcp: join: check re-adding init endp with != id

2024-08-26 Thread Matthieu Baerts (NGI0)
54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git

[PATCH net 06/15] mptcp: pm: reset MPC endp ID when re-added

2024-08-26 Thread Matthieu Baerts (NGI0)
int, the same variable is updated if the address match the one of the initial subflow. Fixes: 3ad14f54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 13 +++

[PATCH net 05/15] mptcp: pm: skip connecting to already established sf

2024-08-26 Thread Matthieu Baerts (NGI0)
to be, but not the ones being closed. Fixes: d84ad04941c3 ("mptcp: skip connecting the connected address") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletion

[PATCH net 04/15] mptcp: pm: send ACK on an active subflow

2024-08-26 Thread Matthieu Baerts (NGI0)
Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 042cd65d974a..380647f7bc33 100644 --- a/net/mptcp/pm_netlink.c +++ b/net/mptcp/pm_netlink.c @@ -765,9

[PATCH net 03/15] selftests: mptcp: join: check removing ID 0 endpoint

2024-08-26 Thread Matthieu Baerts (NGI0)
is not fixing anything wrong in the selftests, but it validates the previous fix for an issue introduced by this commit ID. Fixes: 3ad14f54bd74 ("mptcp: more accurate MPC endpoint tracking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (

[PATCH net 02/15] mptcp: pm: fix RM_ADDR ID for the initial subflow

2024-08-26 Thread Matthieu Baerts (NGI0)
") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 28 +++- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 9ee68ab4665e..042c

[PATCH net 01/15] mptcp: pm: reuse ID 0 after delete and re-add

2024-08-26 Thread Matthieu Baerts (NGI0)
cking") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 3e4ad801786f..9ee68ab4665e 100644 --- a

[PATCH net 00/15] mptcp: more fixes for the in-kernel PM

2024-08-26 Thread Matthieu Baerts (NGI0)
low -- has been reached. A fix for v5.10. Patch 15 validates the previous patch. Signed-off-by: Matthieu Baerts (NGI0) --- Matthieu Baerts (NGI0) (15): mptcp: pm: reuse ID 0 after delete and re-add mptcp: pm: fix RM_ADDR ID for the initial subflow selftests: mptcp: join: check r

[PATCH net 14/14] mptcp: pm: avoid possible UaF when selecting endp

2024-08-19 Thread Matthieu Baerts (NGI0)
...@redhat.com Fixes: 01cacb00b35c ("mptcp: add netlink-based PM") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 64 +++--- 1 file changed, 34 insertions(+), 30 deletion

[PATCH net 13/14] selftests: mptcp: join: validate fullmesh endp on 1st sf

2024-08-19 Thread Matthieu Baerts (NGI0)
example, but that's always a bit cryptic. Packetdrill seems better fitted for that. Fixes: 4f49d63352da ("selftests: mptcp: add fullmesh testcases") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_j

[PATCH net 12/14] mptcp: pm: fullmesh: select the right ID later

2024-08-19 Thread Matthieu Baerts (NGI0)
. Note that msk->mpc_endpoint_id might not be set when receiving the first ADD_ADDR from the server. So better to compare the addresses. Fixes: 1a0d6136c5f0 ("mptcp: local addresses fullmesh") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) -

[PATCH net 11/14] mptcp: pm: only in-kernel cannot have entries with ID 0

2024-08-19 Thread Matthieu Baerts (NGI0)
") Cc: sta...@vger.kernel.org Acked-by: Geliang Tang Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm.c | 3 --- net/mptcp/pm_netlink.c | 4 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/net/mptcp/pm.c b/net/mptcp/pm.c ind

[PATCH net 10/14] mptcp: pm: check add_addr_accept_max before accepting new ADD_ADDR

2024-08-19 Thread Matthieu Baerts (NGI0)
The limits might have changed in between, it is best to check them before accepting new ADD_ADDR. Fixes: d0876b2284cf ("mptcp: add the incoming RM_ADDR support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c |

[PATCH net 09/14] mptcp: pm: only decrement add_addr_accepted for MPJ req

2024-08-19 Thread Matthieu Baerts (NGI0)
lve that is to decrement the counter only if the attached subflow was an MP_JOIN to a remote id that was not 0, and initiated by the host receiving the RM_ADDR. Fixes: d0876b2284cf ("mptcp: add the incoming RM_ADDR support") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-of

[PATCH net 08/14] mptcp: pm: only mark 'subflow' endp as available

2024-08-19 Thread Matthieu Baerts (NGI0)
g available, and the decrement is done no matter if a subflow using this ID is currently available, because the subflow could have been closed before. Fixes: 06faa2271034 ("mptcp: remove multi addresses and subflows in PM") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-o

[PATCH net 07/14] mptcp: pm: remove mptcp_pm_remove_subflow()

2024-08-19 Thread Matthieu Baerts (NGI0)
Note that it is not a bug fix, but it will help backporting the following commits. Fixes: 0ee4261a3681 ("mptcp: implement mptcp_pm_remove_subflow") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm.c | 10 -- ne

[PATCH net 06/14] selftests: mptcp: join: test for flush/re-add endpoints

2024-08-19 Thread Matthieu Baerts (NGI0)
lows in PM") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 30 + 1 file changed, 30 insertions(+) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/selftest

[PATCH net 05/14] mptcp: pm: re-using ID of unused flushed subflows

2024-08-19 Thread Matthieu Baerts (NGI0)
se cases. Note that mptcp_pm_remove_addrs_and_subflows() helper is only called for flushing operations, not to remove a specific set of addresses and subflows. Fixes: 06faa2271034 ("mptcp: remove multi addresses and subflows in PM") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed

[PATCH net 04/14] selftests: mptcp: join: check re-using ID of closed subflow

2024-08-19 Thread Matthieu Baerts (NGI0)
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: b6c08380860b ("mptcp: remove addr and subflow in PM netlink") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Ma

[PATCH net 03/14] mptcp: pm: re-using ID of unused removed subflows

2024-08-19 Thread Matthieu Baerts (NGI0)
ecremented if the ID was marked as being used to reflect the reality, but also to allow adding new endpoints after that. Fixes: b6c08380860b ("mptcp: remove addr and subflow in PM netlink") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --

[PATCH net 02/14] selftests: mptcp: join: check re-using ID of unused ADD_ADDR

2024-08-19 Thread Matthieu Baerts (NGI0)
org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/selftests/net/mptcp/mp

[PATCH net 01/14] mptcp: pm: re-using ID of unused removed ADD_ADDR

2024-08-19 Thread Matthieu Baerts (NGI0)
PM netlink") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 4cae2aa7be5c..26f0329e16bb 10

[PATCH net 00/14] mptcp: pm: fix IDs not being reusable

2024-08-19 Thread Matthieu Baerts (NGI0)
the fullmesh mode sets the ID 0 when a new subflow using the source address of the initial subflow is created. Patch 13 covers this case. This issue is present since v5.15. - Patch 14 avoid possible UaF when selecting an address from the endpoints list. Signed-off-by: Matthieu Baerts (NG

[PATCH bpf-next 2/2] selftests: bpf: remove duplicated UAPI if_xdp headers

2024-08-16 Thread Matthieu Baerts (NGI0)
/uapi/linux/if_xdp.h' differs from latest version at 'include/uapi/linux/if_xdp.h' Signed-off-by: Matthieu Baerts (NGI0) --- tools/include/uapi/linux/if_xdp.h | 173 -- tools/lib/bpf/Makefile| 3 - 2 files changed, 176 deletions(-)

[PATCH bpf-next 1/2] selftests: bpf: use KHDR_INCLUDES for the UAPI headers

2024-08-16 Thread Matthieu Baerts (NGI0)
l required to use 'tools/include/uapi' -- APIDIR, which corresponds to TOOLS_INCLUDES from lib.mk -- for the BPF programs, not to conflict with what is already defined in vmlinux.h. Link: https://docs.kernel.org/dev-tools/kselftest.html#contributing-new-tests-details [1]

[PATCH bpf-next 0/2] selftests: bpf: avoid duplicated UAPI headers

2024-08-16 Thread Matthieu Baerts (NGI0)
See patch 1/2 for more details about that. Link: https://lore.kernel.org/all/08f925cd-e267-4a6b-84b1-792515c4e...@kernel.org/T/#u [1] Signed-off-by: Matthieu Baerts (NGI0) --- Matthieu Baerts (NGI0) (2): selftests: bpf: use KHDR_INCLUDES for the UAPI headers selftests: bpf: remove

[PATCH net] selftests: net: lib: kill PIDs before del netns

2024-08-13 Thread Matthieu Baerts (NGI0)
after the end of some tests, and help with the debugging of some issues. That's why this modification is seen as a "fix". Fixes: 25ae948b4478 ("selftests/net: add lib.sh") Acked-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/lib.sh

[PATCH bpf-next v4 2/2] selftests/bpf: Add mptcp subflow subtest

2024-08-05 Thread Matthieu Baerts (NGI0)
using setsockopts. Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76 Signed-off-by: Geliang Tang Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- Notes: - v2 -> v3: - Use './mptcp_pm_nl_ctl' instead of 'ip mptcp', not supported by

[PATCH bpf-next v4 1/2] selftests/bpf: Add mptcp subflow example

2024-08-05 Thread Matthieu Baerts (NGI0)
MPTCP repo https://github.com/multipath-tcp/mptcp_net-next (the "scripts" branch), and it has been adapted by Geliang. Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76 Co-developed-by: Geliang Tang Signed-off-by: Geliang Tang Signed-off-by: Nicolas Rybowski Reviewed

[PATCH bpf-next v4 0/2] selftests/bpf: new MPTCP subflow subtest

2024-08-05 Thread Matthieu Baerts (NGI0)
Tessares, but he did this work while working for them, and his email address is no longer available. - A new MPTCP BPF subtest validating the new BPF program added in the first patch. Signed-off-by: Matthieu Baerts (NGI0) --- Changes in v4: - Drop former patch 2/3: MPTCP's pm_nl_ctl req

[PATCH net 7/7] selftests: mptcp: join: test both signal & subflow

2024-07-31 Thread Matthieu Baerts (NGI0)
h here is not fixing anything wrong in the selftests, but it validates the previous fix for an issue introduced by this commit ID. Fixes: 86e39e04482b ("mptcp: keep track of local endpoint still available for each msk") Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --

[PATCH net 4/7] mptcp: pm: don't try to create sf if alloc failed

2024-07-31 Thread Matthieu Baerts (NGI0)
in-kernel path-manager. With this PM, mptcp_pm_alloc_anno_list() should only fail in case of memory pressure. Fixes: b6c08380860b ("mptcp: remove addr and subflow in PM netlink") Cc: sta...@vger.kernel.org Suggested-by: Paolo Abeni Reviewed-by: Mat Martineau Signed-off-by: Matthieu Ba

[PATCH net 6/7] selftests: mptcp: join: ability to invert ADD_ADDR check

2024-07-31 Thread Matthieu Baerts (NGI0)
lable for each msk") Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 40 - 1 file changed, 26 insertions(+), 14 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/too

[PATCH net 5/7] mptcp: pm: do not ignore 'subflow' if 'signal' flag is also set

2024-07-31 Thread Matthieu Baerts (NGI0)
OK to assume that the limits should be properly configured before establishing new connections. Fixes: 86e39e04482b ("mptcp: keep track of local endpoint still available for each msk") Cc: sta...@vger.kernel.org Suggested-by: Paolo Abeni Reviewed-by: Mat Martineau Signed-off-by: Matthi

[PATCH net 2/7] mptcp: pm: deny endp with signal + subflow + port

2024-07-31 Thread Matthieu Baerts (NGI0)
not possible. Fixes: 09f12c3ab7a5 ("mptcp: allow to use port and non-signal in set_flags") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net

[PATCH net 3/7] mptcp: pm: reduce indentation blocks

2024-07-31 Thread Matthieu Baerts (NGI0)
That will simplify the following commits. No functional changes intended. Suggested-by: Paolo Abeni Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/net/mptcp

[PATCH net 0/7] mptcp: fix endpoints with 'signal' and 'subflow' flags

2024-07-31 Thread Matthieu Baerts (NGI0)
good -- but it still need to be verified by running the selftests -- if not, no need to worry, many CIs will use the selftests from the last stable version to validate previous stable releases. Signed-off-by: Matthieu Baerts (NGI0) --- Matthieu Baerts (NGI0) (7): mptcp: fully established after ADD_

[PATCH net 1/7] mptcp: fully established after ADD_ADDR echo on MPJ

2024-07-31 Thread Matthieu Baerts (NGI0)
ineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/options.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/net/mptcp/options.c b/net/mptcp/options.c index 8a68382a4fe9..ac2f1a54cc43 100644 --- a/net/mptcp/options.c +++ b/net/mptcp/options.c @@ -958,7 +

[PATCH net 7/7] selftests: mptcp: join: check backup support in signal endp

2024-07-27 Thread Matthieu Baerts (NGI0)
e 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: 4596a2c1b7f5 ("mptcp: allow creating non-backup subflows") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-

[PATCH net 6/7] mptcp: pm: fix backup support in signal endpoints

2024-07-27 Thread Matthieu Baerts (NGI0)
Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/507 Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm.c | 12 net/mptcp/pm_netlink.c | 18 ++ net/mptcp/pm_userspace.c | 18 ++ net/mptcp/prot

[PATCH net 5/7] selftests: mptcp: join: validate backup in MPJ

2024-07-27 Thread Matthieu Baerts (NGI0)
in the selftests, but it will help to validate a new fix for an issue introduced by this commit ID. Fixes: 4596a2c1b7f5 ("mptcp: allow creating non-backup subflows") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- tools/testin

[PATCH net 4/7] mptcp: mib: count MPJ with backup flag

2024-07-27 Thread Matthieu Baerts (NGI0)
allow creating non-backup subflows") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/mib.c | 2 ++ net/mptcp/mib.h | 2 ++ net/mptcp/subflow.c | 6 ++ 3 files changed, 10 insertions(+) diff --git a/net/mptcp/mib.c b/net/m

[PATCH net 3/7] mptcp: pm: only set request_bkup flag when sending MP_PRIO

2024-07-27 Thread Matthieu Baerts (NGI0)
ned-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index ea9e5817b9e9..59527335c68e 100644 --- a/net/mptcp/pm_netlink.c +++ b/net/mptcp/pm_netlink.c @@ -471,7 +471,6 @@ static void __mptc

[PATCH net 2/7] mptcp: distinguish rcv vs sent backup flag in requests

2024-07-27 Thread Matthieu Baerts (NGI0)
org Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/options.c | 2 +- net/mptcp/protocol.h | 1 + net/mptcp/subflow.c | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/net/mptcp/options.c b/net/mptcp/options.c index 8e8dcfbc2993..8a68382a4fe9 10

[PATCH net 1/7] mptcp: sched: check both directions for backup

2024-07-27 Thread Matthieu Baerts (NGI0)
ote that the use of these two flags in the path-manager are going to be fixed in the next commits, but this change here is needed not to modify the behaviour. Fixes: f296234c98a8 ("mptcp: Add handling of incoming MP_JOIN requests") Cc: sta...@vger.kernel.org Reviewed-by: Mat Martineau

[PATCH net 0/7] mptcp: fix inconsistent backup usage

2024-07-27 Thread Matthieu Baerts (NGI0)
lftest as well. Signed-off-by: Matthieu Baerts (NGI0) --- Matthieu Baerts (NGI0) (7): mptcp: sched: check both directions for backup mptcp: distinguish rcv vs sent backup flag in requests mptcp: pm: only set request_bkup flag when sending MP_PRIO mptcp: mib: count MPJ wi

[PATCH net 5/5] selftests: mptcp: always close input's FD if opened

2024-07-27 Thread Matthieu Baerts (NGI0)
: Liu Jing Reviewed-by: Matthieu Baerts (NGI0) Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_connect.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/testing/selftests/net/mptcp/mptcp_connect.c b/tools/testing/selftests/net/mptc

[PATCH net 4/5] selftests: mptcp: fix error path

2024-07-27 Thread Matthieu Baerts (NGI0)
From: Paolo Abeni pm_nl_check_endpoint() currently calls an not existing helper to mark the test as failed. Fix the wrong call. Fixes: 03668c65d153 ("selftests: mptcp: join: rework detailed report") Cc: sta...@vger.kernel.org Signed-off-by: Paolo Abeni Reviewed-by: Matthieu Ba

[PATCH net 3/5] selftests: mptcp: add explicit test case for remove/readd

2024-07-27 Thread Matthieu Baerts (NGI0)
From: Paolo Abeni Delete and re-create a signal endpoint and ensure that the PM actually deletes and re-create the subflow. Signed-off-by: Paolo Abeni Reviewed-by: Matthieu Baerts (NGI0) Signed-off-by: Matthieu Baerts (NGI0) --- tools/testing/selftests/net/mptcp/mptcp_join.sh | 29

[PATCH net 2/5] mptcp: fix NL PM announced address accounting

2024-07-27 Thread Matthieu Baerts (NGI0)
that. Address the issue properly updating the signaled address counter every time the NL PM removes such addresses. Fixes: 01cacb00b35c ("mptcp: add netlink-based PM") Cc: sta...@vger.kernel.org Signed-off-by: Paolo Abeni Reviewed-by: Matthieu Baerts (NGI0) Signed-off-by: Matthieu Ba

[PATCH net 1/5] mptcp: fix user-space PM announced address accounting

2024-07-27 Thread Matthieu Baerts (NGI0)
sake. Fixes: 8b1c94da1e48 ("mptcp: only send RM_ADDR in nl_cmd_remove") Cc: sta...@vger.kernel.org Signed-off-by: Paolo Abeni Reviewed-by: Matthieu Baerts (NGI0) Signed-off-by: Matthieu Baerts (NGI0) --- net/mptcp/pm_netlink.c | 17 + 1 file changed, 13 insertions(

[PATCH net 0/5] mptcp: fix signal endpoint readd

2024-07-27 Thread Matthieu Baerts (NGI0)
roduced in v6.6., and the other one in v5.17. Link: https://github.com/multipath-tcp/mptcp_net-next/issues/501 [1] Signed-off-by: Matthieu Baerts (NGI0) --- Liu Jing (1): selftests: mptcp: always close input's FD if opened Paolo Abeni (4): mptcp: fix user-space PM announce

[PATCH net-next] selftests: mptcp: lib: fix shellcheck errors

2024-07-12 Thread Matthieu Baerts (NGI0)
any spaces in their names. Still, better to fix them to easily spot new issues. Fixes: f265d3119a29 ("selftests: mptcp: lib: use setup/cleanup_ns helpers") Signed-off-by: Matthieu Baerts (NGI0) --- Notes: - The mentioned commit is currently only in 'net-next', not in '

[PATCH bpf-next v3 3/3] selftests/bpf: Add mptcp subflow subtest

2024-07-03 Thread Matthieu Baerts (NGI0)
using setsockopts. Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/76 Signed-off-by: Geliang Tang Reviewed-by: Mat Martineau Signed-off-by: Matthieu Baerts (NGI0) --- Notes: - v2 -> v3: - Use './mptcp_pm_nl_ctl' instead of 'ip mptcp', not supported by

[PATCH bpf-next v3 2/3] selftests/bpf: Add mptcp pm_nl_ctl link

2024-07-03 Thread Matthieu Baerts (NGI0)
ERS needs to be updated since a new file is added in a non covered place. Signed-off-by: Geliang Tang Reviewed-by: Matthieu Baerts (NGI0) Signed-off-by: Matthieu Baerts (NGI0) --- MAINTAINERS | 1 + tools/testing/selftests/bpf/Makefile | 3 ++- tools/te

  1   2   3   >