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

From: Christoph Hellwig (
Date: Wed Oct 16 2002 - 08:11:04 EST

On Tue, Oct 15, 2002 at 06:20:55PM -0400, 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
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

This archive was generated by hypermail 2b29 : Wed Oct 23 2002 - 22:00:28 EST