diff options
| author | Eric Dumazet <[email protected]> | 2021-03-11 12:35:06 -0800 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2021-03-11 18:35:31 -0800 |
| commit | ac3959fd0dcc0e49298ea5cadfe257db0e58ef8b (patch) | |
| tree | 7c530d5490a1fc493fd3df9483501b01bd040d3a /tools/perf/scripts/python/stackcollapse.py | |
| parent | a7abf3cd76e1e190a2c22afe5ffd0f695c7641b0 (diff) | |
tcp: remove obsolete check in __tcp_retransmit_skb()
TSQ provides a nice way to avoid bufferbloat on individual socket,
including retransmit packets. We can get rid of the old
heuristic:
/* Do not sent more than we queued. 1/4 is reserved for possible
* copying overhead: fragmentation, tunneling, mangling etc.
*/
if (refcount_read(&sk->sk_wmem_alloc) >
min_t(u32, sk->sk_wmem_queued + (sk->sk_wmem_queued >> 2),
sk->sk_sndbuf))
return -EAGAIN;
This heuristic was giving false positives according to Jakub,
whenever TX completions are delayed above RTT. (Ack packets
are processed by TCP stack before clones are orphaned/freed)
Signed-off-by: Eric Dumazet <[email protected]>
Reported-by: Jakub Kicinski <[email protected]>
Cc: Neal Cardwell <[email protected]>
Cc: Yuchung Cheng <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions