diff options
author | J. Bruce Fields <bfields@redhat.com> | 2010-10-23 11:23:46 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2010-10-25 17:59:33 -0400 |
commit | 01dba075d571f5a8b7dcb153fdfd14e981c4cee3 (patch) | |
tree | b443dff71fd4addcd11b873f356bbd247cadaa43 | |
parent | ac9303eb74471bc2567960b47497a8bfbe1e5a03 (diff) |
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>
-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) | |||
330 | "svc_xprt_enqueue: " | 330 | "svc_xprt_enqueue: " |
331 | "threads and transports both waiting??\n"); | 331 | "threads and transports both waiting??\n"); |
332 | 332 | ||
333 | if (test_bit(XPT_DEAD, &xprt->xpt_flags)) { | ||
334 | /* Don't enqueue dead transports */ | ||
335 | dprintk("svc: transport %p is dead, not enqueued\n", xprt); | ||
336 | goto out_unlock; | ||
337 | } | ||
338 | |||
339 | pool->sp_stats.packets++; | 333 | pool->sp_stats.packets++; |
340 | 334 | ||
341 | /* Mark transport as busy. It will remain in this state until | 335 | /* Mark transport as busy. It will remain in this state until |