]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
NFSv4.1: Handle session reset and bind_conn_to_session before lease check
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 21 Nov 2012 14:22:14 +0000 (09:22 -0500)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 21 Nov 2012 14:28:42 +0000 (09:28 -0500)
We can't send a SEQUENCE op unless the session is OK, so it is pointless
to handle the CHECK_LEASE state before we've dealt with SESSION_RESET
and BIND_CONN_TO_SESSION.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/nfs4state.c

index e0a28dffd29dc20ee472fa634a4165208873c8aa..f3d1bc48c9c4a273d99ba86e58b2f12e70154a4a 100644 (file)
@@ -2114,15 +2114,6 @@ static void nfs4_state_manager(struct nfs_client *clp)
                        continue;
                }
 
-               if (test_and_clear_bit(NFS4CLNT_CHECK_LEASE, &clp->cl_state)) {
-                       section = "check lease";
-                       status = nfs4_check_lease(clp);
-                       if (status < 0)
-                               goto out_error;
-                       if (test_bit(NFS4CLNT_LEASE_EXPIRED, &clp->cl_state))
-                               continue;
-               }
-
                /* Initialize or reset the session */
                if (test_and_clear_bit(NFS4CLNT_SESSION_RESET, &clp->cl_state)) {
                        section = "reset session";
@@ -2143,6 +2134,14 @@ static void nfs4_state_manager(struct nfs_client *clp)
                        continue;
                }
 
+               if (test_and_clear_bit(NFS4CLNT_CHECK_LEASE, &clp->cl_state)) {
+                       section = "check lease";
+                       status = nfs4_check_lease(clp);
+                       if (status < 0)
+                               goto out_error;
+                       continue;
+               }
+
                /* Recall session slots */
                if (test_and_clear_bit(NFS4CLNT_RECALL_SLOT, &clp->cl_state)) {
                        section = "recall slot";