diff options
| author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2013-04-01 14:27:29 -0400 |
|---|---|---|
| committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2013-04-05 17:03:53 -0400 |
| commit | dbb21c25a35a71baf413f5176f028ee11b88cfbc (patch) | |
| tree | 4d8a13d4c22bf9b8c8adcbe0f75066c1c6e5debe | |
| parent | cb6071874703565762c190e1e7ebde762e525063 (diff) | |
NFSv4: Handle NFS4ERR_DELAY and NFS4ERR_GRACE in nfs4_lock_delegation_recall
A server shouldn't normally return NFS4ERR_GRACE if the client holds a
delegation, since no conflicting lock reclaims can be granted, however
the spec does not require the server to grant the lock in this
instance.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: stable@vger.kernel.org
| -rw-r--r-- | fs/nfs/nfs4proc.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 81343944e096..dc4037c52cf7 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c | |||
| @@ -5176,6 +5176,12 @@ int nfs4_lock_delegation_recall(struct nfs4_state *state, struct file_lock *fl) | |||
| 5176 | nfs4_schedule_stateid_recovery(server, state); | 5176 | nfs4_schedule_stateid_recovery(server, state); |
| 5177 | err = 0; | 5177 | err = 0; |
| 5178 | goto out; | 5178 | goto out; |
| 5179 | case -NFS4ERR_DELAY: | ||
| 5180 | case -NFS4ERR_GRACE: | ||
| 5181 | set_bit(NFS_DELEGATED_STATE, &state->flags); | ||
| 5182 | ssleep(1); | ||
| 5183 | err = -EAGAIN; | ||
| 5184 | goto out; | ||
| 5179 | case -ENOMEM: | 5185 | case -ENOMEM: |
| 5180 | case -NFS4ERR_DENIED: | 5186 | case -NFS4ERR_DENIED: |
| 5181 | /* kill_proc(fl->fl_pid, SIGLOST, 1); */ | 5187 | /* kill_proc(fl->fl_pid, SIGLOST, 1); */ |
