summaryrefslogtreecommitdiff
path: root/fs/nfsd
diff options
context:
space:
mode:
authorJ. Bruce Fields <bfields@redhat.com>2011-09-06 16:48:57 -0400
committerJ. Bruce Fields <bfields@redhat.com>2011-09-13 18:27:31 -0400
commit4d71ab8751c1d749f9fdf84ec094989adf579493 (patch)
tree82e3d8ece2e19e5b817e682e2959770158aed099 /fs/nfsd
parent4581d1409977c5fe686a4ed487cdce2e50031826 (diff)
nfsd4: split up find_stateid
Minor cleanup. Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd')
-rw-r--r--fs/nfsd/nfs4state.c34
1 files changed, 21 insertions, 13 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index f0eccc236a0d..aa088bc3b169 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -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;