diff options
author | J. Bruce Fields <bfields@redhat.com> | 2011-10-10 17:44:19 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2011-10-11 12:15:15 -0400 |
commit | b6d2f1ca3c1162f51098969e9c52fd099720416a (patch) | |
tree | bf8fa7abd3b601902ac034248a07b274b33e45fd /fs/nfsd/nfs4state.c | |
parent | a084daf512bb66fa3c8e21c7027daea521179cd0 (diff) |
nfsd4: more robust ignoring of WANT bits in OPEN
Mask out the WANT bits right at the start instead of on each use.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Diffstat (limited to 'fs/nfsd/nfs4state.c')
-rw-r--r-- | fs/nfsd/nfs4state.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index 5f35f35a2da0..2042805da960 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c | |||
@@ -2345,8 +2345,7 @@ static inline __be32 init_open_stateid(struct nfs4_ol_stateid *stp, struct nfs4_ | |||
2345 | stp->st_file = fp; | 2345 | stp->st_file = fp; |
2346 | stp->st_access_bmap = 0; | 2346 | stp->st_access_bmap = 0; |
2347 | stp->st_deny_bmap = 0; | 2347 | stp->st_deny_bmap = 0; |
2348 | __set_bit(open->op_share_access & ~NFS4_SHARE_WANT_MASK, | 2348 | __set_bit(open->op_share_access, &stp->st_access_bmap); |
2349 | &stp->st_access_bmap); | ||
2350 | __set_bit(open->op_share_deny, &stp->st_deny_bmap); | 2349 | __set_bit(open->op_share_deny, &stp->st_deny_bmap); |
2351 | stp->st_openstp = NULL; | 2350 | stp->st_openstp = NULL; |
2352 | return nfs_ok; | 2351 | return nfs_ok; |
@@ -2690,7 +2689,7 @@ nfsd4_truncate(struct svc_rqst *rqstp, struct svc_fh *fh, | |||
2690 | static __be32 | 2689 | static __be32 |
2691 | nfs4_upgrade_open(struct svc_rqst *rqstp, struct nfs4_file *fp, struct svc_fh *cur_fh, struct nfs4_ol_stateid *stp, struct nfsd4_open *open) | 2690 | nfs4_upgrade_open(struct svc_rqst *rqstp, struct nfs4_file *fp, struct svc_fh *cur_fh, struct nfs4_ol_stateid *stp, struct nfsd4_open *open) |
2692 | { | 2691 | { |
2693 | u32 op_share_access = open->op_share_access & ~NFS4_SHARE_WANT_MASK; | 2692 | u32 op_share_access = open->op_share_access; |
2694 | bool new_access; | 2693 | bool new_access; |
2695 | __be32 status; | 2694 | __be32 status; |
2696 | 2695 | ||