diff options
| author | Jeff Layton <jlayton@poochiereds.net> | 2016-05-05 12:53:47 +0200 |
|---|---|---|
| committer | J. Bruce Fields <bfields@redhat.com> | 2016-05-13 21:34:47 +0200 |
| commit | 14b7f4a1edf15808cf52dc38277ffe1d84854839 (patch) | |
| tree | b4fcfa64ff59ee0ed5f6ef01d7b828de534e3af9 /fs/nfsd/nfs4state.c | |
| parent | Remove unnecessary allocation (diff) | |
| download | linux-14b7f4a1edf15808cf52dc38277ffe1d84854839.tar.xz linux-14b7f4a1edf15808cf52dc38277ffe1d84854839.zip | |
nfsd: handle seqid wraparound in nfsd4_preprocess_layout_stateid
Move the existing static function to an inline helper, and call it.
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jeff Layton <jeff.layton@primarydata.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfs4state.c')
| -rw-r--r-- | fs/nfsd/nfs4state.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index 0462eeddfff9..f5f82e145018 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c @@ -4651,12 +4651,6 @@ grace_disallows_io(struct net *net, struct inode *inode) return opens_in_grace(net) && mandatory_lock(inode); } -/* Returns true iff a is later than b: */ -static bool stateid_generation_after(stateid_t *a, stateid_t *b) -{ - return (s32)(a->si_generation - b->si_generation) > 0; -} - static __be32 check_stateid_generation(stateid_t *in, stateid_t *ref, bool has_session) { /* @@ -4670,7 +4664,7 @@ static __be32 check_stateid_generation(stateid_t *in, stateid_t *ref, bool has_s return nfs_ok; /* If the client sends us a stateid from the future, it's buggy: */ - if (stateid_generation_after(in, ref)) + if (nfsd4_stateid_generation_after(in, ref)) return nfserr_bad_stateid; /* * However, we could see a stateid from the past, even from a |