]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
TCP: remove special treatment for hardware (ifnet) TLS
authorAndrew Gallatin <gallatin@FreeBSD.org>
Wed, 19 Aug 2020 17:59:06 +0000 (17:59 +0000)
committerAndrew Gallatin <gallatin@FreeBSD.org>
Wed, 19 Aug 2020 17:59:06 +0000 (17:59 +0000)
commitb99781834fed7d67fbe68c11e6f0f34e4048fc8a
tree459f25a4199487936fc6bc3b53f9a3bd72775259
parent697718b9b6af0af274f08e97d27296c85198660b
TCP: remove special treatment for hardware (ifnet) TLS

Remove most special treatment for ifnet TLS in the TCP stack, except
for code to avoid mixing handshakes and bulk data.

This code made heroic efforts to send down entire TLS records to
NICs. It was added to improve the PCIe bus efficiency of older TLS
offload NICs which did not keep state per-session, and so would need
to re-DMA the first part(s) of a TLS record if a TLS record was sent
in multiple TCP packets or TSOs. Newer TLS offload NICs do not need
this feature.

At Netflix, we've run extensive QoE tests which show that this feature
reduces client quality metrics, presumably because the effort to send
TLS records atomically causes the server to both wait too long to send
data (leading to buffers running dry), and to send too much data at
once (leading to packet loss).

Reviewed by: hselasky,  jhb, rrs
Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D26103
sys/netinet/tcp_output.c
sys/netinet/tcp_stacks/bbr.c
sys/netinet/tcp_stacks/rack.c