diff options
author | NeilBrown <neilb@suse.de> | 2021-11-29 05:51:25 +0100 |
---|---|---|
committer | Chuck Lever <chuck.lever@oracle.com> | 2021-12-13 19:42:50 +0100 |
commit | df5e49c880ea0776806b8a9f8ab95e035272cf6f (patch) | |
tree | 57020650034662cd4daa43163b5e06403b801997 /fs/lockd/svc.c | |
parent | NFSD: handle errors better in write_ports_addfd() (diff) | |
download | linux-df5e49c880ea0776806b8a9f8ab95e035272cf6f.tar.xz linux-df5e49c880ea0776806b8a9f8ab95e035272cf6f.zip |
SUNRPC: change svc_get() to return the svc.
It is common for 'get' functions to return the object that was 'got',
and there are a couple of places where users of svc_get() would be a
little simpler if svc_get() did that.
Make it so.
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'fs/lockd/svc.c')
-rw-r--r-- | fs/lockd/svc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/lockd/svc.c b/fs/lockd/svc.c index b220e1b91726..2f50d5b2a8a4 100644 --- a/fs/lockd/svc.c +++ b/fs/lockd/svc.c @@ -430,14 +430,12 @@ static struct svc_serv *lockd_create_svc(void) /* * Check whether we're already up and running. */ - if (nlmsvc_rqst) { + if (nlmsvc_rqst) /* * Note: increase service usage, because later in case of error * svc_destroy() will be called. */ - svc_get(nlmsvc_rqst->rq_server); - return nlmsvc_rqst->rq_server; - } + return svc_get(nlmsvc_rqst->rq_server); /* * Sanity check: if there's no pid, |