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] Memory test post relocation
Date: Fri, 14 Nov 2014 16:21:04 +0100	[thread overview]
Message-ID: <20141114162104.5cd462e1@lilith> (raw)
In-Reply-To: <1415974738890-196181.post@n7.nabble.com>

Hello Dave,

On Fri, 14 Nov 2014 07:18:58 -0700 (MST), DaveKucharczyk
<david.kucharczyk@gmail.com> wrote:
> Hi Albert,
> Thanks for the great information.
> 
> Albert ARIBAUD (U-Boot) wrote
> >  Baaad, bad. The first time you change something in your code, your
> > relocation offset might change and this will make U-Boot crash and burn in
> > interesting ways.
> > 
> > Just define CONFIG_SYS_TEXT_BASE to some low address and let relocation
> > happen. You can find your actual relocation address in the global data
> > structure. 
> 
> If I set CONFIG_SYS_TEXT_BASE to a low address then I can't run a memory
> test starting at the lowest address because that's where the U-Boot code
> will be. 

No, that's not where it'll be; it'll relocate as high as it can.

> I think the best thing would be to run a memory test on the full 2GB before
> relocation happens. Is that possible? 

Not with U-Boot in DDR. :)

> Albert ARIBAUD (U-Boot) wrote
> > How do you know the lowest address used by your stack during your memory
> > test?
> 
> I know the address of the stack pointer from DEBUG message...

So you know it *at a given time*, but you have no assurance that it
won't go further down.

> I erroneously thought that was the beginning of the stack. The stack expands
> down form that address, but how much?  I don't see a global data stack size
> variable.

There is none physically, i.e. that's just the size you reserve for it,
but it could grow beyond that, although it's not likely. Still, that
means you don't test all DDR.

> Before, we have #define CONFIG_STACKSIZE	(128 * 1024) set in the header
> file.     
> 
> Basically what is the best way to run the memory test? If I can run it
> before relocation then it would make things very simple. 

If you happen to have an SPL running from some RAM and not from DDR,
then you could perform the full DDR test there.

Amicalement,
-- 
Albert.

  parent reply	other threads:[~2014-11-14 15:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-13 22:19 [U-Boot] Memory test post relocation DaveKucharczyk
2014-11-14  8:20 ` Albert ARIBAUD
2014-11-14 14:18   ` DaveKucharczyk
2014-11-14 14:27     ` Fabio Estevam
2014-11-14 14:34       ` DaveKucharczyk
2014-11-14 15:21     ` Albert ARIBAUD [this message]
2014-11-14 15:54       ` DaveKucharczyk
2014-11-14 16:07         ` DaveKucharczyk
2014-11-14 17:50           ` Albert ARIBAUD
2014-11-14 17:51         ` Albert ARIBAUD
2014-11-14 18:41           ` DaveKucharczyk
2014-11-14 20:20             ` DaveKucharczyk
2014-11-14 22:22               ` DaveKucharczyk
2014-11-15 12:13                 ` Albert ARIBAUD
2014-11-15 12:26               ` Albert ARIBAUD
2014-11-15 12:37                 ` Albert ARIBAUD
2014-11-14 16:15     ` 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=20141114162104.5cd462e1@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