diff options
| author |   Christoph Hellwig <hch@lst.de> | 2011-07-23 17:36:50 +0200 |
|---|---|---|
| committer |   Al Viro <viro@zeniv.linux.org.uk> | 2011-07-25 14:25:38 -0400 |
| commit | 6311b10800a4bdc3f2c85d01fb113cf49bb83770 (patch) | |
| tree | 4798068e025fa64c9c8656e2a28427f120d447a0 /fs/xfs/linux-2.6/xfs_acl.c | |
| parent | 9p: do no return 0 from ->check_acl without actually checking (diff) | |
| download | linux-dev-6311b10800a4bdc3f2c85d01fb113cf49bb83770.tar.xz linux-dev-6311b10800a4bdc3f2c85d01fb113cf49bb83770.zip | |
xfs: cache negative ACLs if there is no attribute fork
Always set up a negative ACL cache entry if the inode doesn't have an
attribute fork. That behaves much better than doing this check inside
->check_acl.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to '')
| -rw-r--r-- | fs/xfs/linux-2.6/xfs_acl.c | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/fs/xfs/linux-2.6/xfs_acl.c b/fs/xfs/linux-2.6/xfs_acl.c index f6d065ac56b5..13c4e4fd5c6e 100644 --- a/fs/xfs/linux-2.6/xfs_acl.c +++ b/fs/xfs/linux-2.6/xfs_acl.c @@ -221,21 +221,10 @@ xfs_set_acl(struct inode *inode, int type, struct posix_acl *acl) int xfs_check_acl(struct inode *inode, int mask) { - struct xfs_inode *ip; struct posix_acl *acl; int error = -EAGAIN; - ip = XFS_I(inode); - trace_xfs_check_acl(ip); - - /* - * If there is no attribute fork no ACL exists on this inode and - * we can skip the whole exercise. - * - * FIXME! Fill the cache! Locking? - */ - if (!XFS_IFORK_Q(ip)) - return -EAGAIN; + trace_xfs_check_acl(XFS_I(inode)); acl = xfs_get_acl(inode, ACL_TYPE_ACCESS); if (IS_ERR(acl)) |