diff mbox series

[RFC] net: enable SO_REUSEPORT for AF_TIPC sockets

Message ID dec1f621-a770-4c9a-89e9-e0f26ab470e2@suse.com (mailing list archive)
State RFC
Delegated to: Netdev Maintainers
Headers show
Series [RFC] net: enable SO_REUSEPORT for AF_TIPC sockets | expand

Checks

Context Check Description
netdev/series_format warning Single patches do not need cover letters; Target tree name not specified in the subject
netdev/tree_selection success Guessed tree name to be net-next, async
netdev/ynl success Generated files up to date; no warnings/errors; no diff in generated;
netdev/fixes_present success Fixes tag not required for -next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 14 this patch: 14
netdev/build_tools success Errors and warnings before: 26 (+0) this patch: 26 (+0)
netdev/cc_maintainers fail 2 blamed authors not CCed: kuniyu@amazon.com kuba@kernel.org; 5 maintainers not CCed: kuniyu@amazon.com pabeni@redhat.com horms@kernel.org willemb@google.com kuba@kernel.org
netdev/build_clang success Errors and warnings before: 26 this patch: 26
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success Fixes tag looks correct
netdev/build_allmodconfig_warn success Errors and warnings before: 2597 this patch: 2597
netdev/checkpatch warning WARNING: From:/Signed-off-by: email address mismatch: 'From: Nicolas Morey <nicolas.morey@suse.com>' != 'Signed-off-by: Nicolas Morey <nmorey@suse.com>'
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 13 this patch: 13
netdev/source_inline success Was 0 now: 0

Commit Message

Nicolas Morey March 12, 2025, 1:48 p.m. UTC
Commit 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets") disabled
SO_REUSEPORT for all non inet sockets, including AF_TIPC sockets which broke
one of our customer applications.
Re-enable SO_REUSEPORT for AF_TIPC to restore the original behaviour.

Fixes: 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets")
Signed-off-by: Nicolas Morey <nmorey@suse.com>
---
 include/net/sock.h | 5 +++++
 net/core/sock.c    | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

Comments

Kuniyuki Iwashima March 12, 2025, 4:35 p.m. UTC | #1
From: Nicolas Morey <nicolas.morey@suse.com>
Date: Wed, 12 Mar 2025 14:48:01 +0100
> Commit 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets") disabled
> SO_REUSEPORT for all non inet sockets, including AF_TIPC sockets which broke
> one of our customer applications.
> Re-enable SO_REUSEPORT for AF_TIPC to restore the original behaviour.

AFAIU, AF_TIPC does not actually implement SO_REUSEPORT logic, no ?
If so, please tell your customer not to set it on AF_TIPC sockets.

There were similar reports about AF_VSOCK and AF_UNIX, and we told
that the userspace should not set SO_REUSEPORT for such sockets
that do not support the option.

https://lore.kernel.org/stable/CAGxU2F57EgVGbPifRuCvrUVjx06mrOXNdLcPdqhV9bdM0VqGvg@mail.gmail.com/
https://github.com/amazonlinux/amazon-linux-2023/issues/901


