diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2010-10-23 17:23:46 +0200 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2010-10-25 23:59:33 +0200 |
| commit | 01dba075d571f5a8b7dcb153fdfd14e981c4cee3 (patch) | |
| tree | b443dff71fd4addcd11b873f356bbd247cadaa43 /net | |
| parent | svcrpc: assume svc_delete_xprt() called only once (diff) | |
| download | linux-01dba075d571f5a8b7dcb153fdfd14e981c4cee3.tar.xz linux-01dba075d571f5a8b7dcb153fdfd14e981c4cee3.zip | |
svcrpc: no need for XPT_DEAD check in svc_xprt_enqueue
If any xprt marked DEAD is also left BUSY for the rest of its life, then
the XPT_DEAD check here is superfluous--we'll get the same result from
the XPT_BUSY check just after.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net')
| -rw-r--r-- | net/sunrpc/svc_xprt.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/net/sunrpc/svc_xprt.c b/net/sunrpc/svc_xprt.c index c65a47218290..c82fe739fbdc 100644 --- a/net/sunrpc/svc_xprt.c +++ b/net/sunrpc/svc_xprt.c @@ -330,12 +330,6 @@ void svc_xprt_enqueue(struct svc_xprt *xprt) "svc_xprt_enqueue: " "threads and transports both waiting??\n"); - if (test_bit(XPT_DEAD, &xprt->xpt_flags)) { - /* Don't enqueue dead transports */ - dprintk("svc: transport %p is dead, not enqueued\n", xprt); - goto out_unlock; - } - pool->sp_stats.packets++; /* Mark transport as busy. It will remain in this state until |