From: Aaron Williams <Aaron.Williams@caviumnetworks.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] MIPS: Introduce --gc-sections for MIPS
Date: Sat, 16 Apr 2011 02:14:39 -0700 [thread overview]
Message-ID: <201104160214.39675.Aaron.Williams@caviumnetworks.com> (raw)
In-Reply-To: <4DA86961.9080608@pobox.com>
I just tried it for our Octeon boards and saw a noticeable reduction in size.
Our bootloader shrank from 1107840 to 941600 bytes. Size isn't too big of an
issue for us, though it's nice to see it shrink to under 1MB again.
-Aaron
On Friday, April 15, 2011 08:50:57 AM Shinya Kuribayashi wrote:
> On 04/16/2011 12:16 AM, daniel.schwierzeck at googlemail.com wrote:
> > All architectures but MIPS are using --gc-sections on final linking.
> > This patch introduces that feature for MIPS to reduce the memory and
> > flash footprint.
> >
> > Signed-off-by: Daniel Schwierzeck <daniel.schwierzeck@googlemail.com>
> > Cc: Shinya Kuribayashi <skuribay@pobox.com>
> > Cc: Wolfgang Denk <wd@denx.de>
> > Cc: Stefan Roese <sr@denx.de>
> > Cc: Thomas Lange <thomas@corelatus.se>
> > Cc: Vlad Lungu <vlad.lungu@windriver.com>
> > ---
> > Tested with ELDK-4.1 and self-built gcc-4.3.3/binutils-2.19.1 on my
> > boards. I could not test the other boards so I would appreciate it if
> > the maintainers in CC could do a quick test on their boards.
> >
> > arch/mips/config.mk | 2 ++
> > board/dbau1x00/u-boot.lds | 10 +++++-----
> > board/gth2/u-boot.lds | 10 +++++-----
> > board/incaip/u-boot.lds | 10 +++++-----
> > board/micronas/vct/u-boot.lds | 10 +++++-----
> > board/pb1x00/u-boot.lds | 10 +++++-----
> > board/qemu-mips/u-boot.lds | 10 +++++-----
> > board/tb0229/u-boot.lds | 10 +++++-----
> > examples/standalone/mips.lds | 10 +++++-----
> > 9 files changed, 42 insertions(+), 40 deletions(-)
>
> Basically, changes made look good. I'm going to apply after a couple
> of weeks (or a month) of interval, even without reports from the
> maintainers. As the patch has been proposed in the last minutes during
> the merge window, we'll make it in the v2011.06 release, thanks.
>
next prev parent reply other threads:[~2011-04-16 9:14 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-15 15:16 [U-Boot] [PATCH] MIPS: Introduce --gc-sections for MIPS daniel.schwierzeck at googlemail.com
2011-04-15 15:50 ` Shinya Kuribayashi
2011-04-16 9:14 ` Aaron Williams [this message]
2011-05-09 15:34 ` Shinya Kuribayashi
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=201104160214.39675.Aaron.Williams@caviumnetworks.com \
--to=aaron.williams@caviumnetworks.com \
--cc=u-boot@lists.denx.de \
/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