tcp: fix possible deadlock in tcp_send_fin()
authorEric Dumazet <edumazet@google.com>
Wed, 22 Apr 2015 01:32:24 +0000 (18:32 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 22 Apr 2015 18:13:11 +0000 (14:13 -0400)
commitd83769a580f1132ac26439f50068a29b02be535e
treeb18a104e87454d5505c3d0e5f1c2e7f26ce9b89c
parent5e6c94a999f67f120c6bbba71bbee840dfee6338
tcp: fix possible deadlock in tcp_send_fin()

Using sk_stream_alloc_skb() in tcp_send_fin() is dangerous in
case a huge process is killed by OOM, and tcp_mem[2] is hit.

To be able to free memory we need to make progress, so this
patch allows FIN packets to not care about tcp_mem[2], if
skb allocation succeeded.

In a follow-up patch, we might abort tcp_send_fin() infinite loop
in case TIF_MEMDIE is set on this thread, as memory allocator
did its best getting extra memory already.

This patch reverts d22e15371811 ("tcp: fix tcp fin memory accounting")

Fixes: d22e15371811 ("tcp: fix tcp fin memory accounting")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_output.c