diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2009-08-28 12:29:03 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-09-08 11:09:04 -0700 |
| commit | 18f4c644773bc8de1fd9c5182b30c231aafb94ef (patch) | |
| tree | b423bdb22304c6af3ad0f26b873e6081fafc8f01 /fs/jffs2/acl.c | |
| parent | 1d5ccd1c422d7d292a9e45248aa36771900c6331 (diff) | |
| download | op-kernel-dev-18f4c644773bc8de1fd9c5182b30c231aafb94ef.zip op-kernel-dev-18f4c644773bc8de1fd9c5182b30c231aafb94ef.tar.gz | |
jffs2/jfs/xfs: switch over to 'check_acl' rather than 'permission()'
This avoids an indirect call in the VFS for each path component lookup.
Well, at least as long as you own the directory in question, and the ACL
check is unnecessary.
Reviewed-by: James Morris <jmorris@namei.org>
Acked-by: Serge Hallyn <serue@us.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/jffs2/acl.c')
| -rw-r--r-- | fs/jffs2/acl.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/jffs2/acl.c b/fs/jffs2/acl.c index 8fcb62392..7edb62e 100644 --- a/fs/jffs2/acl.c +++ b/fs/jffs2/acl.c @@ -258,7 +258,7 @@ static int jffs2_set_acl(struct inode *inode, int type, struct posix_acl *acl) return rc; } -static int jffs2_check_acl(struct inode *inode, int mask) +int jffs2_check_acl(struct inode *inode, int mask) { struct posix_acl *acl; int rc; @@ -274,11 +274,6 @@ static int jffs2_check_acl(struct inode *inode, int mask) return -EAGAIN; } -int jffs2_permission(struct inode *inode, int mask) -{ - return generic_permission(inode, mask, jffs2_check_acl); -} - int jffs2_init_acl_pre(struct inode *dir_i, struct inode *inode, int *i_mode) { struct posix_acl *acl, *clone; |
