From: Dirk Behme <dirk.behme@de.bosch.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: Add option to disable code relocation
Date: Wed, 8 Feb 2012 07:42:12 +0100 [thread overview]
Message-ID: <4F321944.2070803@de.bosch.com> (raw)
In-Reply-To: <CALButCKit3KDQXRY2HcmxXVFgx=BBPdO=_KPwAenPM9vty+HzQ@mail.gmail.com>
On 08.02.2012 00:48, Graeme Russ wrote:
> Hi Wolfgang,
>
> On Wed, Feb 8, 2012 at 10:36 AM, Wolfgang Denk <wd@denx.de> wrote:
>> Dear Graeme Russ,
>>
>> In message <CALButCKfG+guStJP+M5E=NSr34VPhzgbREbxQuXD6028sw6x6A@mail.gmail.com> you wrote:
>>>> If SPL was to determing the relocation address, it would also have to
>>>> read the environment, because there are a number of environment
>>>> variables which can cause (dynamically) the relocation address to
>>>> change.
>>> But this is not neccessarily the case for every board (or even every arch)
>> Not neccessarily, but possible.
>>
>>> For those boards/arches which CAN calculate the relocation address (either
>>> because it is fixed do to npn-variable RAM size, or fixed in relation to
>>> the maximum RAM address) why should we prohibit a method of skipping the
>>> redundant copy operation in a way which is 100% transparent to everyone
>>> else?
>> Can we please focus on unifying the boot process first, before we try
>> to come up with micro-optimizations?
>
> Agreed - And as I have stated before, this capability will come about as
> a natural side-effect of the boot process unification (see current x86
> repo ready for pull for illustrative example plus the INIT_CALL code I
> am currently working on). So there is no need to look at hackish ways
> to sidestep 'relocation' until then (at which point they won't be
> needed anyway)
>
>> Most of the people who complain here that they need to skip
>> relocation are probably wrong in at least two accounts:
>>
>> - They are not actually talking about relocation at all.
>
> Correct - The 'issue' is the additional in-RAM copy
Yes.
Best regards
Dirk
next prev parent reply other threads:[~2012-02-08 6:42 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-05 6:44 [U-Boot] [PATCH] arm: Add option to disable code relocation Simon Glass
2012-02-05 7:39 ` Mike Frysinger
2012-02-05 12:05 ` Marek Vasut
2012-02-05 20:38 ` Mike Frysinger
2012-02-05 21:40 ` Simon Glass
2012-02-05 22:44 ` Wolfgang Denk
2012-02-05 23:23 ` Graeme Russ
2012-02-05 23:32 ` Simon Glass
2012-02-05 23:37 ` Graeme Russ
2012-02-05 23:41 ` Simon Glass
2012-02-05 23:46 ` Graeme Russ
2012-02-07 9:52 ` Graeme Russ
2012-02-06 7:51 ` Wolfgang Denk
2012-02-06 8:43 ` Graeme Russ
2012-02-06 14:49 ` Tom Rini
2012-02-06 19:27 ` Mike Frysinger
2012-02-06 19:46 ` Tom Rini
2012-02-06 20:25 ` Graeme Russ
2012-02-07 6:41 ` Dirk Behme
2012-02-07 23:23 ` Wolfgang Denk
2012-02-07 23:28 ` Graeme Russ
2012-02-07 23:36 ` Wolfgang Denk
2012-02-07 23:48 ` Graeme Russ
2012-02-08 6:42 ` Dirk Behme [this message]
2012-02-08 6:51 ` Dirk Behme
2012-02-08 7:12 ` Simon Glass
2012-02-08 7:16 ` Dirk Behme
2012-02-08 22:05 ` Graeme Russ
2012-02-09 3:38 ` Graeme Russ
2012-02-09 18:30 ` Simon Glass
2012-02-08 14:03 ` Wolfgang Denk
2012-02-06 21:17 ` Albert ARIBAUD
2012-02-06 22:24 ` Wolfgang Denk
2012-02-07 6:51 ` Dirk Behme
2012-02-07 7:25 ` Aneesh V
2012-02-05 23:32 ` Simon Glass
2012-02-05 12:05 ` Marek Vasut
2012-02-05 18:54 ` Wolfgang Denk
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=4F321944.2070803@de.bosch.com \
--to=dirk.behme@de.bosch.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