All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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 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.