diff options
| author | J. Bruce Fields <bfields@redhat.com> | 2014-08-20 16:18:32 -0400 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2014-08-28 15:51:35 -0400 |
| commit | ae89254da6879cffa6a17327e5f3f60217b718cf (patch) | |
| tree | 0dfb859c94c64bc53c491038d36ca1d06cafce8b | |
| parent | 18c01ab30288d9d0a7d80b08b659531f37ed379d (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>
| -rw-r--r-- | net/sunrpc/svc_xprt.c | 4 |
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) "svc_recv: service %p, transport not NULL!\n", rqstp); - /* Make sure the task pointer is set! */ - if (WARN_ON_ONCE(!rqstp->rq_task)) - rqstp->rq_task = current_task; - err = svc_alloc_arg(rqstp); if (err) return err; |