Commit b1202515 authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller
Browse files

net: lockless SO_{TYPE|PROTOCOL|DOMAIN|ERROR } setsockopt() 



This options can not be set and return -ENOPROTOOPT,
no need to acqure socket lock.

Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8ebfb6db

net/core/sock.c

+5 −6
Original line number Diff line number Diff line
@@ -1135,6 +1135,11 @@ int sk_setsockopt(struct sock *sk, int level, int optname, 
	case SO_PASSPIDFD:

		assign_bit(SOCK_PASSPIDFD, &sock->flags, valbool);

		return 0;

	case SO_TYPE:

	case SO_PROTOCOL:

	case SO_DOMAIN:

	case SO_ERROR:

		return -ENOPROTOOPT;

	}



	sockopt_lock_sock(sk);
@@ -1152,12 +1157,6 @@ int sk_setsockopt(struct sock *sk, int level, int optname, 
	case SO_REUSEPORT:

		sk->sk_reuseport = valbool;

		break;

	case SO_TYPE:

	case SO_PROTOCOL:

	case SO_DOMAIN:

	case SO_ERROR:

		ret = -ENOPROTOOPT;

		break;

	case SO_DONTROUTE:

		sock_valbool_flag(sk, SOCK_LOCALROUTE, valbool);

		sk_dst_reset(sk);