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 /net | |
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>
Diffstat (limited to 'net')
-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) | |||
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; |