diff options
author | J. Bruce Fields <bfields@citi.umich.edu> | 2009-04-29 19:45:36 +0200 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2009-04-29 22:44:34 +0200 |
commit | f64f79ea5f5e02ba8585f35a10b4a3bcab0cea52 (patch) | |
tree | 0b78de112c3a015c3f719064d1fa86583c7d5f0b /fs/nfsd/nfs4state.c | |
parent | nfsd: quiet compile warning (diff) | |
download | linux-f64f79ea5f5e02ba8585f35a10b4a3bcab0cea52.tar.xz linux-f64f79ea5f5e02ba8585f35a10b4a3bcab0cea52.zip |
nfsd4: setclientid_confirm callback-change fixes
This setclientid_confirm case should allow the client to change
callbacks, but it currently has a dummy implementation that just turns
off callbacks completely. That dummy implementation isn't completely
correct either, though:
- There's no need to remove any client recovery directory in
this case.
- New clientid confirm verifiers should be generated (and
returned) in setclientid; there's no need to generate a new
one here.
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/nfsd/nfs4state.c')
-rw-r--r-- | fs/nfsd/nfs4state.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index d24dd12ddb4d..7e1fcc3aade4 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c @@ -1688,8 +1688,6 @@ nfsd4_setclientid_confirm(struct svc_rqst *rqstp, /* XXX: We just turn off callbacks until we can handle * change request correctly. */ atomic_set(&conf->cl_callback.cb_set, 0); - gen_confirm(conf); - nfsd4_remove_clid_dir(unconf); expire_client(unconf); status = nfs_ok; |