Re: [f2fs-dev] [PATCH 04/18] fs: add generic xattr_acl handlers

2013-12-17 Thread Andreas Gruenbacher
Christoph, > +static int > +posix_acl_xattr_set(struct dentry *dentry, const char *name, > + const void *value, size_t size, int flags, int type) > +{ > + struct inode *inode = dentry->d_inode; > + struct posix_acl *acl = NULL; > + int ret; > + > + if (!IS_POSIXACL(inod

[f2fs-dev] [PATCH 04/18] fs: add generic xattr_acl handlers

2013-12-11 Thread Christoph Hellwig
With the ->set_acl inode operation we can implement the Posix ACL xattr handlers in generic code instead of duplicating them all over the tree. Signed-off-by: Christoph Hellwig Reviewed-by: Jan Kara --- fs/xattr_acl.c | 102 +++ include/linu

[f2fs-dev] [PATCH 04/18] fs: add generic xattr_acl handlers

2013-12-02 Thread Christoph Hellwig
With the ->set_acl inode operation we can implement the Posix ACL xattr handlers in generic code instead of duplicating them all over the tree. Signed-off-by: Christoph Hellwig --- fs/xattr_acl.c | 95 +++ include/linux/posix_acl_xattr.h |

Re: [f2fs-dev] [PATCH 04/18] fs: add generic xattr_acl handlers

2013-12-02 Thread Jan Kara
On Sun 01-12-13 03:59:07, Christoph Hellwig wrote: > With the ->set_acl inode operation we can implement the Posix ACL > xattr handlers in generic code instead of duplicating them all > over the tree. Looks good. You can add: Reviewed-by: Jan Kara