Merge branch 'mptcp-subflow-init'
authorDavid S. Miller <davem@davemloft.net>
Mon, 17 Apr 2023 07:18:34 +0000 (08:18 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 17 Apr 2023 07:18:34 +0000 (08:18 +0100)
commit28f610d0868d961059f66fb4a3b2de082d52a4f6
tree74d36f2d30e7ddd302f4f4c13378e92daa4fd5f8
parent0475135f8c81dccd556234e14ec65888e71994db
parent8d547809a5d74aacf022d1df7a508c631088aaec
Merge branch 'mptcp-subflow-init'

Matthieu Baerts says:

====================
mptcp: refactor first subflow init

This series refactors the initialisation of the first subflow of a
listen socket. The first subflow allocation is no longer done at the
initialisation of the socket but later, when the connection request is
received or when requested by the userspace.

This is needed not just because Paolo likes to refactor things but
because this simplifies the code and makes the behaviour more consistent
with the rest. Also, this is a prerequisite for future patches adding
proper support of SELinux/LSM labels with MPTCP and accept(2).

In [1], Ondrej Mosnacek explained they discovered the (userspace-facing)
sockets returned by accept(2) when using MPTCP always end up with the
label representing the kernel (typically system_u:system_r:kernel_t:s0),
while it would make more sense to inherit the context from the parent
socket (the one that is passed to accept(2)).

Before being able to properly support that on SELinux/LSM side, patches
2-3/5 prepare the code to simplify the patch 4/5 moving the allocation.

Patch 1/5 is a small clean-up seen while working on the series and patch
5/5 is a small improvement when closing unaccepted sockets.

[1] https://lore.kernel.org/netdev/CAFqZXNs2LF-OoQBUiiSEyranJUXkPLcCfBkMkwFeM6qEwMKCTw@mail.gmail.com/
====================

Signed-off-by: David S. Miller <davem@davemloft.net>