On Fri, 29 Oct 2021, Florian Westphal wrote:
> First patch adds ipv4/ipv6 TRANSPARENT and FREEBIND, second
> patch adds a selftest to mptcp_connect.sh.
>
> Only change is in patch 2 which has more verbose SKIP messages
> and an updated mptcp/config to account for IPV6_MULTIPLE_TABLES
> and other toggles required for the tproxy test case.
>
> Florian Westphal (2):
> mptcp: sockopt: add SOL_IP freebind & transparent options
> selftests: mptcp: add tproxy test case
>
> net/mptcp/sockopt.c | 66 ++++++++++++++-
> net/mptcp/subflow.c | 3 +-
> .../selftests/net/mptcp/config | 7 ++-
> .../selftests/net/mptcp/mptcp_connect.c | 51 +++++++++++-
> .../selftests/net/mptcp/mptcp_connect.sh | 80 +++++++++++++++++++
> 5 files changed, 202 insertions(+), 5 deletions(-)
> --
> 2.32.0
Looks good to me, thanks for tracking down the missing config.
It seemed a little odd at first that sync_socket_options() copies the
transparent/freebind values without considering whether the protocol (v4
or v6) used in the setsockopt matches the subflow being synced, but the IP
code doesn't really care for historical reasons. Looks like ipv6 used to
just use the SOL_IP options for these anyway, until someone noticed it
didn't work on AF_INET6 SOCK_RAW sockets.
Reviewed-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
--
Mat Martineau
Intel