diff options
author | J. Bruce Fields <bfields@redhat.com> | 2010-10-23 11:16:10 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2010-10-25 17:59:32 -0400 |
commit | ac9303eb74471bc2567960b47497a8bfbe1e5a03 (patch) | |
tree | 17f9882d81b56d2787d0c556cfa6d76ea40a948b /net/sunrpc/svc_xprt.c | |
parent | 7e4fdd0744fcb9f08854c37643bf529c5945cc36 (diff) |
svcrpc: assume svc_delete_xprt() called only once
As long as DEAD exports are left BUSY, and svc_delete_xprt is called
only with BUSY held, then svc_delete_xprt() will never be called on an
xprt that is already DEAD.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/sunrpc/svc_xprt.c')
-rw-r--r-- | net/sunrpc/svc_xprt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sunrpc/svc_xprt.c b/net/sunrpc/svc_xprt.c index 1b0673c6e986..c65a47218290 100644 --- a/net/sunrpc/svc_xprt.c +++ b/net/sunrpc/svc_xprt.c | |||
@@ -903,7 +903,7 @@ void svc_delete_xprt(struct svc_xprt *xprt) | |||
903 | 903 | ||
904 | /* Only do this once */ | 904 | /* Only do this once */ |
905 | if (test_and_set_bit(XPT_DEAD, &xprt->xpt_flags)) | 905 | if (test_and_set_bit(XPT_DEAD, &xprt->xpt_flags)) |
906 | return; | 906 | BUG(); |
907 | 907 | ||
908 | dprintk("svc: svc_delete_xprt(%p)\n", xprt); | 908 | dprintk("svc: svc_delete_xprt(%p)\n", xprt); |
909 | xprt->xpt_ops->xpo_detach(xprt); | 909 | xprt->xpt_ops->xpo_detach(xprt); |