Commit 6bfc93ef authored by Trond Myklebust's avatar Trond Myklebust

NFSv4: Teach NFSv4 to cache locks when we hold a delegation

 Now that we have a method of dealing with delegation recalls, actually
 enable the caching of posix and BSD locks.
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
parent 888e694c
...@@ -2918,6 +2918,10 @@ static int nfs4_proc_unlck(struct nfs4_state *state, int cmd, struct file_lock * ...@@ -2918,6 +2918,10 @@ static int nfs4_proc_unlck(struct nfs4_state *state, int cmd, struct file_lock *
struct nfs4_lock_state *lsp; struct nfs4_lock_state *lsp;
int status; int status;
/* Is this a delegated lock? */
if (test_bit(NFS_DELEGATED_STATE, &state->flags))
return do_vfs_lock(request->fl_file, request);
status = nfs4_set_lock_state(state, request); status = nfs4_set_lock_state(state, request);
if (status != 0) if (status != 0)
return status; return status;
...@@ -3032,6 +3036,9 @@ static int nfs4_lock_reclaim(struct nfs4_state *state, struct file_lock *request ...@@ -3032,6 +3036,9 @@ static int nfs4_lock_reclaim(struct nfs4_state *state, struct file_lock *request
struct nfs4_exception exception = { }; struct nfs4_exception exception = { };
int err; int err;
/* Cache the lock if possible... */
if (test_bit(NFS_DELEGATED_STATE, &state->flags))
return 0;
do { do {
err = _nfs4_do_setlk(state, F_SETLK, request, 1); err = _nfs4_do_setlk(state, F_SETLK, request, 1);
if (err != -NFS4ERR_DELAY) if (err != -NFS4ERR_DELAY)
...@@ -3047,6 +3054,9 @@ static int nfs4_lock_expired(struct nfs4_state *state, struct file_lock *request ...@@ -3047,6 +3054,9 @@ static int nfs4_lock_expired(struct nfs4_state *state, struct file_lock *request
struct nfs4_exception exception = { }; struct nfs4_exception exception = { };
int err; int err;
err = nfs4_set_lock_state(state, request);
if (err != 0)
return err;
do { do {
err = _nfs4_do_setlk(state, F_SETLK, request, 0); err = _nfs4_do_setlk(state, F_SETLK, request, 0);
if (err != -NFS4ERR_DELAY) if (err != -NFS4ERR_DELAY)
...@@ -3062,15 +3072,25 @@ static int _nfs4_proc_setlk(struct nfs4_state *state, int cmd, struct file_lock ...@@ -3062,15 +3072,25 @@ static int _nfs4_proc_setlk(struct nfs4_state *state, int cmd, struct file_lock
int status; int status;
down_read(&clp->cl_sem); down_read(&clp->cl_sem);
status = nfs4_set_lock_state(state, request); /* Is this a delegated open? */
if (status == 0) if (test_bit(NFS_DELEGATED_STATE, &state->flags)) {
status = _nfs4_do_setlk(state, cmd, request, 0); /* Yes: cache locks! */
if (status == 0) { status = do_vfs_lock(request->fl_file, request);
/* Note: we always want to sleep here! */ /* ...but avoid races with delegation recall... */
request->fl_flags |= FL_SLEEP; if (status < 0 || test_bit(NFS_DELEGATED_STATE, &state->flags))
if (do_vfs_lock(request->fl_file, request) < 0) goto out;
printk(KERN_WARNING "%s: VFS is out of sync with lock manager!\n", __FUNCTION__);
} }
status = nfs4_set_lock_state(state, request);
if (status != 0)
goto out;
status = _nfs4_do_setlk(state, cmd, request, 0);
if (status != 0)
goto out;
/* Note: we always want to sleep here! */
request->fl_flags |= FL_SLEEP;
if (do_vfs_lock(request->fl_file, request) < 0)
printk(KERN_WARNING "%s: VFS is out of sync with lock manager!\n", __FUNCTION__);
out:
up_read(&clp->cl_sem); up_read(&clp->cl_sem);
return status; return status;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment