linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] patches to allow DTB to be appended to the ARM zImage
Date: Sun, 12 Jun 2011 16:19:31 +0100	[thread overview]
Message-ID: <20110612151931.GK10283@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20110612145751.GA4013@ponder.secretlab.ca>

On Sun, Jun 12, 2011 at 08:57:51AM -0600, Grant Likely wrote:
> On Sun, Jun 12, 2011 at 04:15:23PM +0200, Arnd Bergmann wrote:
> > But when you have both atag and DT and the atag overrides the DT, that
> > means we have incorrect information in the DT, and code might later
> > rely on that information.
> > 
> > IMHO when we allow passing a DT to a kernel while booting from an
> > existing boot loader that only knows about atag, the code that loads
> > the DT should be responsible for updating the DT with the atag information,
> > not pass two conflicting sets of data into the actual kernel.
> 
> I completely agree here.  I /started/ from the position that ATAGs and
> DTB would coexist, and after extensive debate[1] my opinion turned around
> to it should be one or the other.  Otherwise there are all kinds of
> questions about accuracy of the information and which takes
> precedence.

And we've ended up with a fucked up situation which is extremely
fragile, and actually makes me _NOT_ want to convert any existing
platforms to use DT in the least.

This I view as a fundamental blocker which needs addressing before
anyone can make use of DT on ARM.

DT is _not_ the authoritive source of information on systems with
ATAGs.

Imagine this situation: you have your PC.  It provides memory information
through the E820 interface.  You convert your kernel to use DT and it
only uses the information passed from the DT blob, which it loaded as
part of your kernel off disk.

However, your RAM size has changed.  Should the kernel continue to believe
the memory information found in the encapsulated DT blob, or should it
continue to get it from the E820 interface?

It's precisely the same problem here.  The E820 interface _has_ to take
precedence because that is the _authoritive_ source of information.

  reply	other threads:[~2011-06-12 15:19 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-12  6:06 [PATCH 0/3] patches to allow DTB to be appended to the ARM zImage Nicolas Pitre
2011-06-12  6:06 ` [PATCH 1/3] ARM: zImage: ensure it is always a multiple of 64 bits in size Nicolas Pitre
2011-06-13 10:43   ` Tony Lindgren
2011-06-13 11:24   ` Russell King - ARM Linux
2011-06-13 14:06     ` Nicolas Pitre
2011-06-12  6:06 ` [PATCH 2/3] ARM: zImage: Allow the appending of a device tree binary Nicolas Pitre
2011-06-12 15:01   ` Grant Likely
2011-06-13 10:46   ` Tony Lindgren
2011-06-13 11:26     ` Russell King - ARM Linux
2011-06-13 14:08       ` Nicolas Pitre
2011-06-12  6:06 ` [PATCH 3/3] ARM: zImage: make sure appended DTB doesn't get overwritten by kernel .bss Nicolas Pitre
2011-06-13 10:47   ` Tony Lindgren
2011-06-12  8:15 ` [PATCH 0/3] patches to allow DTB to be appended to the ARM zImage Russell King - ARM Linux
2011-06-12  8:34   ` Shawn Guo
2011-06-12  9:21     ` Russell King - ARM Linux
2011-06-12  9:38       ` Shawn Guo
2011-06-12  9:52         ` Russell King - ARM Linux
2011-06-12 10:42           ` Shawn Guo
2011-06-12 10:40             ` Russell King - ARM Linux
2011-06-13 23:04               ` David Brown
2011-06-13 23:13                 ` Nicolas Pitre
2011-06-14  7:09                   ` Nicolas Pitre
2011-06-14 11:25                     ` Shawn Guo
2011-06-14 14:53                     ` Tony Lindgren
2011-06-14 17:28                       ` Nicolas Pitre
2011-06-14 20:32                         ` Arnd Bergmann
2011-06-14 21:21                           ` Nicolas Pitre
2011-06-14 21:42                             ` Arnd Bergmann
2011-06-14 22:06                               ` Grant Likely
2011-06-15  8:08                                 ` Tony Lindgren
2011-06-14 22:32                           ` Rob Herring
2011-06-14 23:50                             ` Nicolas Pitre
2011-06-15  2:09                               ` Rob Herring
2011-06-15  2:21                                 ` Nicolas Pitre
2011-06-14 21:38                     ` David Brown
2011-06-14 23:27                     ` [PATCH] Support multiple MEM tags with atags->fdt conversion David Brown
2011-06-15 19:50                       ` Nicolas Pitre
2011-06-15 20:15                         ` David Brown
2011-06-15 20:20                           ` Nicolas Pitre
2011-06-16  1:43                         ` David Gibson
2011-06-20  4:03                           ` Nicolas Pitre
2011-06-20  4:53                             ` David Gibson
2011-06-17 20:23                         ` David Brown
2011-06-12 11:22     ` [PATCH 0/3] patches to allow DTB to be appended to the ARM zImage Petr Štetiar
2011-06-12 11:58       ` Russell King - ARM Linux
2011-06-12 14:15         ` Arnd Bergmann
2011-06-12 14:34           ` Russell King - ARM Linux
2011-06-12 15:01             ` Arnd Bergmann
2011-06-12 15:35               ` Russell King - ARM Linux
2011-06-12 15:45                 ` Nicolas Pitre
2011-06-13 20:24                 ` Dmitry Eremin-Solenikov
2011-06-13 22:05                   ` Russell King - ARM Linux
2011-06-13 23:33                     ` Grant Likely
2011-06-12 14:57           ` Grant Likely
2011-06-12 15:19             ` Russell King - ARM Linux [this message]
2011-06-12 15:47               ` Nicolas Pitre
2011-06-12 15:59                 ` Russell King - ARM Linux
2011-06-12 18:59                   ` Nicolas Pitre
2011-06-13  9:51                     ` Tony Lindgren
2011-06-13 14:14                       ` Nicolas Pitre
2011-06-13 14:20                         ` Russell King - ARM Linux
2011-06-13 15:02                           ` Tony Lindgren
2011-06-13 15:14                           ` Nicolas Pitre
2011-06-13 15:17                             ` Grant Likely
2011-06-12 19:26                   ` Warner Losh
2011-06-13  9:59                   ` Tony Lindgren
2011-06-12 15:41         ` Nicolas Pitre
2011-06-14  0:13           ` David Brown
2011-09-06 11:23             ` Linus Walleij
2011-06-21  1:40       ` David Gibson
2011-06-13  4:31 ` Grant Likely
2011-06-13 20:44   ` Nicolas Pitre
2011-09-05 15:43     ` Tony Lindgren
2011-09-05 19:32       ` Nicolas Pitre

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=20110612151931.GK10283@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).