All of lore.kernel.org
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/5] [ARM] Auto calculate ZRELADDR and provide option for exceptions
Date: Thu, 10 Jun 2010 10:16:08 +0100	[thread overview]
Message-ID: <20100610091608.GA28305@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20100610090015.GD3422@pengutronix.de>

On Thu, Jun 10, 2010 at 11:00:15AM +0200, Uwe Kleine-K?nig wrote:
> What do you think about requiring r4 to be set to physoffset as I did in
> my series?  This way zImage would already know PHYSOFFSET and so didn't
> need to guess ZRELADDR.  OK, until most bootloaders are fixed we need
> to guess, too.  But at least this would provide a way to stop guessing
> wrong for the affected platforms.

That means it won't get implemented.  Look at the facts.  It's taken
_years_ (5+ years) for boot loaders to start passing a value in r1.
We then switched to the ATAG stuff, and it's again taking years for
boot loaders to start passing right ATAG stuff - and most of them don't
get it right.  Eg, lots of uboot are happy to print out the memory
information on the terminal as they start up, but don't pass memory
information to the kernel.

So if we want phys offset in r4, we better realise that it'll take
something like five years of nagging to get it in place, and even then
people will continue to use boot loaders which don't have support.
And if it's only necessary for a handful of platforms, I doubt anyone's
really going to bother - or even validate that r4 is correct.

You have a whole set of other problems - how do you cope with existing
boot loaders which happen to call the kernel with a value in r4 which
_could_ potentially be valid, but should not be used?  At the moment,
r4 could contain _any_ value what so ever.

Let's face it - advertising that "the kernel will eventually start using
r4" doesn't solve the problem either - we did that with the ATAG list
and we know the results from that.

Maybe the right experiment to try this time - if we want r4 to contain
a value - is to say that kernel 2.6.37 will require a value in r4, and
won't boot without it, and will therefore be incompatible with old
boot loaders.

  reply	other threads:[~2010-06-10  9:16 UTC|newest]

