Re: [RFC 03/11] slim down debugfs

From: Al Viro
Date: Sat Feb 23 2008 - 07:38:17 EST


On Tue, Feb 19, 2008 at 05:04:38AM +0100, Arnd Bergmann wrote:
> With most of debugfs now copied to generic code in libfs,
> we can remove the original copy and replace it with thin
> wrappers around libfs.
>
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> Index: linux-2.6/fs/Kconfig
> ===================================================================
> --- linux-2.6.orig/fs/Kconfig
> +++ linux-2.6/fs/Kconfig
> @@ -1001,6 +1001,14 @@ config CONFIGFS_FS
> Both sysfs and configfs can and should exist together on the
> same system. One is not a replacement for the other.
>
> +config LIBFS
> + tristate
> + default m
> + help
> + libfs is a helper library used by many of the simpler file
> + systems. Parts of libfs can be modular when all of its users
> + are modules as well, and the users should select this symbol.

NAK. For one thing, you need dependencies or selects and none of the
patches later in the series introduces them. For another, neither
dependencies nor selects work well if you have non-modular users of
that sucker. Seriously, try to add those and do allmodconfig. Then
look what a mess it produces.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/