diff options
author | Eric Dumazet <edumazet@google.com> | 2019-12-12 12:55:30 -0800 |
---|---|---|
committer | Jakub Kicinski <jakub.kicinski@netronome.com> | 2019-12-13 21:58:40 -0800 |
commit | ee2aabd3fc2eef4c1a0ebdadccc76fbff74b94fc (patch) | |
tree | ff49c797d50165533687b248d7531010a9b24339 /usr | |
parent | 1f85e6267caca44b30c54711652b0726fadbb131 (diff) |
tcp: refine tcp_write_queue_empty() implementation
Due to how tcp_sendmsg() is implemented, we can have an empty
skb at the tail of the write queue.
Most [1] tcp_write_queue_empty() callers want to know if there is
anything to send (payload and/or FIN)
Instead of checking if the sk_write_queue is empty, we need
to test if tp->write_seq == tp->snd_nxt
[1] tcp_send_fin() was the only caller that expected to
see if an skb was in the write queue, I have changed the code
to reuse the tcp_write_queue_tail() result.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Neal Cardwell <ncardwell@google.com>
Acked-by: Soheil Hassas Yeganeh <soheil@google.com>
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Diffstat (limited to 'usr')
0 files changed, 0 insertions, 0 deletions