Thread overview: 138+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-03  7:36 [PATCH 0/5] Makefile.boot cleanup Eric Miao
2010-06-03  7:36 ` [PATCH 1/5] [ARM] Remove unused PARAMS_PHYS from arch/arm/boot/compressed Eric Miao
2010-06-10  8:48   ` Uwe Kleine-König
2010-06-03  7:36 ` [PATCH 2/5] [ARM] Remove unused INITRD_PHYS " Eric Miao
2010-06-10  8:27   ` Uwe Kleine-König
2010-06-10  9:55     ` Eric Miao
2010-06-03  7:36 ` [PATCH 3/5] [ARM] Make TEXT_OFFSET a configurable option Eric Miao
2010-06-10  8:50   ` Uwe Kleine-König
2010-06-10  9:45     ` Eric Miao
2010-06-10 10:00       ` Uwe Kleine-König
2010-06-03  7:36 ` [PATCH 4/5] [ARM] Auto calculate ZRELADDR and provide option for exceptions Eric Miao
2010-06-07  8:51   ` Tony Lindgren
2010-06-07 15:12   ` Marek Vasut
2010-06-08  6:12     ` Eric Miao
2010-06-10  9:00   ` Uwe Kleine-König
2010-06-10  9:16     ` Russell King - ARM Linux [this message]
2010-06-10  9:35       ` Uwe Kleine-König
2010-06-10  9:38       ` Eric Miao
2010-06-10  9:43         ` Uwe Kleine-König
2010-06-10  9:47           ` Eric Miao
2010-06-10  9:18     ` Eric Miao
2010-06-10  9:58       ` Uwe Kleine-König
2010-06-10 10:20         ` Eric Miao
2010-06-10 22:06           ` Linus Walleij
2010-06-10 22:23       ` Linus Walleij
2010-06-11  1:34         ` Eric Miao
2010-06-10 18:40     ` Nicolas Pitre
2010-06-10 22:17   ` Linus Walleij
2010-06-11  6:37     ` Uwe Kleine-König
2010-06-17 19:33       ` Russell King - ARM Linux
2010-06-11  4:27   ` srinidhi
2010-09-02 21:21   ` Russell King - ARM Linux
2010-09-02 21:27     ` Russell King - ARM Linux
2010-09-03  7:48       ` Uwe Kleine-König
2010-09-03  9:55         ` Uwe Kleine-König
2010-09-03  9:56           ` [PATCH 1/7] ARM: specify ZRELADDR for ARCH_TEGRA Uwe Kleine-König
2010-09-03 17:46             ` Erik Gilling
2010-09-03 19:01               ` Uwe Kleine-König
2010-09-03 19:19                 ` Erik Gilling
2010-09-03 19:23                   ` Uwe Kleine-König
2010-09-08  9:11                     ` Russell King - ARM Linux
2010-09-08 11:56                       ` Russell King - ARM Linux
2010-09-08 12:16                         ` Uwe Kleine-König
2010-09-08 22:07                           ` Russell King - ARM Linux
2010-09-09  7:24                             ` Uwe Kleine-König
2010-09-09 16:56                               ` Erik Gilling
2010-09-03  9:56           ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to calculate zreladdr 
2010-09-03  9:56             ` Uwe Kleine-König
2010-09-03 13:39             ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to Eric Miao
2010-09-03 13:39               ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to calculate zreladdr Eric Miao
2010-09-03 18:53               ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to 
2010-09-03 18:53                 ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to calculate zreladdr Uwe Kleine-König
2010-09-14  8:08             ` Paul Mundt
2010-09-14  8:08               ` Paul Mundt
2010-09-14 12:23               ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to 
2010-09-14 12:23                 ` [PATCH 2/7] [RFC] ARM: shmobile: remove unneeded hack to calculate zreladdr Uwe Kleine-König
2010-09-03  9:56           ` [PATCH 3/7] ARM: specify ZRELADDR for ARCH_H720X Uwe Kleine-König
2010-09-03 13:40             ` Eric Miao
2010-09-03  9:56           ` [PATCH 4/7] ARM: specify ZRELADDR for ARCH_LPC32XX Uwe Kleine-König
2010-09-03 13:41             ` Eric Miao
2010-09-03  9:56           ` [PATCH 5/7] ARM: don't expose CONFIG_ZRELADDR to the users Uwe Kleine-König
2010-09-03 13:30             ` Eric Miao
2010-09-03 18:53               ` Uwe Kleine-König
2010-09-04  8:52                 ` Eric Miao
2010-09-07 22:39                   ` Mathieu Rondonneau
2010-09-08  5:04                     ` Uwe Kleine-König
2010-09-03  9:56           ` [PATCH 6/7] ARM: fix ordering of defaults for config ZRELADDR Uwe Kleine-König
2010-09-03  9:56           ` [PATCH 7/7] ARM: removed unused zreladdr specifications in all Makefile.boot files Uwe Kleine-König
2010-09-03 15:01             ` Eric Miao
2010-09-03 18:48               ` [PATCH 7/7 v2] " Uwe Kleine-König
2010-09-03 20:46           ` [PATCH 4/5] [ARM] Auto calculate ZRELADDR and provide option for exceptions Uwe Kleine-König
2010-09-05 18:52           ` Uwe Kleine-König
2010-09-08  9:14             ` Russell King - ARM Linux
2010-09-08  9:11           ` Russell King - ARM Linux
2010-09-08 20:31             ` Robert Schwebel
2010-09-03 13:33         ` Eric Miao
2010-06-03  7:36 ` [PATCH 5/5] [ARM] Move inclusion of Makefile.boot into arch/arm/boot/bootp Eric Miao
2010-06-10  9:01   ` Uwe Kleine-König
2010-06-03  7:43 ` [PATCH 0/5] Makefile.boot cleanup Eric Miao
2010-06-03  8:00   ` Uwe Kleine-König
2010-06-03  8:27     ` Eric Miao
2010-06-03  7:57 ` Uwe Kleine-König
2010-06-10 18:44   ` Nicolas Pitre
2010-06-11  4:55     ` Uwe Kleine-König
2010-06-17 19:22   ` Nicolas Pitre
2010-06-18  1:31     ` Eric Miao
2010-06-18  2:30     ` Eric Miao
2010-06-10 10:24 ` [PATCH 0/10] cleanup in early boot code Uwe Kleine-König
2010-06-10 10:27   ` [PATCH 01/10] arm/zImage: don't hard code the stack size twice Uwe Kleine-König
2010-06-10 10:31     ` Eric Miao
2010-06-10 18:47       ` Nicolas Pitre
2010-06-10 10:27   ` [PATCH 02/10] arm/zImage: some comments for __armv3_mpu_cache_on Uwe Kleine-König
2010-06-10 21:36     ` Sergei Shtylyov
2010-06-16  3:16     ` Eric Miao
2010-06-16  9:19       ` Russell King - ARM Linux
2010-06-10 10:27   ` [PATCH 03/10] arm/zImage: __armv3_mpu_cache_flush: respect should-be-zero specification Uwe Kleine-König
2010-06-11  1:23     ` Eric Miao
2010-06-10 10:27   ` [PATCH 04/10] arm/zImage: fix comments for cache_on, cache_off and cache_clean_flush Uwe Kleine-König
2010-06-17  9:28     ` Eric Miao
2010-06-10 10:27   ` [PATCH 05/10] arm/zImage: annotate debug functions about corrupted registers Uwe Kleine-König
2010-06-17  9:37     ` Eric Miao
2010-06-17  9:49       ` Uwe Kleine-König
2010-06-10 10:27   ` [PATCH 06/10] arm: remove bit-rotten STANDALONE_DEBUG for decompressor Uwe Kleine-König
2010-06-17  9:50     ` Eric Miao
2010-06-10 10:27   ` [PATCH 07/10] arm/zImage: don't define unused symbol initrd_phys Uwe Kleine-König
2010-06-17  9:19     ` Eric Miao
2010-06-10 10:27   ` [PATCH 08/10] arm: deprecate support for old way to pass kernel parameters Uwe Kleine-König
2010-06-17  9:58     ` Eric Miao
2010-06-17 23:00       ` Russell King - ARM Linux
2010-06-10 10:27   ` [PATCH 09/10] arm/uImage: require passing a LOADADDR when building with RUNTIME_PHYSOFFSET Uwe Kleine-König
2010-06-10 10:27   ` [PATCH 10/10] arm: Allow PHYS_OFFSET to be runtime determined Uwe Kleine-König
2010-06-10 10:28   ` [PATCH 0/10] cleanup in early boot code Eric Miao
2010-07-05 14:13 ` [PATCH 0/5] Makefile.boot cleanup Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 01/11] ARM: zImage: don't hard code the stack size twice Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 02/11] ARM: zImage: some comments for __armv3_mpu_cache_on Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 03/11] ARM: zImage: __armv3_mpu_cache_flush: respect should-be-zero specification Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 04/11] ARM: zImage: fix comments for cache_on, cache_off and cache_clean_flush Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 05/11] ARM: zImage: annotate debug functions about corrupted registers Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 06/11] ARM: remove bit-rotten STANDALONE_DEBUG for decompressor Uwe Kleine-König
2010-07-05 15:36     ` Nicolas Pitre
2010-07-05 14:15   ` [PATCH 07/11] ARM: zImage: don't define unused symbol initrd_phys Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 08/11] ARM: deprecate support for old way to pass kernel parameters Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 09/11] ARM: uImage: require passing a LOADADDR when building with RUNTIME_PHYSOFFSET Uwe Kleine-König
2010-07-05 16:03     ` Nicolas Pitre
2010-07-05 14:15   ` [PATCH 10/11] ARM: Remove unused PARAMS_PHYS from arch/arm/boot/compressed Uwe Kleine-König
2010-07-05 14:15   ` [PATCH 11/11] Auto calculate ZRELADDR and provide option for exceptions Uwe Kleine-König
2010-07-05 15:31     ` Nicolas Pitre
2010-07-06  2:09       ` Eric Miao
2010-07-31 15:54     ` Russell King - ARM Linux
2010-07-31 17:44       ` Nicolas Pitre
2010-08-02  8:42       ` [PATCH] ARM: don't expose CONFIG_ZRELADDR to the users Uwe Kleine-König
2010-07-07 14:45   ` [PATCH 0/5] Makefile.boot cleanup Uwe Kleine-König
2010-07-07 15:54     ` Nicolas Pitre
2010-07-08  7:08       ` Uwe Kleine-König
2010-07-08 20:49         ` Nicolas Pitre
2010-07-10  9:16         ` Eric Miao
2010-07-21  8:20         ` Uwe Kleine-König
2010-07-29 16:17           ` Git pull request, was " 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=20100610091608.GA28305@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 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.