Re: Quota patches

Christoph Hellwig (hch@infradead.org)
Mon, 20 May 2002 15:07:57 +0100


On Mon, May 20, 2002 at 03:55:31PM +0200, Jan Kara wrote:
> quota-2.5.15-3-register - this patch implements registering/unregistering of quota
> formats

Please don't use the big kernel lock for a newly added list.
Also using <linux/lists.h> would clean up the list handling.

> quota-2.5.15-4-getstats - this patch removes Q_GETSTATS call and creates /proc/fs/quota
> entry instead

Yuck, even more /proc abuse. Please convert it to the seq_file interface
at least. Using individual sysctls per value would be much better.

> quota-2.5.15-7-quotactl - implementation of generic quotactl interface (probably the
> biggest patch). Interface is moved from dquot.c to quota.c file. Pointers
> to quota operations in superblock are now not filled on quota_on() but
> on mount so filesystem can override them (for example ext3 would like to
> check on quota_on() that quotafile lies on proper device and turn on
> data-journaling on it - at least when we'll have journaled quota :)).

The vfs_get*/vfs_set* names sound too generic, could you please rename them
to vfs_get_quota*/vfs_set_quota*?

Also I think any quota supporting filesystem should set the quota operations
explicitly to make the intention clearer.

> quota-2.5.15-12-compat - implements backward compatible quotactl() interface. It's
> configurable whether it should be used at all and whether is should behave
> as interface in Linus's (the oldest interface) or Alan's (old interface for
> new quota format) kernel.

I don't think we want to keep old userspace interface in 2.5, it just
bloats the kernel and requiring quota tools for a development kernel that
are already required by all vendor kernels sounds sane to me.

Else your patches look very good to me, I look forward to finally see
properly working quota support in a mainline kernel.

Christoph

-
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/