From: Sam Ravnborg <sam@ravnborg.org>
To: Christoph Hellwig <hch@infradead.org>,
Alexey Dobriyan <adobriyan@sw.ru>,
akpm@osdl.org, linux-kernel@vger.kernel.org, devel@openvz.org
Subject: Re: [PATCH] Move kasprintf.o to obj-y
Date: Mon, 24 Sep 2007 10:29:40 +0200 [thread overview]
Message-ID: <20070924082940.GA1981@uranus.ravnborg.org> (raw)
In-Reply-To: <20070924080948.GA8525@infradead.org>
On Mon, Sep 24, 2007 at 09:09:48AM +0100, Christoph Hellwig wrote:
> On Mon, Sep 24, 2007 at 09:59:49AM +0200, Sam Ravnborg wrote:
> > > Please kill lib-y while you're at it. It's useless and a constant
> > > source of pain like this.
> > Kernel-bloat is another "constant source of pain".
> > But the troubles are that increased blot does not result in compiler erros.
> >
> > And your proposal to kil lib-y is a counter-act here.
>
> Killing lib-y doesn't create kernel bloat if done right. Just introduce
> proper Kconfig dependencies for the truely optional parts. But if you
> look at the list of objects in lib-y you'll see that they fall in basically
> three categories:
>
> (1) always used by core code - should be obj-y
> (2) generic implementation for arch-specific functionality, should be
> guarded by Kconfig for clarity reasons anyway
> (3) library code often but not always used by code that's not always
> built in. Here we run into the move to obj-y to avoid compiler
> warning when used as module issue all the time. Adding a CONFIG
> for the bigger onces actually avoids bloat over obj-y here, and
> when it's small enough obj-y is a lot safer in the presence of
> modular users
>
> Really, trying to use the old static library use on demand concept simply
> doesn't fit the way the kernel is built with it's modules and dependencies.
I objected to just killing lib-y (the way I read your original proposal).
If we introduce proper kconfig dependencies as in (3) then I am all for it.
Sam
prev parent reply other threads:[~2007-09-24 8:28 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-24 7:18 [PATCH] Move kasprintf.o to obj-y Alexey Dobriyan
2007-09-24 7:41 ` Christoph Hellwig
2007-09-24 7:59 ` Sam Ravnborg
2007-09-24 8:09 ` Christoph Hellwig
2007-09-24 8:21 ` Andrew Morton
2007-09-24 8:29 ` Sam Ravnborg [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070924082940.GA1981@uranus.ravnborg.org \
--to=sam@ravnborg.org \
--cc=adobriyan@sw.ru \
--cc=akpm@osdl.org \
--cc=devel@openvz.org \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.