All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Christoph Hellwig <hch@infradead.org>
Cc: Sam Ravnborg <sam@ravnborg.org>,
	Alexey Dobriyan <adobriyan@sw.ru>,
	linux-kernel@vger.kernel.org, devel@openvz.org
Subject: Re: [PATCH] Move kasprintf.o to obj-y
Date: Mon, 24 Sep 2007 01:21:17 -0700	[thread overview]
Message-ID: <20070924012117.7827f3c7.akpm@linux-foundation.org> (raw)
In-Reply-To: <20070924080948.GA8525@infradead.org>

On Mon, 24 Sep 2007 09:09:48 +0100 Christoph Hellwig <hch@infradead.org> 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.

yup, it duplicates what the config system does and is a common source of
problems.

  reply	other threads:[~2007-09-24  8:22 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 [this message]
2007-09-24  8:29       ` Sam Ravnborg

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=20070924012117.7827f3c7.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=adobriyan@sw.ru \
    --cc=devel@openvz.org \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sam@ravnborg.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.