]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
TCP: remove special treatment for hardware (ifnet) TLS
authorgallatin <gallatin@FreeBSD.org>
Wed, 19 Aug 2020 17:59:06 +0000 (17:59 +0000)
committergallatin <gallatin@FreeBSD.org>
Wed, 19 Aug 2020 17:59:06 +0000 (17:59 +0000)
commit2c8d463a1db2eb8a4a28b2a55170fe8e3c74ca4f
tree459f25a4199487936fc6bc3b53f9a3bd72775259
parent30e8108315561adacb98fb772b2fe7d56f04edf4
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