diff options
author | Jeff Layton <jlayton@kernel.org> | 2022-09-26 21:41:01 +0300 |
---|---|---|
committer | Chuck Lever <chuck.lever@oracle.com> | 2022-09-26 21:50:57 +0300 |
commit | b95239ca4954a0d48b19c09ce7e8f31b453b4216 (patch) | |
tree | f675a2b66ad391f8dbe42c4f1516e1b54a6a1151 /fs/nfsd/state.h | |
parent | 25fbe1fca14142beae6c882f7906510363d42bff (diff) | |
download | linux-b95239ca4954a0d48b19c09ce7e8f31b453b4216.tar.xz |
nfsd: make nfsd4_run_cb a bool return function
queue_work can return false and not queue anything, if the work is
already queued. If that happens in the case of a CB_RECALL, we'll have
taken an extra reference to the stid that will never be put. Ensure we
throw a warning in that case.
Signed-off-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'fs/nfsd/state.h')
-rw-r--r-- | fs/nfsd/state.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfsd/state.h b/fs/nfsd/state.h index b3477087a9fc..e2daef3cc003 100644 --- a/fs/nfsd/state.h +++ b/fs/nfsd/state.h @@ -692,7 +692,7 @@ extern void nfsd4_probe_callback_sync(struct nfs4_client *clp); extern void nfsd4_change_callback(struct nfs4_client *clp, struct nfs4_cb_conn *); extern void nfsd4_init_cb(struct nfsd4_callback *cb, struct nfs4_client *clp, const struct nfsd4_callback_ops *ops, enum nfsd4_cb_op op); -extern void nfsd4_run_cb(struct nfsd4_callback *cb); +extern bool nfsd4_run_cb(struct nfsd4_callback *cb); extern int nfsd4_create_callback_queue(void); extern void nfsd4_destroy_callback_queue(void); extern void nfsd4_shutdown_callback(struct nfs4_client *); |