diff options
author | J. Bruce Fields <bfields@citi.umich.edu> | 2007-07-19 04:49:20 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-19 13:04:52 -0400 |
commit | c7d51402d2a64c5b96531f9900bb368020ebbbbb (patch) | |
tree | b870f7352ceb21799df933a31662b748edcad1e6 /fs/nfsd/vfs.c | |
parent | e22841c637dc8b308b40f59d64a5b6683d458ab7 (diff) |
knfsd: clean up EX_RDONLY
Share a little common code, reverse the arguments for consistency, drop the
unnecessary "inline", and lowercase the name.
Signed-off-by: "J. Bruce Fields" <bfields@citi.umich.edu>
Acked-by: Neil Brown <neilb@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/nfsd/vfs.c')
-rw-r--r-- | fs/nfsd/vfs.c | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c index f2684e57cf22..ee96a897a29e 100644 --- a/fs/nfsd/vfs.c +++ b/fs/nfsd/vfs.c | |||
@@ -1797,16 +1797,9 @@ nfsd_statfs(struct svc_rqst *rqstp, struct svc_fh *fhp, struct kstatfs *stat) | |||
1797 | return err; | 1797 | return err; |
1798 | } | 1798 | } |
1799 | 1799 | ||
1800 | static inline int EX_RDONLY(struct svc_export *exp, struct svc_rqst *rqstp) | 1800 | static int exp_rdonly(struct svc_rqst *rqstp, struct svc_export *exp) |
1801 | { | 1801 | { |
1802 | struct exp_flavor_info *f; | 1802 | return nfsexp_flags(rqstp, exp) & NFSEXP_READONLY; |
1803 | struct exp_flavor_info *end = exp->ex_flavors + exp->ex_nflavors; | ||
1804 | |||
1805 | for (f = exp->ex_flavors; f < end; f++) { | ||
1806 | if (f->pseudoflavor == rqstp->rq_flavor) | ||
1807 | return f->flags & NFSEXP_READONLY; | ||
1808 | } | ||
1809 | return exp->ex_flags & NFSEXP_READONLY; | ||
1810 | } | 1803 | } |
1811 | 1804 | ||
1812 | /* | 1805 | /* |
@@ -1845,7 +1838,7 @@ nfsd_permission(struct svc_rqst *rqstp, struct svc_export *exp, | |||
1845 | */ | 1838 | */ |
1846 | if (!(acc & MAY_LOCAL_ACCESS)) | 1839 | if (!(acc & MAY_LOCAL_ACCESS)) |
1847 | if (acc & (MAY_WRITE | MAY_SATTR | MAY_TRUNC)) { | 1840 | if (acc & (MAY_WRITE | MAY_SATTR | MAY_TRUNC)) { |
1848 | if (EX_RDONLY(exp, rqstp) || IS_RDONLY(inode)) | 1841 | if (exp_rdonly(rqstp, exp) || IS_RDONLY(inode)) |
1849 | return nfserr_rofs; | 1842 | return nfserr_rofs; |
1850 | if (/* (acc & MAY_WRITE) && */ IS_IMMUTABLE(inode)) | 1843 | if (/* (acc & MAY_WRITE) && */ IS_IMMUTABLE(inode)) |
1851 | return nfserr_perm; | 1844 | return nfserr_perm; |