> 
> Fixes: 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets")
> Signed-off-by: Nicolas Morey <nmorey@suse.com>
> ---
>  include/net/sock.h | 5 +++++
>  net/core/sock.c    | 2 +-
>  2 files changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/include/net/sock.h b/include/net/sock.h
> index 7ef728324e4e..d14f6ffedcd5 100644
> --- a/include/net/sock.h
> +++ b/include/net/sock.h
> @@ -2755,6 +2755,11 @@ static inline bool sk_is_vsock(const struct sock *sk)
>  	return sk->sk_family == AF_VSOCK;
>  }
>  
> +static inline bool sk_is_tipc(const struct sock *sk)
> +{
> +	return sk->sk_family == AF_TIPC;
> +}
> +
>  /**
>   * sk_eat_skb - Release a skb if it is no longer needed
>   * @sk: socket to eat this skb from
> diff --git a/net/core/sock.c b/net/core/sock.c
> index 6c0e87f97fa4..d4ad4cdff997 100644
> --- a/net/core/sock.c
> +++ b/net/core/sock.c
> @@ -1300,7 +1300,7 @@ int sk_setsockopt(struct sock *sk, int level, int optname,
>  		sk->sk_reuse = (valbool ? SK_CAN_REUSE : SK_NO_REUSE);
>  		break;
>  	case SO_REUSEPORT:
> -		if (valbool && !sk_is_inet(sk))
> +		if (valbool && !sk_is_inet(sk) && !sk_is_tipc(sk))
>  			ret = -EOPNOTSUPP;
>  		else
>  			sk->sk_reuseport = valbool;
> -- 
> 2.45.2
Nicolas Morey March 12, 2025, 5:44 p.m. UTC | #2
On 2025-03-12 17:35, Kuniyuki Iwashima wrote:
> From: Nicolas Morey <nicolas.morey@suse.com>
> Date: Wed, 12 Mar 2025 14:48:01 +0100
>> Commit 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets") disabled
>> SO_REUSEPORT for all non inet sockets, including AF_TIPC sockets which broke
>> one of our customer applications.
>> Re-enable SO_REUSEPORT for AF_TIPC to restore the original behaviour.
> 
> AFAIU, AF_TIPC does not actually implement SO_REUSEPORT logic, no ?
> If so, please tell your customer not to set it on AF_TIPC sockets.
> 
> There were similar reports about AF_VSOCK and AF_UNIX, and we told
> that the userspace should not set SO_REUSEPORT for such sockets
> that do not support the option.
> 
> https://lore.kernel.org/stable/CAGxU2F57EgVGbPifRuCvrUVjx06mrOXNdLcPdqhV9bdM0VqGvg@mail.gmail.com/
> https://github.com/amazonlinux/amazon-linux-2023/issues/901
> 
> 
Isn't the sk_reuseport inherited/used by the underlying UDP socket ?

Nicolas
Kuniyuki Iwashima March 12, 2025, 6:20 p.m. UTC | #3
From: Nicolas Morey <nicolas.morey@suse.com>
Date: Wed, 12 Mar 2025 18:44:10 +0100
> On 2025-03-12 17:35, Kuniyuki Iwashima wrote:
> > From: Nicolas Morey <nicolas.morey@suse.com>
> > Date: Wed, 12 Mar 2025 14:48:01 +0100
> >> Commit 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets") disabled
> >> SO_REUSEPORT for all non inet sockets, including AF_TIPC sockets which broke
> >> one of our customer applications.
> >> Re-enable SO_REUSEPORT for AF_TIPC to restore the original behaviour.
> > 
> > AFAIU, AF_TIPC does not actually implement SO_REUSEPORT logic, no ?
> > If so, please tell your customer not to set it on AF_TIPC sockets.
> > 
> > There were similar reports about AF_VSOCK and AF_UNIX, and we told
> > that the userspace should not set SO_REUSEPORT for such sockets
> > that do not support the option.
> > 
> > https://lore.kernel.org/stable/CAGxU2F57EgVGbPifRuCvrUVjx06mrOXNdLcPdqhV9bdM0VqGvg@mail.gmail.com/
> > https://github.com/amazonlinux/amazon-linux-2023/issues/901
> > 
> > 
> Isn't the sk_reuseport inherited/used by the underlying UDP socket ?

tipc_udp_enable() calls udp_sock_create() and udp_sock_create[46]()
creates a new UDP socket and bind()s without setting SO_REUSEPORT.
Nicolas Morey March 12, 2025, 7:45 p.m. UTC | #4
On 2025-03-12 19:20, Kuniyuki Iwashima wrote:
> From: Nicolas Morey <nicolas.morey@suse.com>
> Date: Wed, 12 Mar 2025 18:44:10 +0100
>> On 2025-03-12 17:35, Kuniyuki Iwashima wrote:
>>> From: Nicolas Morey <nicolas.morey@suse.com>
>>> Date: Wed, 12 Mar 2025 14:48:01 +0100
>>>> Commit 5b0af621c3f6 ("net: restrict SO_REUSEPORT to inet sockets") disabled
>>>> SO_REUSEPORT for all non inet sockets, including AF_TIPC sockets which broke
>>>> one of our customer applications.
>>>> Re-enable SO_REUSEPORT for AF_TIPC to restore the original behaviour.
>>>
>>> AFAIU, AF_TIPC does not actually implement SO_REUSEPORT logic, no ?
>>> If so, please tell your customer not to set it on AF_TIPC sockets.
>>>
>>> There were similar reports about AF_VSOCK and AF_UNIX, and we told
>>> that the userspace should not set SO_REUSEPORT for such sockets
>>> that do not support the option.
>>>
>>> https://lore.kernel.org/stable/CAGxU2F57EgVGbPifRuCvrUVjx06mrOXNdLcPdqhV9bdM0VqGvg@mail.gmail.com/
>>> https://github.com/amazonlinux/amazon-linux-2023/issues/901
>>>
>>>
>> Isn't the sk_reuseport inherited/used by the underlying UDP socket ?
> 
> tipc_udp_enable() calls udp_sock_create() and udp_sock_create[46]()
> creates a new UDP socket and bind()s without setting SO_REUSEPORT.

Thanks for the feedback. We'll advise to fix the userland code then.

Nicolas
diff mbox series

Patch

diff --git a/include/net/sock.h b/include/net/sock.h
index 7ef728324e4e..d14f6ffedcd5 100644
--- a/include/net/sock.h
+++ b/include/net/sock.h
@@ -2755,6 +2755,11 @@  static inline bool sk_is_vsock(const struct sock *sk)
 	return sk->sk_family == AF_VSOCK;
 }
 
+static inline bool sk_is_tipc(const struct sock *sk)
+{
+	return sk->sk_family == AF_TIPC;
+}
+
 /**
  * sk_eat_skb - Release a skb if it is no longer needed
  * @sk: socket to eat this skb from
diff --git a/net/core/sock.c b/net/core/sock.c
index 6c0e87f97fa4..d4ad4cdff997 100644
--- a/net/core/sock.c
+++ b/net/core/sock.c
@@ -1300,7 +1300,7 @@  int sk_setsockopt(struct sock *sk, int level, int optname,
 		sk->sk_reuse = (valbool ? SK_CAN_REUSE : SK_NO_REUSE);
 		break;
 	case SO_REUSEPORT:
-		if (valbool && !sk_is_inet(sk))
+		if (valbool && !sk_is_inet(sk) && !sk_is_tipc(sk))
 			ret = -EOPNOTSUPP;
 		else
 			sk->sk_reuseport = valbool;