Merge branch 'tcp_conn_request_unification'
authorDavid S. Miller <davem@davemloft.net>
Fri, 27 Jun 2014 22:53:54 +0000 (15:53 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 27 Jun 2014 22:53:54 +0000 (15:53 -0700)
Octavian Purdila says:

====================
tcp: remove code duplication in tcp_v[46]_conn_request

This patch series unifies the TCPv4 and TCPv6 connection request flow
in a single new function (tcp_conn_request).

The first 3 patches are small cleanups and fixes found during the code
merge process.

The next patches add new methods in tcp_request_sock_ops to abstract
the IPv4/IPv6 operations and keep the TCP connection request flow
common.

To identify potential performance issues this patch has been tested
by measuring the connection per second rate with nginx and a httperf
like client (to allow for concurrent connection requests - 256 CC were
used during testing) using the loopback interface. A dual-core i5 Ivy
Bridge processor was used and each process was bounded to a different
core to make results consistent.

Results for IPv4, unit is connections per second, higher is better, 20
measurements have been collected:

before after
min 27917 27962
max 28262 28366
avg 28094.1 28212.75
stdev 87.35 97.26

Results for IPv6, unit is connections per second, higher is better, 20
measurements have been collected:

before after
min 24813 24877
max 25029 25119
avg 24935.5 25017
stdev 64.13 62.93

Changes since v1:

 * add benchmarking datapoints

 * fix a few issues in the last patch (IPv6 related)
====================

Signed-off-by: David S. Miller <davem@davemloft.net>

Trivial merge