public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime.ripard@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] sunxi: Fix boot of Cubietruk and al.
Date: Thu, 19 Oct 2017 15:24:57 +0200	[thread overview]
Message-ID: <20171019132457.dcsamtfncimf53af@flea> (raw)
In-Reply-To: <829444bb-fa9e-40a4-3b58-f5506557f89b@arm.com>

On Thu, Oct 19, 2017 at 02:03:55PM +0100, Andre Przywara wrote:
> Hi,
> 
> On 19/10/17 09:26, Maxime Ripard wrote:
> > Hi,
> > 
> > Most featureful boards, such as the Cubietruck, have been broken since
> > the release 2017.09.
> > 
> > This is due to a size increase of the binary that will trip us across
> > the size we've been using in the u-boot-sunxi-with-spl.bin file.
> > 
> > We would have two ways to work around it. The first one would be to
> > just increase the offset of the environment. However, since it would
> > break all the environments of our users and possibly the custom
> > partition scheme that they would have created, it doesn't really seem
> > like a smart move.
> 
> Is that really such a problem? How many people rely on having their
> custom environment preserved over an update? (That's an honest question)

All of them, I guess. In your U-boot upgrade script, do you do a 'env
default -a; saveenv' all the time ?

I know I don't.

And I guess the question should be turned the other way around. Should
you expect your environment to be erased / ignored by any upgrade?

There's never been any documentation on this as far as I know, so
there's probably people that will expect it to be fixed, and things
keep on booting.

> I see that the environment is hardcoded to 0x88000 in env/Kconfig.
> Where does this value come from? Why is it this rather arbitrary value?

It predates my involvement in U-Boot, so I don't really know, but I
guess some arbitrary value needed to be picked and someone did (maybe
Hans or Enrick) because it seemed reasonable at the time.

> I believe the real limit should be 1MB - ENV_SIZE.
> Most sane partitioning tools put the first partition at 1MB, so this
> leaves the space below that to the bootloader/firmware.

Is that 1MB after the partition table or 1MB since the beginning of
the device?

> This would put the actual limit at 856 KB for now - that should be
> enough for everybody ;-)
> We could even push this further by reducing ENV_SIZE to say 64KB.
> 
> At least that would buy us some time to address this issue in a more
> sustainable way.

Yeah, but even if we could address the issues mentionned above, we'd
still need to take care of the partitions for let's say the
environment that would need to be moved as well. This is just not
practical.

I guess we could introduce a new image with more room for the u-boot
binary, and advertise it as such. But we would still have to build the
old one for quite some time.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171019/75b7f702/attachment.sig>

  reply	other threads:[~2017-10-19 13:24 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-19  8:26 [U-Boot] [PATCH 0/3] sunxi: Fix boot of Cubietruk and al Maxime Ripard
2017-10-19  8:26 ` [U-Boot] [PATCH 1/3] ARM: sunxi: Disable USB host options by default Maxime Ripard
2017-10-19  8:48   ` Alexander Graf
2017-10-19  9:11     ` Andre Przywara
2017-10-19 11:55       ` Maxime Ripard
2017-10-19  8:54   ` Peter Robinson
2017-10-19 11:44   ` Mark Kettenis
2017-10-19  8:26 ` [U-Boot] [PATCH 2/3] ARM: sunxi: Disable FAT write " Maxime Ripard
2017-10-19  8:26 ` [U-Boot] [PATCH 3/3] efi_loader: Do not enable it by default for sunxi Maxime Ripard
2017-10-19  8:43   ` Peter Robinson
2017-10-19  9:01     ` Maxime Ripard
2017-10-19  9:06       ` Peter Robinson
2017-10-19  9:12         ` Peter Robinson
2017-10-19 11:43           ` Maxime Ripard
2017-10-19 21:40             ` Rob Clark
2017-10-20  7:20               ` Maxime Ripard
2017-10-20 12:27                 ` Peter Robinson
2017-10-20 12:36                   ` Maxime Ripard
2017-10-20 12:54                     ` Tom Rini
2017-10-20 16:39                       ` Maxime Ripard
2017-10-20 17:57                         ` Tom Rini
2017-10-20 18:52                         ` Peter Robinson
2017-10-20 12:56                     ` Peter Robinson
2017-10-19  9:12         ` Maxime Ripard
2017-10-23 13:35     ` Heinrich Schuchardt
2017-10-19  8:51   ` Alexander Graf
2017-10-19 10:54     ` Jonathan Gray
2017-10-19 11:52     ` Maxime Ripard
2017-10-19 11:39   ` Mark Kettenis
2017-10-19 11:48     ` Maxime Ripard
2017-10-19 13:24   ` Tom Rini
2017-10-19  8:44 ` [U-Boot] [PATCH 0/3] sunxi: Fix boot of Cubietruk and al Alexander Graf
2017-10-19  9:11   ` Maxime Ripard
2017-10-19 12:10     ` Alexander Graf
2017-10-19  9:10 ` Siarhei Siamashka
2017-10-19 13:20   ` Tom Rini
2017-10-19 13:26   ` Maxime Ripard
2017-10-19 13:03 ` Andre Przywara
2017-10-19 13:24   ` Maxime Ripard [this message]
2017-10-19 13:31     ` Tom Rini
2017-10-19 14:42     ` Andre Przywara
2017-10-19 14:58       ` Maxime Ripard
2017-10-20 21:33         ` Dennis Gilmore
2017-10-23  7:35           ` Maxime Ripard
2017-10-24 17:05             ` Dennis Gilmore
2017-10-24 17:21               ` Andre Przywara
2017-10-25  9:42                 ` Maxime Ripard
2017-10-25  9:55                   ` Jagan Teki
2017-10-25 12:30                     ` Maxime Ripard
2017-10-25 10:01                   ` Andre Przywara
2017-10-25 11:58                 ` Siarhei Siamashka
2017-10-25 13:42                   ` Andre Przywara
     [not found]                     ` <20171025184101.aihe47qongf52e7c@excalibur.cnev.de>
2017-10-26  1:46                       ` Dennis Gilmore
2017-10-25 13:45                   ` Tom Rini
2017-10-25 13:50                   ` Maxime Ripard
2017-10-20 21:31     ` Dennis Gilmore
2017-10-19 13:28 ` Tom Rini
2017-10-19 13:50   ` Maxime Ripard

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=20171019132457.dcsamtfncimf53af@flea \
    --to=maxime.ripard@free-electrons.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