]> git.baikalelectronics.ru Git - kernel.git/commit
tcp: do_tcp_sendpages() must try to push data out on oom conditions
authorWilly Tarreau <w@1wt.eu>
Thu, 17 May 2012 11:14:14 +0000 (11:14 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 17 May 2012 22:31:43 +0000 (18:31 -0400)
commit23a1ec2e1fc354bed91ceb977061592e19df49c7
treef2da25538f8fcf48ae7a74b3b040d58628b52094
parent0f5e7554e7eca97c614a0183112feb7937458e95
tcp: do_tcp_sendpages() must try to push data out on oom conditions

Since recent changes on TCP splicing (starting with commits 11038879
"tcp: allow splice() to build full TSO packets" and 0b9cb7c3 "tcp:
tcp_sendpages() should call tcp_push() once"), I started seeing
massive stalls when forwarding traffic between two sockets using
splice() when pipe buffers were larger than socket buffers.

Latest changes (net: netdev_alloc_skb() use build_skb()) made the
problem even more apparent.

The reason seems to be that if do_tcp_sendpages() fails on out of memory
condition without being able to send at least one byte, tcp_push() is not
called and the buffers cannot be flushed.

After applying the attached patch, I cannot reproduce the stalls at all
and the data rate it perfectly stable and steady under any condition
which previously caused the problem to be permanent.

The issue seems to have been there since before the kernel migrated to
git, which makes me think that the stalls I occasionally experienced
with tux during stress-tests years ago were probably related to the
same issue.

This issue was first encountered on 3.0.31 and 3.2.17, so please backport
to -stable.

Signed-off-by: Willy Tarreau <w@1wt.eu>
Acked-by: Eric Dumazet <edumazet@google.com>
Cc: <stable@vger.kernel.org>
net/ipv4/tcp.c