diff options
| author |   Eric Dumazet <edumazet@google.com> | 2013-04-12 11:31:52 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2013-04-12 18:17:06 -0400 |
| commit | d6a4a10411764cf1c3a5dad4f06c5ebe5194488b (patch) | |
| tree | 54d1e4c0c92bc780d9e2f8a822c1c8dc271df70d /net/ipv4/tcp_output.c | |
| parent | act_csum: fix possible use after free (diff) | |
| download | linux-dev-d6a4a10411764cf1c3a5dad4f06c5ebe5194488b.tar.xz linux-dev-d6a4a10411764cf1c3a5dad4f06c5ebe5194488b.zip | |
tcp: GSO should be TSQ friendly
I noticed that TSQ (TCP Small queues) was less effective when TSO is
turned off, and GSO is on. If BQL is not enabled, TSQ has then no
effect.
It turns out the GSO engine frees the original gso_skb at the time the
fragments are generated and queued to the NIC.
We should instead call the tcp_wfree() destructor for the last fragment,
to keep the flow control as intended in TSQ. This effectively limits
the number of queued packets on qdisc + NIC layers.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Tom Herbert <therbert@google.com>
Cc: Yuchung Cheng <ycheng@google.com>
Cc: Nandita Dukkipati <nanditad@google.com>
Cc: Neal Cardwell <ncardwell@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_output.c')
| -rw-r--r-- | net/ipv4/tcp_output.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index af354c98fdb5..d12694353540 100644 --- a/net/ipv4/tcp_output.c +++ b/net/ipv4/tcp_output.c @@ -787,7 +787,7 @@ void __init tcp_tasklet_init(void) * We cant xmit new skbs from this context, as we might already * hold qdisc lock. */ -static void tcp_wfree(struct sk_buff *skb) +void tcp_wfree(struct sk_buff *skb) { struct sock *sk = skb->sk; struct tcp_sock *tp = tcp_sk(sk); |