aboutsummaryrefslogtreecommitdiffstats
path: root/net/sunrpc
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@redhat.com>2014-08-20 16:18:32 -0400
committerJ. Bruce Fields <bfields@redhat.com>2014-08-28 15:51:35 -0400
commitae89254da6879cffa6a17327e5f3f60217b718cf (patch)
tree0dfb859c94c64bc53c491038d36ca1d06cafce8b /net/sunrpc
parent18c01ab30288d9d0a7d80b08b659531f37ed379d (diff)
SUNRPC: Fix compile on non-x86
current_task appears to be x86-only, oops. Let's just delete this check entirely: Any developer that adds a new user without setting rq_task will get a crash the first time they test it. I also don't think there are normally any important locks held here, and I can't see any other reason why killing a server thread would bring the whole box down. So the effort to fail gracefully here looks like overkill. Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> Fixes: 983c684466e0 "SUNRPC: get rid of the request wait queue" Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'net/sunrpc')
-rw-r--r--net/sunrpc/svc_xprt.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/net/sunrpc/svc_xprt.c b/net/sunrpc/svc_xprt.c
index c0db66d81e34..c179ca2a5aa4 100644
--- a/net/sunrpc/svc_xprt.c
+++ b/net/sunrpc/svc_xprt.c
@@ -771,10 +771,6 @@ int svc_recv(struct svc_rqst *rqstp, long timeout)
771 "svc_recv: service %p, transport not NULL!\n", 771 "svc_recv: service %p, transport not NULL!\n",
772 rqstp); 772 rqstp);
773 773
774 /* Make sure the task pointer is set! */
775 if (WARN_ON_ONCE(!rqstp->rq_task))
776 rqstp->rq_task = current_task;
777
778 err = svc_alloc_arg(rqstp); 774 err = svc_alloc_arg(rqstp);
779 if (err) 775 if (err)
780 return err; 776 return err;