tcp: allow zerocopy with fastopen
authorWillem de Bruijn <willemb@google.com>
Fri, 25 Jan 2019 16:17:23 +0000 (11:17 -0500)
committerDavid S. Miller <davem@davemloft.net>
Sat, 26 Jan 2019 06:41:08 +0000 (22:41 -0800)
commitf859a448470304135f7a1af0083b99e188873bb4
tree670841050b51d1c16a3633ac229fd43e02b42f82
parent84239b445964b15045e1418ebf28e74121ace309
tcp: allow zerocopy with fastopen

Accept MSG_ZEROCOPY in all the TCP states that allow sendmsg. Remove
the explicit check for ESTABLISHED and CLOSE_WAIT states.

This requires correctly handling zerocopy state (uarg, sk_zckey) in
all paths reachable from other TCP states. Such as the EPIPE case
in sk_stream_wait_connect, which a sendmsg() in incorrect state will
now hit. Most paths are already safe.

Only extension needed is for TCP Fastopen active open. This can build
an skb with data in tcp_send_syn_data. Pass the uarg along with other
fastopen state, so that this skb also generates a zerocopy
notification on release.

Tested with active and passive tcp fastopen packetdrill scripts at
https://github.com/wdebruij/packetdrill/commit/1747eef03d25a2404e8132817d0f1244fd6f129d

Signed-off-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/tcp.h
net/ipv4/tcp.c
net/ipv4/tcp_output.c