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

tcp/dccp: try to not exhaust ip_local_port_range in connect() 



A long standing problem on busy servers is the tiny available TCP port
range (/proc/sys/net/ipv4/ip_local_port_range) and the default
sequential allocation of source ports in connect() system call.

If a host is having a lot of active TCP sessions, chances are
very high that all ports are in use by at least one flow,
and subsequent bind(0) attempts fail, or have to scan a big portion of
space to find a slot.

In this patch, I changed the starting point in __inet_hash_connect()
so that we try to favor even [1] ports, leaving odd ports for bind()
users.

We still perform a sequential search, so there is no guarantee, but
if connect() targets are very different, end result is we leave
more ports available to bind(), and we spread them all over the range,
lowering time for both connect() and bind() to find a slot.

This strategy only works well if /proc/sys/net/ipv4/ip_local_port_range
is even, ie if start/end values have different parity.

Therefore, default /proc/sys/net/ipv4/ip_local_port_range was changed to
32768 - 60999 (instead of 32768 - 61000)

There is no change on security aspects here, only some poor hashing
schemes could be eventually impacted by this change.

[1] : The odd/even property depends on ip_local_port_range values parity

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

Documentation/networking/ip-sysctl.txt

+5 −3
Original line number Diff line number Diff line
@@ -751,8 +751,10 @@ IP Variables: 
ip_local_port_range - 2 INTEGERS

	Defines the local port range that is used by TCP and UDP to

	choose the local port. The first number is the first, the

	second the last local port number. The default values are

	32768 and 61000 respectively.

	second the last local port number.

	If possible, it is better these numbers have different parity.

	(one even and one odd values)

	The default values are 32768 and 60999 respectively.



ip_local_reserved_ports - list of comma separated ranges

	Specify the ports which are reserved for known third-party
@@ -775,7 +777,7 @@ ip_local_reserved_ports - list of comma separated ranges 
	ip_local_port_range, e.g.:



	$ cat /proc/sys/net/ipv4/ip_local_port_range

	32000	61000

	32000	60999

	$ cat /proc/sys/net/ipv4/ip_local_reserved_ports

	8080,9148

net/ipv4/af_inet.c

+1 −1
Original line number Diff line number Diff line
@@ -1595,7 +1595,7 @@ static __net_init int inet_init_net(struct net *net) 
	 */

	seqlock_init(&net->ipv4.ip_local_ports.lock);

	net->ipv4.ip_local_ports.range[0] =  32768;

	net->ipv4.ip_local_ports.range[1] =  61000;

	net->ipv4.ip_local_ports.range[1] =  60999;



	seqlock_init(&net->ipv4.ping_group_range.lock);

	/*

net/ipv4/inet_hashtables.c

+8 −2
Original line number Diff line number Diff line
@@ -502,8 +502,14 @@ int __inet_hash_connect(struct inet_timewait_death_row *death_row, 
		inet_get_local_port_range(net, &low, &high);

		remaining = (high - low) + 1;



		/* By starting with offset being an even number,

		 * we tend to leave about 50% of ports for other uses,

		 * like bind(0).

		 */

		offset &= ~1;



		local_bh_disable();

		for (i = 1; i <= remaining; i++) {

		for (i = 0; i < remaining; i++) {

			port = low + (i + offset) % remaining;

			if (inet_is_local_reserved_port(net, port))

				continue;
@@ -547,7 +553,7 @@ int __inet_hash_connect(struct inet_timewait_death_row *death_row, 
		return -EADDRNOTAVAIL;



ok:

		hint += i;

		hint += (i + 2) & ~1;



		/* Head lock still held and bh's disabled */

		inet_bind_hash(sk, tb, port);

CRA Git | Maintained and supported by SUSTech CRA and CCSE