* [PATCH] nfs41: sunrpc: xs_tcp_read_callback() should use xprt_force_disconnect()
@ 2009-05-26 17:32 Ricardo Labiaga
2009-05-28 11:22 ` Benny Halevy
0 siblings, 1 reply; 2+ messages in thread
From: Ricardo Labiaga @ 2009-05-26 17:32 UTC (permalink / raw)
To: trond.myklebust; +Cc: bhalevy, pnfs, linux-nfs, Ricardo Labiaga
xs_tcp_read_callback() drops the connection when the number of expected
callbacks is exceeded. Use xprt_force_disconnect(), ensuring tasks on
the pending queue are awaken on disconnect.
Signed-off-by: Ricardo Labiaga <Ricardo.Labiaga@netapp.com>
---
net/sunrpc/xprtsock.c | 7 +------
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c
index 1a268bf..7445d4b 100644
--- a/net/sunrpc/xprtsock.c
+++ b/net/sunrpc/xprtsock.c
@@ -1147,13 +1147,8 @@ static inline int xs_tcp_read_callback(struct rpc_xprt *xprt,
req = xprt_alloc_bc_request(xprt);
if (req == NULL) {
- /*
- * Schedule an autoclose RPC call
- */
printk(KERN_WARNING "Callback slot table overflowed\n");
- set_bit(XPRT_CLOSE_WAIT, &xprt->state);
- if (test_and_set_bit(XPRT_LOCKED, &xprt->state) == 0)
- queue_work(rpciod_workqueue, &xprt->task_cleanup);
+ xprt_force_disconnect(xprt);
return -1;
}
--
1.5.4.3
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] nfs41: sunrpc: xs_tcp_read_callback() should use xprt_force_disconnect()
2009-05-26 17:32 [PATCH] nfs41: sunrpc: xs_tcp_read_callback() should use xprt_force_disconnect() Ricardo Labiaga
@ 2009-05-28 11:22 ` Benny Halevy
0 siblings, 0 replies; 2+ messages in thread
From: Benny Halevy @ 2009-05-28 11:22 UTC (permalink / raw)
To: Ricardo Labiaga; +Cc: trond.myklebust, pnfs, linux-nfs
Merged onto my nfs41-all and nfs41-for-{2.6.31,next} branches.
Thanks!
Benny
On May. 26, 2009, 20:32 +0300, Ricardo Labiaga <Ricardo.Labiaga@netapp.com> wrote:
> xs_tcp_read_callback() drops the connection when the number of expected
> callbacks is exceeded. Use xprt_force_disconnect(), ensuring tasks on
> the pending queue are awaken on disconnect.
>
> Signed-off-by: Ricardo Labiaga <Ricardo.Labiaga@netapp.com>
> ---
> net/sunrpc/xprtsock.c | 7 +------
> 1 files changed, 1 insertions(+), 6 deletions(-)
>
> diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c
> index 1a268bf..7445d4b 100644
> --- a/net/sunrpc/xprtsock.c
> +++ b/net/sunrpc/xprtsock.c
> @@ -1147,13 +1147,8 @@ static inline int xs_tcp_read_callback(struct rpc_xprt *xprt,
>
> req = xprt_alloc_bc_request(xprt);
> if (req == NULL) {
> - /*
> - * Schedule an autoclose RPC call
> - */
> printk(KERN_WARNING "Callback slot table overflowed\n");
> - set_bit(XPRT_CLOSE_WAIT, &xprt->state);
> - if (test_and_set_bit(XPRT_LOCKED, &xprt->state) == 0)
> - queue_work(rpciod_workqueue, &xprt->task_cleanup);
> + xprt_force_disconnect(xprt);
> return -1;
> }
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-05-28 11:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-05-26 17:32 [PATCH] nfs41: sunrpc: xs_tcp_read_callback() should use xprt_force_disconnect() Ricardo Labiaga
2009-05-28 11:22 ` Benny Halevy
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.