Commit da56ac02 authored by Olga Kornievskaia's avatar Olga Kornievskaia Committed by Greg Kroah-Hartman

Fixing lease renewal

commit 8faaa6d5 upstream.

Commit c9fdeb28 removed a 'continue' after checking if the lease needs
to be renewed. However, if client hasn't moved, the code falls down to
starting reboot recovery erroneously (ie., sends open reclaim and gets
back stale_clientid error) before recovering from getting stale_clientid
on the renew operation.
Signed-off-by: default avatarOlga Kornievskaia <kolga@netapp.com>
Fixes: c9fdeb28 (NFS: Add basic migration support to state manager thread)
Signed-off-by: default avatarTrond Myklebust <trond.myklebust@primarydata.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0238de7a
...@@ -2372,6 +2372,7 @@ static void nfs4_state_manager(struct nfs_client *clp) ...@@ -2372,6 +2372,7 @@ static void nfs4_state_manager(struct nfs_client *clp)
status = nfs4_check_lease(clp); status = nfs4_check_lease(clp);
if (status < 0) if (status < 0)
goto out_error; goto out_error;
continue;
} }
if (test_and_clear_bit(NFS4CLNT_MOVED, &clp->cl_state)) { if (test_and_clear_bit(NFS4CLNT_MOVED, &clp->cl_state)) {
......
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