From: Joakim Tjernlund <Joakim.Tjernlund@transmode.se>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Fixup entries
Date: Wed, 9 Apr 2008 18:19:59 +0200 [thread overview]
Message-ID: <005a01c89a5d$8e9c67b0$abd53710$@Tjernlund@transmode.se> (raw)
In-Reply-To: <1207752413.6954.52.camel@localhost.localdomain>
> -----Original Message-----
> From: Kenneth Johansson [mailto:kenneth at southpole.se]
> Sent: den 9 april 2008 16:47
> To: joakim.tjernlund at transmode.se
> Cc: u-boot-users at lists.sourceforge.net
> Subject: Re: [U-Boot-Users] Fixup entries
>
>
> On Wed, 2008-04-09 at 13:23 +0200, Joakim Tjernlund wrote:
> > On Wed, 2008-04-09 at 11:32 +0200, Kenneth Johansson wrote:
> > > What is the fixup section used for ?
> > >
> > > of the 274 cards that build on my MAKEALL not one has any entries into
> > > this section.
> >
> > These are needed for relocation of function pointers(and possible some
> > think more). There was an attempt by Grant Likely to enable this
> > functionalty, but some toolchains had problems with so the function
> > was disabled. If you undo commit
> > 1c3dd43338a077165e7e0309cb3994e65d2bdbf8 you will enable it again for
> > powerpc.
> >
> > Jocke
>
> aaahhaa so that is how it's working.
>
> I remember having problem understanding this when I did the command
> section and ended up doing a manual relocation of the command table. But
> compiling with -mrelocatable and going over the .fixup section instead I
> can now remove that extra code and it works fine.
>
> It's not easy to understand the ABI.
>
> I removed the -fPIC and only used -mrelocatable and that works too.
>
> Wonder if there is something else in u-boot that use function pointers
> and work by accident due to the fact that the old address in flash is
> still valid.
Oh yes, there are a few such pointers. And the conversion to using fixups instead
is a bit lazy. Several manual relocations has only been nullified by setting
gd->reloc_off=0 instead of #ifdef:ing out the relevant code. Feel
free to send patches :)
Jocke
next prev parent reply other threads:[~2008-04-09 16:19 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-09 9:32 [U-Boot-Users] Fixup entries Kenneth Johansson
2008-04-09 11:23 ` Joakim Tjernlund
2008-04-09 14:46 ` Kenneth Johansson
2008-04-09 16:19 ` Joakim Tjernlund [this message]
2008-04-09 23:00 ` kenneth johansson
2008-04-09 23:03 ` Joakim Tjernlund
2008-04-09 23:52 ` kenneth johansson
2008-04-14 19:08 ` Kenneth Johansson
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='005a01c89a5d$8e9c67b0$abd53710$@Tjernlund@transmode.se' \
--to=joakim.tjernlund@transmode.se \
--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.