public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v1] arm: remove unneeded symbol offsets and _TEXT_BASE
Date: Tue, 15 Oct 2013 14:04:55 +0200	[thread overview]
Message-ID: <20131015140455.397e94c5@lilith> (raw)
In-Reply-To: <20131013191633.3171a883@lilith>

On Sun, 13 Oct 2013 19:16:33 +0200, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:

> I don't think so. The switch from _end to __rel_dyn_end is done
> precisely to avoid using a symbol for which an offset is needed (_end,
> because linker-defined, can only have absolute relocations) and use
> instead a compiler-defined, relocatable, symbol (__rel_dyn_end, compiled
> from sections.c, always takes relative relocation) which does not need
> an offset. This switch thus belongs in this patch.

Actually, _end could be made a compiler-generated symbol too. This
would be more consistent with what is done with the BSS and image start
and end symbols, and would allow keeping _end here. Will revert change
here and send v2 as a two-patch series as you suggest.

> Amicalement,

Amicalement,
-- 
Albert.

  parent reply	other threads:[~2013-10-15 12:04 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-13  7:10 [U-Boot] [PATCH v1] arm: remove unneeded symbol offsets and _TEXT_BASE Albert ARIBAUD
2013-10-13  8:03 ` Albert ARIBAUD
2013-10-13 15:00 ` Benoît Thébaudeau
2013-10-13 17:16   ` Albert ARIBAUD
2013-10-14  6:59     ` [U-Boot] Handling hangs (was: [PATCH v1] arm: remove unneeded symbol offsets and _TEXT_BASE) Albert ARIBAUD
2013-10-14 10:22       ` Benoît Thébaudeau
2013-10-14 11:32         ` Albert ARIBAUD
2013-10-14 11:47           ` Albert ARIBAUD
2013-10-15 11:07     ` [U-Boot] [PATCH v1] arm: remove unneeded symbol offsets and _TEXT_BASE Benoît Thébaudeau
2013-10-15 11:21       ` Albert ARIBAUD
2013-10-15 12:04     ` Albert ARIBAUD [this message]
2013-11-07 14:15 ` [U-Boot] [PATCH v2 1/2] arm: make _end compiler-generated Albert ARIBAUD
2013-11-07 14:15   ` [U-Boot] [PATCH v2 2/2] arm: remove unneeded symbol offsets and _TEXT_BASE Albert ARIBAUD
2013-11-08 21:40     ` Benoît Thébaudeau
2013-11-08 21:40   ` [U-Boot] [PATCH v2 1/2] arm: make _end compiler-generated Benoît Thébaudeau

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=20131015140455.397e94c5@lilith \
    --to=albert.u.boot@aribaud.net \
    --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