nfsd: simplify nfsd_renew
[ Upstream commit b4587eb2cf4b6271f67fb93b75f7de2a2026e853 ] You can take the single-exit thing too far, I think. Signed-off-by: J. Bruce Fields <bfields@redhat.com> Signed-off-by: Chuck Lever <chuck.lever@oracle.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
238fe9021e
commit
01b03fcec1
1 changed files with 3 additions and 6 deletions
|
@ -5322,15 +5322,12 @@ nfsd4_renew(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
|
||||||
trace_nfsd_clid_renew(clid);
|
trace_nfsd_clid_renew(clid);
|
||||||
status = lookup_clientid(clid, cstate, nn, false);
|
status = lookup_clientid(clid, cstate, nn, false);
|
||||||
if (status)
|
if (status)
|
||||||
goto out;
|
return status;
|
||||||
clp = cstate->clp;
|
clp = cstate->clp;
|
||||||
status = nfserr_cb_path_down;
|
|
||||||
if (!list_empty(&clp->cl_delegations)
|
if (!list_empty(&clp->cl_delegations)
|
||||||
&& clp->cl_cb_state != NFSD4_CB_UP)
|
&& clp->cl_cb_state != NFSD4_CB_UP)
|
||||||
goto out;
|
return nfserr_cb_path_down;
|
||||||
status = nfs_ok;
|
return nfs_ok;
|
||||||
out:
|
|
||||||
return status;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
void
|
void
|
||||||
|
|
Loading…
Add table
Reference in a new issue