diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2009-03-11 15:29:24 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2009-03-19 15:17:36 -0400 |
commit | 55420c24a0d4d1fce70ca713f84aa00b6b74a70e (patch) | |
tree | 9e7e931122cb77e7ae1249b6dde81d83f60a894d /net | |
parent | b61d59fffd3e5b6037c92b4c840605831de8a251 (diff) |
SUNRPC: Ensure we close the socket on EPIPE errors too...
As long as one task is holding the socket lock, then calls to
xprt_force_disconnect(xprt) will not succeed in shutting down the socket.
In particular, this would mean that a server initiated shutdown will not
succeed until the lock is relinquished.
In order to avoid the deadlock, we should ensure that xs_tcp_send_request()
closes the socket on EPIPE errors too.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/sunrpc/xprtsock.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c index f05a56e597ef..fbc8725c20cb 100644 --- a/net/sunrpc/xprtsock.c +++ b/net/sunrpc/xprtsock.c | |||
@@ -726,10 +726,10 @@ static int xs_tcp_send_request(struct rpc_task *task) | |||
726 | dprintk("RPC: sendmsg returned unrecognized error %d\n", | 726 | dprintk("RPC: sendmsg returned unrecognized error %d\n", |
727 | -status); | 727 | -status); |
728 | case -ECONNRESET: | 728 | case -ECONNRESET: |
729 | case -EPIPE: | ||
729 | xs_tcp_shutdown(xprt); | 730 | xs_tcp_shutdown(xprt); |
730 | case -ECONNREFUSED: | 731 | case -ECONNREFUSED: |
731 | case -ENOTCONN: | 732 | case -ENOTCONN: |
732 | case -EPIPE: | ||
733 | clear_bit(SOCK_ASYNC_NOSPACE, &transport->sock->flags); | 733 | clear_bit(SOCK_ASYNC_NOSPACE, &transport->sock->flags); |
734 | } | 734 | } |
735 | out: | 735 | out: |