On Tue, Jun 21, 2011 at 12:24 PM, Andreas Dilger [off-list ref] wrote:

On 2011-06-21, at 12:45 PM, Eric Sandeen wrote:

quoted

I was looking at

62d2b5f2dcd3707b070efb16bbfdf6947c38c194 ext4: Always journal quota file modifications

and was thinking - if we always journal, why do we still have all the mount options etc for journaled quota?

I guess without -o usrjquota=foo we don't journal right from mount time, and there are a few other paths that get skipped, but those seem like details...

It seems that ideally - and perhaps this is hindsight - ext4 should have defaulted to journaled, QFMT_VFS_V1 quotas only... is there any reason not to?

This would be the ideal situation for Lustre as well.  The remaining issue,
AFAIK, is that e2fsck needs to do quota accounting internally so that there
is never a need to run a quotacheck on the filesystem.

Since e2fsck is already traversing all of the inodes, this is no extra effort
to keep a handful of bytes (inode count, blocks count) for each user.  I
believe there was someone at Google working on this?  Haven't seen any output
so far, however.

I have the changes for fixing quota usage during e2fsck with support
for QFMT_VFS_V1 format. Currently, I am using tune2fs to create the
quota inodes and set the superblock fields. On the next mount of the
filesystem, the kernel should automatically enable quotas for the
inodes that are specified in superblock.
I will send the patches out for review soon (hopefully this week).

quoted

I'm still not very well versed in vfs quota, but does it seem possible to
move in this direction?  It seems odd that we should still be supporting
unjournaled, vfsold-format quotas in ext4 these days.  What's the point?

I totally agree.

quoted

If it seems possible to move in this direction within the exiting quota
framework I'd be interested in trying to get it done.

Thanks,
-Eric
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Cheers, Andreas





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

Thanks,
--
Aditya
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html