public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Ian Campbell <Ian.Campbell@citrix.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/4] config_distro_bootcmd: Scan all partitions for boot files
Date: Wed, 7 Jan 2015 10:22:09 +0000	[thread overview]
Message-ID: <1420626129.18631.17.camel@citrix.com> (raw)
In-Reply-To: <1420625436.15910.201.camel@collabora.co.uk>

On Wed, 2015-01-07 at 11:10 +0100, Sjoerd Simons wrote:
> On Tue, 2015-01-06 at 17:43 -0700, Stephen Warren wrote:
> > (CCing Dennis so he can comment from a distro perspective re: partition 
> > table bootable flags v.s. scanning all partitions)
> > 
> > On 01/06/2015 10:07 AM, Sjoerd Simons wrote:
> > > On Mon, 2015-01-05 at 13:24 -0700, Stephen Warren wrote:
> > >> On 01/05/2015 10:13 AM, Sjoerd Simons wrote:
> 
> > > Well my thoughts on the matter are above, If folks feel strongly about
> > > this approach being the wrong way I'd love to hear their arguments :).
> > 
> > One issue with this approach is that there's no way for the user to 
> > short-circuit the scanning. If I put a ChromiumOS install on an SD card 
> > and leave it plugged into a system that's going to end up booting from 
> > eMMC since that's where the boot files are, there are lots of partitions 
> > to scan on that SD card, which will be a bit annoying.
> 
> 
> I don't remember exactly how many partitions with fat/ext* filesystems a
> ChromiumOS installation has (order of 3-5 iirc?), but indeed it means
> your boot will be a bit slower due to it probing more partitions.
> Wouldn't expect it to significantly slow down the total boot time
> though.

I thought u-boot would scan that partitions which were marked bootable
first, in which case you just need to set the bit correctly in the
partition table. I might be wrong about that though. (and re-reading
$subject, it seems like changing this is the subject of the thread...)

> I didn't think of this one my WIP is on an Odroid X2 which has a boot
> selector jumper, so I have it always starting from mmc0 (which is either
> SD or EMMC depending on the jumper setting). 
> 
> However, it raises an interesting question. The current convention for
> Exynos is to first scans external storage (SD, mmc 1) and then internal
> storage (eMMC, mmc 0), which opens up a whole different can of worms. As
> that means that e.g. my chromebook will try to boot from whatever random
> SD i've put into it first rather the OS installed on eMMC.  It would be
> nice to have some general guidelines in this area so the behaviour of
> various boards can be somewhat consistent in the default behaviour.

My understanding was that the various ${boot_*} envvars, including
boot_targets, are considered to be user serviceable parts. IOW if you
want to boot from mmc0 only then:
        setenv boot_targets mmc0
        saveenv

Maybe it makes sense for the default boot order to differ depending on
the specific exynos platform though?

Ian.

> (Added Ian Cambell to the cc as he introduce the usage on exynos
> devices)
> 

  reply	other threads:[~2015-01-07 10:22 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-05 17:13 [U-Boot] [PATCH 0/4] Let the distro boot command scan all partitions Sjoerd Simons
2015-01-05 17:13 ` [U-Boot] [PATCH 1/4] fs: Add command to retrieve the filesystem type Sjoerd Simons
2015-01-05 20:18   ` Stephen Warren
2015-01-06 16:40     ` Sjoerd Simons
2015-01-06 17:05       ` Stephen Warren
2015-02-02 18:57   ` [U-Boot] [U-Boot, " Tom Rini
2015-01-05 17:13 ` [U-Boot] [PATCH 2/4] part: let list put the list in an environment variable Sjoerd Simons
2015-01-05 20:21   ` Stephen Warren
2015-02-02 18:57   ` [U-Boot] [U-Boot, " Tom Rini
2015-01-05 17:13 ` [U-Boot] [PATCH 3/4] config_distro_bootcmd: Scan all partitions for boot files Sjoerd Simons
2015-01-05 20:24   ` Stephen Warren
2015-01-06 17:07     ` Sjoerd Simons
2015-01-07  0:43       ` Stephen Warren
2015-01-07 10:10         ` Sjoerd Simons
2015-01-07 10:22           ` Ian Campbell [this message]
2015-01-07 11:01             ` Sjoerd Simons
2015-01-07 11:17               ` Ian Campbell
2015-01-07 11:46                 ` Sjoerd Simons
2015-01-07 12:47                   ` Ian Campbell
2015-01-10 18:34                   ` Dennis Gilmore
2015-01-07 20:22                 ` Stephen Warren
2015-01-08  9:24                   ` Sjoerd Simons
2015-01-07 20:19           ` Stephen Warren
2015-01-10 18:27         ` Dennis Gilmore
2015-01-12 17:42           ` Stephen Warren
2015-01-12 18:44             ` Dennis Gilmore
2015-01-13  8:40               ` Sjoerd Simons
2015-01-13 20:52                 ` Stephen Warren
2015-02-02 18:57   ` [U-Boot] [U-Boot, " Tom Rini
2015-01-05 17:13 ` [U-Boot] [PATCH 4/4] distro_distro_bootcmd: use CONFIG_BOOTCOMMAND instead of setting bootcmd= Sjoerd Simons
2015-01-05 20:31   ` Stephen Warren
2015-01-06 16:26     ` Sjoerd Simons
2015-02-02 18:57   ` [U-Boot] [U-Boot, " Tom Rini

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=1420626129.18631.17.camel@citrix.com \
    --to=ian.campbell@citrix.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