nfsd4: split up find_stateid
authorJ. Bruce Fields <bfields@redhat.com>
Tue, 6 Sep 2011 20:48:57 +0000 (16:48 -0400)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 13 Sep 2011 22:27:31 +0000 (18:27 -0400)
Minor cleanup.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4state.c

index f0eccc236a0da515bbd7775b7f3a70c7e9f36f9d..aa088bc3b1696398e5e4d558568bc49e10a98604 100644 (file)
@@ -1068,21 +1068,29 @@ same_stateid(stateid_t *id_one, stateid_t *id_two)
        return id_one->si_fileid == id_two->si_fileid;
 }
 
-static struct nfs4_stateid *find_stateid(stateid_t *t, int flags)
+static struct nfs4_stateid *find_stateid(stateid_t *t)
 {
        struct nfs4_stateid *s;
        unsigned int hashval;
 
        hashval = stateid_hashval(t->si_stateownerid, t->si_fileid);
-       list_for_each_entry(s, &stateid_hashtbl[hashval], st_hash) {
-               if (!same_stateid(&s->st_stateid, t))
-                       continue;
-               if (flags & LOCK_STATE && s->st_type != NFS4_LOCK_STID)
-                       return NULL;
-               if (flags & OPEN_STATE && s->st_type != NFS4_OPEN_STID)
-                       return NULL;
+       list_for_each_entry(s, &stateid_hashtbl[hashval], st_hash)
+               if (same_stateid(&s->st_stateid, t))
+                       return s;
+       return NULL;
+}
+
+static struct nfs4_stateid *find_stateid_by_type(stateid_t *t, int flags)
+{
+       struct nfs4_stateid *s;
+
+       s = find_stateid(t);
+       if (!s)
+               return NULL;
+       if (flags & LOCK_STATE && s->st_type == NFS4_LOCK_STID)
+               return s;
+       if (flags & OPEN_STATE && s->st_type == NFS4_OPEN_STID)
                return s;
-               }
        return NULL;
 }
 
@@ -3241,7 +3249,7 @@ __be32 nfs4_validate_stateid(stateid_t *stateid, bool has_session)
                goto out;
 
        status = nfserr_expired;
-       stp = find_stateid(stateid, 0);
+       stp = find_stateid(stateid);
        if (!stp)
                goto out;
        status = nfserr_bad_stateid;
@@ -3306,7 +3314,7 @@ nfs4_preprocess_stateid_op(struct nfsd4_compound_state *cstate,
                        BUG_ON(!*filpp);
                }
        } else { /* open or lock stateid */
-               stp = find_stateid(stateid, flags);
+               stp = find_stateid(stateid);
                if (!stp)
                        goto out;
                status = nfserr_bad_stateid;
@@ -3381,7 +3389,7 @@ nfsd4_free_stateid(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate,
                goto out;
        }
 
-       stp = find_stateid(stateid, 0);
+       stp = find_stateid(stateid);
        if (!stp) {
                ret = nfserr_bad_stateid;
                goto out;
@@ -3440,7 +3448,7 @@ nfs4_preprocess_seqid_op(struct nfsd4_compound_state *cstate, u32 seqid,
        * the confirmed flag is incorrecly set, or the generation 
        * number is incorrect.  
        */
-       *stpp = find_stateid(stateid, flags);
+       *stpp = find_stateid_by_type(stateid, flags);
        if (*stpp == NULL)
                return nfserr_expired;