net: tcp: prefer listeners bound to an address
authorPeter Oskolkov <posk@google.com>
Wed, 12 Dec 2018 21:15:35 +0000 (13:15 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 14 Dec 2018 23:55:20 +0000 (15:55 -0800)
commitd9fbc7f6431fc0e5c0ddedf72206d7c5175c5c9a
tree89f1ed973b4df1a7c87041dc26a3c991f0e699da
parent23b0269e58aee1165133b9696e43992f969b5088
net: tcp: prefer listeners bound to an address

A relatively common use case is to have several IPs configured
on a host, and have different listeners for each of them. We would
like to add a "catch all" listener on addr_any, to match incoming
connections not served by any of the listeners bound to a specific
address.

However, port-only lookups can match addr_any sockets when sockets
listening on specific addresses are present if so_reuseport flag
is set. This patch eliminates lookups into port-only hashtable,
as lookups by (addr,port) tuple are easily available.

In addition, compute_score() is tweaked to _not_ match
addr_any sockets to specific addresses, as hash collisions
could result in the unwanted behavior described above.

Tested: the patch compiles; full test in the last patch in this
patchset. Existing reuseport_* selftests also pass.

Suggested-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Peter Oskolkov <posk@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/inet_hashtables.c