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/jfs/acl.c | |
| parent | 1d5ccd1c422d7d292a9e45248aa36771900c6331 (diff) | |
| download | kernel_samsung_smdk4412-18f4c644773bc8de1fd9c5182b30c231aafb94ef.zip kernel_samsung_smdk4412-18f4c644773bc8de1fd9c5182b30c231aafb94ef.tar.gz kernel_samsung_smdk4412-18f4c644773bc8de1fd9c5182b30c231aafb94ef.tar.bz2 | |
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/jfs/acl.c')
| -rw-r--r-- | fs/jfs/acl.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/jfs/acl.c b/fs/jfs/acl.c index a29c7c3..d66477c 100644 --- a/fs/jfs/acl.c +++ b/fs/jfs/acl.c @@ -114,7 +114,7 @@ out: return rc; } -static int jfs_check_acl(struct inode *inode, int mask) +int jfs_check_acl(struct inode *inode, int mask) { struct posix_acl *acl = jfs_get_acl(inode, ACL_TYPE_ACCESS); @@ -129,11 +129,6 @@ static int jfs_check_acl(struct inode *inode, int mask) return -EAGAIN; } -int jfs_permission(struct inode *inode, int mask) -{ - return generic_permission(inode, mask, jfs_check_acl); -} - int jfs_init_acl(tid_t tid, struct inode *inode, struct inode *dir) { struct posix_acl *acl = NULL; |