Messages in this thread | | | Date | Tue, 8 Oct 2002 18:17:10 -0400 | From | "Theodore Ts'o" <> | Subject | Re: [Ext2-devel] [RFC] [PATCH 3/4] Add extended attributes to ext2/3 |
| |
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/
| |