Re: [Ext2-devel] [RFC] [PATCH 3/4] Add extended attributes to ext2/3

From: Theodore Ts'o (tytso@mit.edu)
Date: Tue Oct 08 2002 - 17:17:10 EST


On Tue, Oct 08, 2002 at 09:41:43PM +0100, Stephen C. Tweedie wrote:
>
> On Tue, Oct 08, 2002 at 08:20:38PM +0100, Christoph Hellwig wrote:
> > On Tue, Oct 08, 2002 at 09:14:00PM +0200, Andreas Gruenbacher wrote:
> > > Users might just fill up all xattr space leaving no space for ACLs (or
> > > similar). If user xattrs are disabled this can no longer occur, so some
> > > administrators might be happy to have a choice.
> >
> > Umm, that's why we have quota..
>
> It's the per-inode extended attribute space that's at risk here,
> quotas don't help.

Well, how about this as a compromise? We define a new superblock
field which reserves a certain amount of space in the EA block for
"system" attributes. The default will be zero, but if we want to
reserve space for the ACL, we can do that by adjusting the superblock
field. (I'll add support into tune2fs for that purpose.)

I'll then remove the CONFIG #ifdef's and the user_xattr mount options.
(I hate having too many mount options, and this sort of thing should
be a per-filesystem run-time decision, not a compile-time option.)

Andreas, does that sound good to you?

                                                - Ted

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



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