diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2010-10-23 17:16:10 +0200 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2010-10-25 23:59:32 +0200 |
| commit | ac9303eb74471bc2567960b47497a8bfbe1e5a03 (patch) | |
| tree | 17f9882d81b56d2787d0c556cfa6d76ea40a948b /net | |
| parent | svcrpc: never clear XPT_BUSY on dead xprt (diff) | |
| download | linux-ac9303eb74471bc2567960b47497a8bfbe1e5a03.tar.xz linux-ac9303eb74471bc2567960b47497a8bfbe1e5a03.zip | |
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')
| -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) /* Only do this once */ if (test_and_set_bit(XPT_DEAD, &xprt->xpt_flags)) - return; + BUG(); dprintk("svc: svc_delete_xprt(%p)\n", xprt); xprt->xpt_ops->xpo_detach(xprt); |