diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-10-04 17:59:08 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-10-19 19:42:33 -0400 |
commit | ae1007d37e00144b72906a4bdc47d517ae91bcc1 (patch) | |
tree | 170a123bd99aefbb086bef9ab75a576ea5df346a /fs/nfs/nfs4proc.c | |
parent | b0ed9dbc24f1fd912b2dd08b995153cafc1d5b1c (diff) |
NFSv4: Don't call nfs4_state_mark_reclaim_reboot() from error handlers
In the case of a server reboot, the state recovery thread starts by calling
nfs4_state_end_reclaim_reboot() in order to avoid edge conditions when
the server reboots while the client is in the middle of recovery.
However, if the client has already marked the nfs4_state as requiring
reboot recovery, then the above behaviour will cause the recovery thread to
treat the open as if it was part of such an edge condition: the open will
be recovered as if it was part of a lease expiration (and all the locks
will be lost).
Fix is to remove the call to nfs4_state_mark_reclaim_reboot from
nfs4_async_handle_error(), and nfs4_handle_exception(). Instead we leave it
to the recovery thread to do this for us.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: stable@kernel.org
Diffstat (limited to 'fs/nfs/nfs4proc.c')
-rw-r--r-- | fs/nfs/nfs4proc.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 7f25246bf38c..88a9d93ec1bd 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c | |||
@@ -255,9 +255,6 @@ static int nfs4_handle_exception(const struct nfs_server *server, int errorcode, | |||
255 | nfs4_state_mark_reclaim_nograce(clp, state); | 255 | nfs4_state_mark_reclaim_nograce(clp, state); |
256 | goto do_state_recovery; | 256 | goto do_state_recovery; |
257 | case -NFS4ERR_STALE_STATEID: | 257 | case -NFS4ERR_STALE_STATEID: |
258 | if (state == NULL) | ||
259 | break; | ||
260 | nfs4_state_mark_reclaim_reboot(clp, state); | ||
261 | case -NFS4ERR_STALE_CLIENTID: | 258 | case -NFS4ERR_STALE_CLIENTID: |
262 | case -NFS4ERR_EXPIRED: | 259 | case -NFS4ERR_EXPIRED: |
263 | goto do_state_recovery; | 260 | goto do_state_recovery; |
@@ -3414,9 +3411,6 @@ nfs4_async_handle_error(struct rpc_task *task, const struct nfs_server *server, | |||
3414 | nfs4_state_mark_reclaim_nograce(clp, state); | 3411 | nfs4_state_mark_reclaim_nograce(clp, state); |
3415 | goto do_state_recovery; | 3412 | goto do_state_recovery; |
3416 | case -NFS4ERR_STALE_STATEID: | 3413 | case -NFS4ERR_STALE_STATEID: |
3417 | if (state == NULL) | ||
3418 | break; | ||
3419 | nfs4_state_mark_reclaim_reboot(clp, state); | ||
3420 | case -NFS4ERR_STALE_CLIENTID: | 3414 | case -NFS4ERR_STALE_CLIENTID: |
3421 | case -NFS4ERR_EXPIRED: | 3415 | case -NFS4ERR_EXPIRED: |
3422 | goto do_state_recovery; | 3416 | goto do_state_recovery; |