All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 7/7] sunxi: Add environment settings to make extlinux.conf booting work
Date: Fri, 01 Aug 2014 15:43:23 -0600	[thread overview]
Message-ID: <53DC09FB.2060607@wwwdotorg.org> (raw)
In-Reply-To: <20140801204307.GA19374@bill-the-cat>

On 08/01/2014 02:43 PM, Tom Rini wrote:
> On Fri, Aug 01, 2014 at 02:22:40PM -0600, Stephen Warren wrote:
>> On 08/01/2014 02:05 PM, Dennis Gilmore wrote:
>>> On Fri, 01 Aug 2014 12:57:31 -0600
>>> Stephen Warren <swarren@wwwdotorg.org> wrote:
>>>
>>>> On 08/01/2014 01:46 AM, Hans de Goede wrote:
>>>>> Automatic booting using an extlinux.conf file requires various
>>>>> environment variables to be set.
>>>>
>>>> Acked-by: Stephen Warren <swarren@nvidia.com>
>>>>
>>>> I'd personally be tempted to set fdt_high=0xffffffff,
>>>> initrd_high=0xffffffff to stop U-Boot copying the DT/initrd from the
>>>> load location to some other location under 256M, but that's just an
>>>> optimization and entirely optional.
>>>
>>> There has been quite a few times where using 0xffffff has caused
>>> issues.
>>
>> What kind of issues?
>>
>> At least for Tegra, I've carefully chosen the values for the various
>> load addresses so that there won't be issues. (Without that I can
>> easily see the potential for issues.) I've seen far more repeated
>> problems when U-Boot moves the DT/initrd around than than when it
>> didn't (none in that case). Besides, it's completely redundant and
>> unnecessary work if the blobs are already loaded at sane addresses,
>> which they are on Tegra at least.
>
> Just how large of a kernel have you thrown on a Tegra?  32MB might seem
> reasonable at first but it wouldn't be overly surprised if someone can
> shove a BSS into there.  I know I shoved DT into 128MB by default
> because a 32bit ARM kernel isn't functional at that size.

There's enough space for the following:

16M decompressed kernel
16M compressed kernel
1M DTB

The kernel I typically boot is ~4.MB zImage, and ~9MB Image (which 
doesn't include BSS).

Note that if the decompressed/compressed kernel overlap, the kernel 
decompresser takes care of relocating the compressed kernel before doing 
the decompression, and IIRC is supposed to take care not to clobber the 
DTB (or initrd?) when doing so.

Fedora's rawhide zImage is a bit larger at ~6MB. That works fine too.

  reply	other threads:[~2014-08-01 21:43 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-01  7:46 [U-Boot] [PATCH 1/7] sunxi-common.h: Remove CONFIG_SYS_BOOT_GET_CMDLINE Hans de Goede
2014-08-01  7:46 ` [U-Boot] [PATCH 2/7] sunxi-common.h: Remove dead #ifdef CONFIG_CMD_NET code block Hans de Goede
2014-08-04 21:19   ` Ian Campbell
2014-08-01  7:46 ` [U-Boot] [PATCH 3/7] sunxi-common.h: Don't undefine CONFIG_CMD_NFS Hans de Goede
2014-08-04 21:19   ` Ian Campbell
2014-08-01  7:46 ` [U-Boot] [PATCH 4/7] sunxi-common.h: Don't undef CONFIG_CMD_NET only to redefine it again later Hans de Goede
2014-08-04 21:20   ` Ian Campbell
2014-08-01  7:46 ` [U-Boot] [PATCH 5/7] sunxi-common.h: Use new generic $bootcmd Hans de Goede
2014-08-01  7:48   ` Hans de Goede
2014-08-01 18:54   ` Stephen Warren
2014-08-04 18:35     ` Hans de Goede
2014-08-04 21:21       ` Ian Campbell
2014-08-01  7:46 ` [U-Boot] [PATCH 6/7] sunxi: Kconfig: move common settings into a shared code block Hans de Goede
2014-08-04 21:26   ` Ian Campbell
2014-08-05  1:24     ` Chen-Yu Tsai
2014-08-05  7:03       ` Ian Campbell
2014-08-01  7:46 ` [U-Boot] [PATCH 7/7] sunxi: Add environment settings to make extlinux.conf booting work Hans de Goede
2014-08-01 18:57   ` Stephen Warren
2014-08-01 20:05     ` Dennis Gilmore
2014-08-01 20:22       ` Stephen Warren
2014-08-01 20:30         ` Dennis Gilmore
2014-08-01 20:43         ` Tom Rini
2014-08-01 21:43           ` Stephen Warren [this message]
2014-08-02  1:53             ` Tom Rini
2014-08-04 17:31               ` Stephen Warren
2014-08-04 20:12                 ` Tom Rini
2014-08-04 21:29       ` Ian Campbell
2014-08-05  8:20         ` Hans de Goede
2014-08-04 21:27   ` Ian Campbell
2014-08-05  8:21     ` Hans de Goede
2014-08-30 21:57   ` Ian Campbell
2014-08-31 10:07     ` Hans de Goede
2014-08-04 21:17 ` [U-Boot] [PATCH 1/7] sunxi-common.h: Remove CONFIG_SYS_BOOT_GET_CMDLINE Ian Campbell

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=53DC09FB.2060607@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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.