From: Shinya Kuribayashi <skuribay@ruby.dti.ne.jp>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH 0/5][MIPS] Fix several problems on relocation
Date: Sun, 21 Oct 2007 00:39:09 +0900 [thread overview]
Message-ID: <471A211D.907@ruby.dti.ne.jp> (raw)
I've pushed the following patchset on top of the upstream:
The following changes since commit 5441f61a3d8b7034f19fc1361183e936198e6dbb:
Detlev Zundel (1):
Fix two typos.
are available in the git repository at:
git://www.denx.de/git/u-boot-mips.git testing
Shinya Kuribayashi (5):
[MIPS] u-boot.lds: Remove duplicated .sdata section
[MIPS] u-boot.lds: Fix __got_start and __got_end
[MIPS] Fix $gp usage
[MIPS] u-boot.lds: Define _gp in a standard manner
[MIPS] Add PIC-related switches to PLATFORM_{CPP,LD}FLAGS and cleanup
board/dbau1x00/u-boot.lds | 14 +++++++-------
board/gth2/u-boot.lds | 14 +++++++-------
board/incaip/u-boot.lds | 14 +++++++-------
board/pb1x00/u-boot.lds | 14 +++++++-------
board/purple/u-boot.lds | 14 +++++++-------
board/tb0229/u-boot.lds | 14 +++++++-------
cpu/mips/config.mk | 4 +---
cpu/mips/start.S | 19 +++++++++++++------
examples/mips.lds | 14 +++++++-------
mips_config.mk | 25 +++++++++++++++++++++++++
10 files changed, 88 insertions(+), 58 deletions(-)
These patches mainly try to fix an outstanding mips bug:
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/25533
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/22160
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/19550
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/18887
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/14935
As already discussed, the cause of this bug is a misalignment between
__got_start and _GLOBAL_OFFSET_TABLE_. The __got_start misalignment
makes num_got_entries larger than needed. As a result, we update GOT
entries too many and corrupt top of the u_boot_cmd data. This will be
fixed by __got_{start,end} cleanups in linker scripts.
In addition to fixing above, I've takan $gp usage fix suggested by Vlad,
and cleaned up linker scripts, etc. Please review and give it a try.
thanks,
Shinya
next reply other threads:[~2007-10-20 15:39 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-10-20 15:39 Shinya Kuribayashi [this message]
2007-10-20 15:40 ` [U-Boot-Users] [PATCH 1/5][MIPS] u-boot.lds: Remove duplicated .sdata section Shinya Kuribayashi
2007-10-20 15:42 ` [U-Boot-Users] [PATCH 2/5][MIPS] u-boot.lds: Fix __got_start and __got_end Shinya Kuribayashi
2007-10-20 15:44 ` [U-Boot-Users] [PATCH 3/5][MIPS] Fix $gp usage Shinya Kuribayashi
2007-10-20 15:45 ` [U-Boot-Users] [PATCH 4/5][MIPS] u-boot.lds: Define _gp in a standard manner Shinya Kuribayashi
2007-10-20 15:47 ` [U-Boot-Users] [MIPS] Add PIC-related switches to PLATFORM_{CPP, LD}FLAGS and cleanup Shinya Kuribayashi
2007-10-20 18:05 ` Wolfgang Denk
2007-10-21 12:12 ` 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=471A211D.907@ruby.dti.ne.jp \
--to=skuribay@ruby.dti.ne.jp \
--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 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.