public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] implementation of generic distro configs
Date: Fri, 6 Dec 2013 12:14:58 -0500	[thread overview]
Message-ID: <20131206171458.GZ420@bill-the-cat> (raw)
In-Reply-To: <1386296295-28658-1-git-send-email-dennis@ausil.us>

On Thu, Dec 05, 2013 at 08:18:10PM -0600, Dennis Gilmore wrote:

> As a followup to http://lists.denx.de/pipermail/u-boot/2013-August/160080.html
> ive put together what i Think is a reasonable starting point for defining a
> unified set of features and options. 
> 
> There are some things that really need some more work yet. mostly to do with
> austomatically loading of the dtb.  All of these patches are currently enabled
> in Fedora's u-boot builds.
> 
> the wandboard in particular has received a lot of testing using
> sysboot command to load a extlinux.conf file. which did pick up bugs
> in teh pxe.c code early on.
> I would appreciate some feedback from a wider audience as we move to
> making this be a default configuration.

I think you've got the breakdown in relative locations wrong.  And, per
the email I sent out after this, I think we should start using fdt_high
at least for its intended purpose, moving the DT out of the way of what
we know about.

The breakdown I see in your series is (or similar):
> +	"fdt_addr_r=0x81100000\0" \
> +	"fdt_addr=0x81200000\0" \
> +	"pxefile_addr_r=0x81300000\0" \
> +	"kernel_addr_r=0x81400000\0" \
> +	"ramdisk_addr_r=0x83400000\0" \

Now the issues we have to deal with are:
1) zImage running into ramdisk.  There's 32MiB here, so that's unlikely,
hopefully, for a while at least.
2) When the zImage runs, it will unpack the kernel to top of memory and
then BSS follows.  You've only left ~17MiB for that, and that's iffy.
When you start enabling function tracing and some other stuff, that's
close, on a single platform image.  I bet a multi-platform runs into the
DT.

#2 is why the defaults on these platforms place the DT in memory after
the kernel image, before the ramdisk.  I think we can get a better
situation out of this by saying the u-boot used parts (pxefile) are
as low as we can (start of memory), throw in the reasonable size, then
place the kernel, 32MiB gap, 1MiB gap for each DT (which I really hope
is more than enough..), then the ramdisk.

This just leaves the worry of 32MiB for a kernel + BSS being an uncaught
conflict.  One could stop worrying about this, largely I think, if we
set fdt_high to memory base + 512MiB and then do some corner case
testing to make sure that we do not relocate on top of the end of a
large ramdisk, and that the only cases which simply do not work are the
cases where you flat out do not have enough memory for what you're
trying to load (giant kernel + giant ramdisk on a nowadays-small 128MiB
DDR system for example).

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131206/1de73ded/attachment.pgp>

      parent reply	other threads:[~2013-12-06 17:14 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-06  2:18 [U-Boot] [RFC] implementation of generic distro configs Dennis Gilmore
2013-12-06  2:18 ` [U-Boot] [PATCH 1/5] add a generic set of configs to enable Distros to more easier support u-boot based systems Dennis Gilmore
2013-12-06 10:53   ` Wolfgang Denk
2013-12-06  2:18 ` [U-Boot] [PATCH 2/5] port wandboards to use the generic distro configs Dennis Gilmore
2013-12-06  3:47   ` Robert Nelson
2013-12-06  5:01     ` Dennis Gilmore
2013-12-06  5:06       ` Dennis Gilmore
2013-12-06  5:13         ` Robert Nelson
2013-12-06  5:07       ` Robert Nelson
2013-12-06 10:59   ` Wolfgang Denk
2013-12-06 14:48     ` Dennis Gilmore
2013-12-06 15:26       ` Wolfgang Denk
2013-12-06 16:28         ` Tom Rini
2013-12-06 20:37           ` Wolfgang Denk
2013-12-06 22:13             ` Tom Rini
2013-12-06 22:59               ` Wolfgang Denk
2013-12-06 22:44             ` Dennis Gilmore
2013-12-06 23:16               ` Wolfgang Denk
2013-12-07  0:09                 ` Dennis Gilmore
2013-12-07 12:20                   ` Wolfgang Denk
2013-12-06  2:18 ` [U-Boot] [PATCH 3/5] port omap4 based devices to use " Dennis Gilmore
2013-12-06  2:18 ` [U-Boot] [PATCH 4/5] port beagleboard " Dennis Gilmore
2013-12-06  2:18 ` [U-Boot] [PATCH 5/5] port beaglebones " Dennis Gilmore
2013-12-06  3:31   ` Dennis Gilmore
2013-12-06 17:14 ` Tom Rini [this message]

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=20131206171458.GZ420@bill-the-cat \
    --to=trini@ti.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