Re: Quota patches

Nathan Scott
2002-05-22T23:30:42Z

hi there,

On Thu, May 23, 2002 at 04:46:28AM +0900, OGAWA Hirofumi wrote:
> What do you think of the following patches for kernel without
> quota support? We already have weak symbol for sys_quotactl().
>
> --- linux-bk/fs/Makefile Wed May 22 01:17:48 2002
> +++ linux-2.5.17/fs/Makefile Thu May 23 03:23:30 2002
> @@ -15,7 +15,7 @@
> namei.o fcntl.o ioctl.o readdir.o select.o fifo.o locks.o \
> dcache.o inode.o attr.o bad_inode.o file.o iobuf.o dnotify.o \
> filesystems.o namespace.o seq_file.o xattr.o libfs.o \
> - fs-writeback.o quota.o
> + fs-writeback.o
>
> ifneq ($(CONFIG_NFSD),n)
> ifneq ($(CONFIG_NFSD),)
> @@ -81,7 +81,7 @@
>
> obj-$(CONFIG_BINFMT_ELF) += binfmt_elf.o
>
> -obj-$(CONFIG_QUOTA) += dquot.o
> +obj-$(CONFIG_QUOTA) += dquot.o quota.o
> obj-$(CONFIG_QFMT_V1) += quota_v1.o
> obj-$(CONFIG_QFMT_V2) += quota_v2.o
>

On Wed, May 22, 2002 at 09:57:00PM +0200, Jan Kara wrote:
> I know this but this file is also needed by XFS people so it isn't
> so easy. But as XFS stuff is not currently in kernel the change is

A change to the patch like this would make this work for us too:

replacing:
-obj-$(CONFIG_QUOTA) += dquot.o
+obj-$(CONFIG_QUOTA) += dquot.o quota.o

with just:
+obj-$(CONFIG_QUOTACTL) += quota.o

Needs to be fleshed out a little bit, of course - both CONFIG_QUOTA
and CONFIG_XFS_QUOTA (from the XFS patches) would need fs/Config.in
changes to add the new $CONFIG_QUOTACTL dependency, but that should
just about do the trick I think.

cheers.

--
Nathan