mptcp: pm: nl: announce deny-join-id0 flag
Subsystems (4)
Matthieu Baerts (NGI0)
maintainerPATCH 1/1 mptcp: pm: nl: announce deny-join-id0 flag
commit 2293c57484ae64c9a3c847c8807db8c26a3a4d41 upstream.
During the connection establishment, a peer can tell the other one that it cannot establish new subflows to the initial IP address and port by setting the 'C' flag [1]. Doing so makes sense when the sender is behind a strict NAT, operating behind a legacy Layer 4 load balancer, or using anycast IP address for example.
When this 'C' flag is set, the path-managers must then not try to establish new subflows to the other peer's initial IP address and port. The in-kernel PM has access to this info, but the userspace PM didn't.
The RFC8684 [1] is strict about that:
(...) therefore the receiver MUST NOT try to open any additional subflows toward this address and port.
So it is important to tell the userspace about that as it is responsible for the respect of this flag.
When a new connection is created and established, the Netlink events now contain the existing but not currently used 'flags' attribute. When MPTCP_PM_EV_FLAG_DENY_JOIN_ID0 is set, it means no other subflows to the initial IP address and port -- info that are also part of the event -- can be established.
[ Conflicts in mptcp_pm.yaml, because the indentation has been modified
in commit ec362192aa9e ("netlink: specs: fix up indentation errors"), which is not in this version. Applying the same modifications, but at a different level. ]
--- Documentation/netlink/specs/mptcp_pm.yaml | 4 ++-- include/uapi/linux/mptcp.h | 2 ++ include/uapi/linux/mptcp_pm.h | 4 ++-- net/mptcp/pm_netlink.c | 7 +++++++ 4 files changed, 13 insertions(+), 4 deletions(-)
Documentation/netlink/specs/mptcp_pm.yaml +2 -2
@@ -28,13 +28,13 @@ definitions: traffic-patterns it can take a long time until the MPTCP_EVENT_ESTABLISHED is sent. Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, sport, - dport, server-side. + dport, server-side, [flags]. - name: established doc: >- A MPTCP connection is established (can start new subflows). Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, sport, - dport, server-side. + dport, server-side, [flags]. - name: closed doc: >-
include/uapi/linux/mptcp.h +2 -0
@@ -31,6 +31,8 @@#define MPTCP_INFO_FLAG_FALLBACK _BITUL(0)#define MPTCP_INFO_FLAG_REMOTE_KEY_RECEIVED _BITUL(1) +#define MPTCP_PM_EV_FLAG_DENY_JOIN_ID0 _BITUL(0) +#define MPTCP_PM_ADDR_FLAG_SIGNAL (1 << 0)#define MPTCP_PM_ADDR_FLAG_SUBFLOW (1 << 1)#define MPTCP_PM_ADDR_FLAG_BACKUP (1 << 2)
include/uapi/linux/mptcp_pm.h +2 -2
@@ -16,10 +16,10 @@* good time to allocate memory and send ADD_ADDR if needed. Depending on the * traffic-patterns it can take a long time until the MPTCP_EVENT_ESTABLISHED * is sent. Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, -* sport, dport, server-side. +* sport, dport, server-side, [flags]. * @MPTCP_EVENT_ESTABLISHED: A MPTCP connection is established (can start new * subflows). Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, -* sport, dport, server-side. +* sport, dport, server-side, [flags]. * @MPTCP_EVENT_CLOSED: A MPTCP connection has stopped. Attribute: token. * @MPTCP_EVENT_ANNOUNCED: A new address has been announced by the peer. * Attributes: token, rem_id, family, daddr4 | daddr6 [, dport].
net/mptcp/pm_netlink.c +7 -0
@@ -408,6 +408,7 @@ static int mptcp_event_created(struct sk_buff *skb, const struct sock *ssk) {int err =nla_put_u32 (skb,MPTCP_ATTR_TOKEN ,READ_ONCE (msk->token)); +u16 flags =0 ;if (err)return err;
@@ -415,6 +416,12 @@ static int mptcp_event_created(struct sk_buff *skb,if (nla_put_u8 (skb,MPTCP_ATTR_SERVER_SIDE ,READ_ONCE (msk->pm.server_side)))return -EMSGSIZE ; +if (READ_ONCE (msk->pm.remote_deny_join_id0)) + flags |=MPTCP_PM_EV_FLAG_DENY_JOIN_ID0 ; + +if (flags &&nla_put_u16 (skb,MPTCP_ATTR_FLAGS , flags)) +return -EMSGSIZE ; +return mptcp_event_add_subflow (skb, ssk); }
Signature · 1 line
2.51.0
Patch "mptcp: pm: nl: announce deny-join-id0 flag" has been added to the 6.16-stable tree
This is a note to let you know that I've just added the patch titled
mptcp: pm: nl: announce deny-join-id0 flag
to the 6.16-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
mptcp-pm-nl-announce-deny-join-id0-flag.patch
and it can be found in the queue-6.16 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
From [email protected] Fri Sep 19 23:50:00 2025
From: "Matthieu Baerts (NGI0)" <[email protected]>
Date: Fri, 19 Sep 2025 23:49:22 +0200
Subject: mptcp: pm: nl: announce deny-join-id0 flag
To: [email protected], [email protected], [email protected]
Cc: MPTCP Upstream <[email protected]>, "Matthieu Baerts (NGI0)" <[email protected]>, Marek Majkowski <[email protected]>, Mat Martineau <[email protected]>, Jakub Kicinski <[email protected]>
Message-ID: <[email protected]>
From: "Matthieu Baerts (NGI0)" <[email protected]>
commit 2293c57484ae64c9a3c847c8807db8c26a3a4d41 upstream.
During the connection establishment, a peer can tell the other one that it cannot establish new subflows to the initial IP address and port by setting the 'C' flag [1]. Doing so makes sense when the sender is behind a strict NAT, operating behind a legacy Layer 4 load balancer, or using anycast IP address for example.
When this 'C' flag is set, the path-managers must then not try to establish new subflows to the other peer's initial IP address and port. The in-kernel PM has access to this info, but the userspace PM didn't.
The RFC8684 [1] is strict about that:
(...) therefore the receiver MUST NOT try to open any additional subflows toward this address and port.
So it is important to tell the userspace about that as it is responsible for the respect of this flag.
When a new connection is created and established, the Netlink events now contain the existing but not currently used 'flags' attribute. When MPTCP_PM_EV_FLAG_DENY_JOIN_ID0 is set, it means no other subflows to the initial IP address and port -- info that are also part of the event -- can be established.
[ Conflicts in mptcp_pm.yaml, because the indentation has been modified
in commit ec362192aa9e ("netlink: specs: fix up indentation errors"), which is not in this version. Applying the same modifications, but at a different level. ]
--- Documentation/netlink/specs/mptcp_pm.yaml | 4 ++-- include/uapi/linux/mptcp.h | 2 ++ include/uapi/linux/mptcp_pm.h | 4 ++-- net/mptcp/pm_netlink.c | 7 +++++++ 4 files changed, 13 insertions(+), 4 deletions(-)
Documentation/netlink/specs/mptcp_pm.yaml +2 -2
@@ -28,13 +28,13 @@ definitions: traffic-patterns it can take a long time until the MPTCP_EVENT_ESTABLISHED is sent. Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, sport, - dport, server-side. + dport, server-side, [flags]. - name: established doc: >- A MPTCP connection is established (can start new subflows). Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, sport, - dport, server-side. + dport, server-side, [flags]. - name: closed doc: >-
include/uapi/linux/mptcp.h +2 -0
@@ -31,6 +31,8 @@#define MPTCP_INFO_FLAG_FALLBACK _BITUL(0)#define MPTCP_INFO_FLAG_REMOTE_KEY_RECEIVED _BITUL(1) +#define MPTCP_PM_EV_FLAG_DENY_JOIN_ID0 _BITUL(0) +#define MPTCP_PM_ADDR_FLAG_SIGNAL (1 << 0)#define MPTCP_PM_ADDR_FLAG_SUBFLOW (1 << 1)#define MPTCP_PM_ADDR_FLAG_BACKUP (1 << 2)
include/uapi/linux/mptcp_pm.h +2 -2
@@ -16,10 +16,10 @@* good time to allocate memory and send ADD_ADDR if needed. Depending on the * traffic-patterns it can take a long time until the MPTCP_EVENT_ESTABLISHED * is sent. Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, -* sport, dport, server-side. +* sport, dport, server-side, [flags]. * @MPTCP_EVENT_ESTABLISHED: A MPTCP connection is established (can start new * subflows). Attributes: token, family, saddr4 | saddr6, daddr4 | daddr6, -* sport, dport, server-side. +* sport, dport, server-side, [flags]. * @MPTCP_EVENT_CLOSED: A MPTCP connection has stopped. Attribute: token. * @MPTCP_EVENT_ANNOUNCED: A new address has been announced by the peer. * Attributes: token, rem_id, family, daddr4 | daddr6 [, dport].
net/mptcp/pm_netlink.c +7 -0
@@ -408,6 +408,7 @@ static int mptcp_event_created(struct sk const struct sock *ssk) {int err =nla_put_u32 (skb,MPTCP_ATTR_TOKEN ,READ_ONCE (msk->token)); +u16 flags =0 ;if (err)return err;
@@ -415,6 +416,12 @@ static int mptcp_event_created(struct skif (nla_put_u8 (skb,MPTCP_ATTR_SERVER_SIDE ,READ_ONCE (msk->pm.server_side)))return -EMSGSIZE ; +if (READ_ONCE (msk->pm.remote_deny_join_id0)) + flags |=MPTCP_PM_EV_FLAG_DENY_JOIN_ID0 ; + +if (flags &&nla_put_u16 (skb,MPTCP_ATTR_FLAGS , flags)) +return -EMSGSIZE ; +return mptcp_event_add_subflow (skb, ssk); }
Patches currently in stable-queue which might be from [email protected] are
queue-6.16/mptcp-tfo-record-deny-join-id0-info.patch queue-6.16/mptcp-pm-nl-announce-deny-join-id0-flag.patch queue-6.16/selftests-mptcp-connect-catch-io-errors-on-listen-side.patch queue-6.16/selftests-mptcp-avoid-spurious-errors-on-tcp-disconnect.patch queue-6.16/selftests-mptcp-userspace-pm-validate-deny-join-id0-.patch queue-6.16/mptcp-propagate-shutdown-to-subflows-when-possible.patch queue-6.16/mptcp-set-remote_deny_join_id0-on-syn-recv.patch queue-6.16/selftests-mptcp-sockopt-fix-error-messages.patch