Re: [PATCH 1/5] Add POSIX Access Control Lists to ext2/3

[Christoph Hellwig]

On Tue, Oct 15, 2002 at 06:20:55PM -0400, [e-mail] wrote:
> 
> The following five patches add Posix ACL support to the 2.5 kernel.  The
> first three patches are generic, and will be useful to other filesystems
> who also have ACL support on deck (in particular, the JFS and XFS
> folks).  The last two patches add ACL support to the ext2 and ext3
> filesystems.  These patches are safe, in that the code paths remain
> largely untouched if they the compile-time option is not enabled, and
> even if ACL support is compiled in, the filesystem must be explicitly
> mounted a mount-option to enable ACL support.
> 
> These pataches require that the extended attribute patches be applied
> first.

Linux, please domn't apply these ACL patches until the inode method mess is
sorted out.

Ted, please either go _always_ through the {get,set}_posix_acl methods
or never.  Currently XFS doesn't know and doesn't want to know
about the so called "egenric ACL representation" used by ext2/ext3.  With
theses methods we'd have to add it to XFS which is fine for me as long as
it the representation generally used for working with ACLs.  That would mean
we'd have to add new syscall or at least VFS-level hooks to the xattr code.

Or just declare the xattr representation the default one, remove
the get_posix_acl/set_posix_acl code from this code and fix up the nfsd
patch (that doesn't seem to be submitted yet) up to us the xattr
representation.