Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] linux: strip installed modules
Date: Thu, 18 Nov 2010 08:37:24 +0100	[thread overview]
Message-ID: <20101118083724.50b78d0e@surf> (raw)
In-Reply-To: <87oc9nad27.fsf@macbook.be.48ers.dk>

On Wed, 17 Nov 2010 21:51:44 +0100
Peter Korsgaard <jacmet@uclibc.org> wrote:

>  Mike> The installed kernel modules should have useless build cruft stripped out
>  Mike> of them.  On my system, a default build went from a very unreasonable 30MB
>  Mike> to a normal 3MB (on disk) and from 14MB to 3MB when compressed due to the
>  Mike> info stored in the kernel modules alone.
> 
> Thanks. We're moving away from per-package stripping though, so I would
> prefer to add it to the toplevel target-finalize step instead though -

Actually, yes and now. The stripping will go back into per-package, but
inside the package infrastructure instead. This is because we want to
do all the cleanup (documentation, headers, useless .a and .la files)
right after the package installation in order to make the "package
management" stuff work (this is something that Lionel already did in
his experiments of package management). So it will not be inside each
package .mk file as it used to be, but it's still going to be done on a
per-package basis.

> find $(TARGET_DIR)/lib/modules -type f -name '*.ko' | xargs $(STRIPCMD) ..

In this particular case, the stripping is very much kernel-specific, so
I don't see a strong reason of adding this to linux/linux.mk. Except
maybe packages that do compile additional kernel modules, such as
linux-fusion (and more could be added).

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

      parent reply	other threads:[~2010-11-18  7:37 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-16 14:59 [Buildroot] [PATCH] linux: strip installed modules Mike Frysinger
2010-11-17 20:51 ` Peter Korsgaard
2010-11-17 23:55   ` [Buildroot] [PATCH v2] " Mike Frysinger
2010-11-24 21:04     ` Peter Korsgaard
2010-11-24 21:20       ` Mike Frysinger
2010-11-24 21:36         ` Peter Korsgaard
2010-11-24 21:47           ` Mike Frysinger
2010-11-18  7:37   ` Thomas Petazzoni [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=20101118083724.50b78d0e@surf \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox