[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200802241159.41895.arnd@arndb.de>
Date: Sun, 24 Feb 2008 11:59:41 +0100
From: Arnd Bergmann <arnd@...db.de>
To: Al Viro <viro@...iv.linux.org.uk>
Cc: linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org,
Christoph Hellwig <hch@....de>, Greg KH <gregkh@...e.de>,
David Howells <dhowells@...hat.com>
Subject: Re: [RFC 03/11] slim down debugfs
On Saturday 23 February 2008, Al Viro wrote:
> > +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.
Sorry about that hunk, I didn't mean to keep that in there. In
my testing, I used the newly introduced parts as a loadable
module, and at one point the intention was to keep the existing
libfs functions as builtin and add a new Kconfig dependency in every
user of the new stuff whenever a file system gets converted.
I figured out later that hardly anyone could even try to build
without them as soon as all the trival file systems are moved
over, but forgot to make it always builtin.
Arnd <><
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists