[Ocfs2-devel] Tiny patch for cached ACLs support
Tristan Ye
tristan.ye at oracle.com
Thu Mar 31 01:40:40 PDT 2011
jeff.liu wrote:
> Hello,
>
> Below is a tiny patch for adding cached ACLs support to OCFS2, I have run a few simple tests like
> getacl/setacl/chacl on my single node OCFS2, it works for me.
Hi Jeff,
Take a look at http://oss.oracle.com/git/ocfs2-test.git/programs/acl_tests,
there is a somewhat comprehensive testing script in existence for posix_acl,
maybe it can be a bit overkilling for your patch, but please just take it as a
double-checker if you wish;-)
>
> Any comments and testing are appreciated.
>
> Thanks,
> -Jeff
>
>
> Signed-off-by: Jeff Liu <jeff.liu at oracle.com>
> ---
> fs/ocfs2/acl.c | 12 +++++++++++-
> fs/ocfs2/inode.c | 4 ++++
> 2 files changed, 15 insertions(+), 1 deletions(-)
>
> diff --git a/fs/ocfs2/acl.c b/fs/ocfs2/acl.c
> index 704f6b1..13c58b0 100644
> --- a/fs/ocfs2/acl.c
> +++ b/fs/ocfs2/acl.c
> @@ -152,6 +152,10 @@ static struct posix_acl *ocfs2_get_acl(struct inode *inode, int type)
> if (!(osb->s_mount_opt & OCFS2_MOUNT_POSIX_ACL))
> return NULL;
>
> + acl = get_cached_acl(inode, type);
> + if (acl != ACL_NOT_CACHED)
> + return acl;
> +
> ret = ocfs2_inode_lock(inode, &di_bh, 0);
> if (ret < 0) {
> mlog_errno(ret);
> @@ -286,6 +290,9 @@ static int ocfs2_set_acl(handle_t *handle,
> else
> ret = ocfs2_xattr_set(inode, name_index, "", value, size, 0);
>
> + if (!ret)
> + set_cached_acl(inode, type, acl);
> +
> kfree(value);
>
> return ret;
> @@ -348,9 +355,12 @@ int ocfs2_acl_chmod(struct inode *inode)
> if (!clone)
> return -ENOMEM;
> ret = posix_acl_chmod_masq(clone, inode->i_mode);
> - if (!ret)
> + if (!ret) {
> ret = ocfs2_set_acl(NULL, inode, NULL, ACL_TYPE_ACCESS,
> clone, NULL, NULL);
> + set_cached_acl(inode, ACL_TYPE_ACCESS, clone);
> + }
> +
> posix_acl_release(clone);
> return ret;
> }
> diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c
> index 4068c6c..80deac8 100644
> --- a/fs/ocfs2/inode.c
> +++ b/fs/ocfs2/inode.c
> @@ -28,6 +28,7 @@
> #include <linux/highmem.h>
> #include <linux/pagemap.h>
> #include <linux/quotaops.h>
> +#include <linux/posix_acl.h>
>
> #include <asm/byteorder.h>
>
> @@ -783,6 +784,9 @@ static int ocfs2_wipe_inode(struct inode *inode,
> goto bail_unlock_dir;
> }
>
> + if (osb->s_mount_opt & OCFS2_MOUNT_POSIX_ACL)
> + forget_all_cached_acls(inode);
> +
> status = ocfs2_remove_refcount_tree(inode, di_bh);
> if (status < 0) {
> mlog_errno(status);
More information about the Ocfs2-devel
mailing list