diff options
author | Paolo Abeni <pabeni@redhat.com> | 2020-07-23 13:02:35 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2020-07-23 11:47:25 -0700 |
commit | 97e617518cbc318113b034a5fb33f49c81701278 (patch) | |
tree | 3fd30a2498ad4b22ce6758d804761d7f3438bd52 /net/mptcp | |
parent | b7514694ed2952684a1e4fc44d83682140fd8cef (diff) | |
download | lwn-97e617518cbc318113b034a5fb33f49c81701278.tar.gz lwn-97e617518cbc318113b034a5fb33f49c81701278.zip |
subflow: use rsk_ops->send_reset()
tcp_send_active_reset() is more prone to transient errors
(memory allocation or xmit queue full): in stress conditions
the kernel may drop the egress packet, and the client will be
stuck.
Reviewed-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
Tested-by: Christoph Paasch <cpaasch@apple.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/mptcp')
-rw-r--r-- | net/mptcp/subflow.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mptcp/subflow.c b/net/mptcp/subflow.c index 3ef445f59556..ada04df6f99f 100644 --- a/net/mptcp/subflow.c +++ b/net/mptcp/subflow.c @@ -524,9 +524,9 @@ out: dispose_child: subflow_drop_ctx(child); tcp_rsk(req)->drop_req = true; - tcp_send_active_reset(child, GFP_ATOMIC); inet_csk_prepare_for_destroy_sock(child); tcp_done(child); + req->rsk_ops->send_reset(sk, skb); /* The last child reference will be released by the caller */ return child; |