On Tue, Oct 12, 2010 at 12:47:38PM +0530, Aneesh Kumar K. V wrote: > On Mon, 11 Oct 2010 20:24:09 -0400, "J. Bruce Fields" <bfields@xxxxxxxxxxxxxx> wrote: > > The one thing I remember not liking before was a flag that told the user > > whether a given ACL was originally mapped from POSIX or not. Is that > > still there? > > We still have that. But we can resolve that once we decide on how to > migrate an existing file system containing posix acl to richacl. Most of > those patches will need to be updated based on the feedback from > different local file system maintainers. That is why those patches are > pushed towards the end and is part of last set. > > What we need in the first step is to get VFS changes reviewed.Once we > agree on the VFS changes done, then we can start looking at the changes > upto NFS richacl nfs support. When get that merged then we can start > having discussion on how local file system maintainers want to migrate > the existing file system with posixacl to richacl. OK. So, personally: I'm resigned to the idea that we want support for this ACL model. The vfs changes look OK to me (and wouldn't be changed by any comments I'd have on the more richacl-specific patches to follow). So that's an ACK from me on the first set of these patches, assuming it's OK with people to merge these things one step at a time. --b. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html