From: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: Nicolas Pitre <nico-vtqb6HGKxmzR7s880joybQ@public.gmane.org>,
Russell King - ARM Linux
<linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
Erik Gilling <konkers-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org>,
Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
Peter De Schrijver
<pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"Colin Cross (ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org)"
<ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org>,
Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>,
"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH] arm/tegra: select AUTO_ZRELADDR by default
Date: Fri, 14 Oct 2011 17:06:02 -0500 [thread overview]
Message-ID: <4E98B24A.2070408@gmail.com> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF173BE1A3A6-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
On 10/14/2011 04:01 PM, Stephen Warren wrote:
> Nicolas Pitre wrote at Friday, October 14, 2011 2:45 PM:
>> On Fri, 14 Oct 2011, Stephen Warren wrote:
> ...
>>> I did originally briefly look into getting U-Boot to boot a zImage, but
>>> that looked like a far more invasive patch. There were rumours of some
>>> chip's custom U-Boot already having such support, but I couldn't find
>>> it, nor any evidence of such support in mainline U-Boot.
>>
>> FRom my clone of the u-Boot repo:
>>
>> $ git grep -l zImage
>> README
>> arch/sh/lib/zimageboot.c
>> arch/x86/lib/zimage.c
>> ...
>>
>> Even the name of some of those files clearly hints zImage support.
>>
>> In any case, loading zImage should be even simpler than loading uImage.
>> It is the same as loading uImage except that you just have to skip the
>> checking and relocating steps.
>
> Just by way of background in case anyone is wondering why I wrote the
> patch I did:
>
> Those files both implement custom commands "zimageboot" and "zboot". I
> was looking for integration with the existing "bootm" command.
>
> The advantage of re-using "bootm" for this is that it already supports
> all the stuff like setting up kernel command-lines, initrds, knowing how
> to pass the FDT to the kernel, and whatever other OS-specific setup might
> be required.
>
> The disadvantage of adding zImage support to bootm is that I'd have to
> teach a bunch of U-Boot image handling code about a new image format; it
> already knows about "legacy uImage", "FIT" images, and I'd have to add a
> third "zImage" format. Doing so would at least require adding a lot of
> "case IMAGE_FORMAT_ZIMAGE" everywhere, but it'd probably be best to add
> some kind of vtable for image formats to move all the image-format
> knowledge into format-specific files, leaving the users of the images
> with much smaller code.
>
> I didn't feel like making such a large change. Hence, I chose to make a
> small change to the existing uImage support.
>
> Now admittedly I did say I didn't find any traces of zImage support, which
> isn't what I'm saying here; I guess I forgot about the stuff I did find
> soon after I chose the path of modifying the uImage formats.
>
FYI, this exact topic has been discussed on the Linaro weekly
boot-architecture call and list. The discussion has been more general in
terms of what can be done to make installing and updating kernels
easier/work for distros.
https://wiki.linaro.org/OfficeofCTO/BootArchitecture
Rob
WARNING: multiple messages have this Message-ID (diff)
From: robherring2@gmail.com (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm/tegra: select AUTO_ZRELADDR by default
Date: Fri, 14 Oct 2011 17:06:02 -0500 [thread overview]
Message-ID: <4E98B24A.2070408@gmail.com> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF173BE1A3A6@HQMAIL01.nvidia.com>
On 10/14/2011 04:01 PM, Stephen Warren wrote:
> Nicolas Pitre wrote at Friday, October 14, 2011 2:45 PM:
>> On Fri, 14 Oct 2011, Stephen Warren wrote:
> ...
>>> I did originally briefly look into getting U-Boot to boot a zImage, but
>>> that looked like a far more invasive patch. There were rumours of some
>>> chip's custom U-Boot already having such support, but I couldn't find
>>> it, nor any evidence of such support in mainline U-Boot.
>>
>> FRom my clone of the u-Boot repo:
>>
>> $ git grep -l zImage
>> README
>> arch/sh/lib/zimageboot.c
>> arch/x86/lib/zimage.c
>> ...
>>
>> Even the name of some of those files clearly hints zImage support.
>>
>> In any case, loading zImage should be even simpler than loading uImage.
>> It is the same as loading uImage except that you just have to skip the
>> checking and relocating steps.
>
> Just by way of background in case anyone is wondering why I wrote the
> patch I did:
>
> Those files both implement custom commands "zimageboot" and "zboot". I
> was looking for integration with the existing "bootm" command.
>
> The advantage of re-using "bootm" for this is that it already supports
> all the stuff like setting up kernel command-lines, initrds, knowing how
> to pass the FDT to the kernel, and whatever other OS-specific setup might
> be required.
>
> The disadvantage of adding zImage support to bootm is that I'd have to
> teach a bunch of U-Boot image handling code about a new image format; it
> already knows about "legacy uImage", "FIT" images, and I'd have to add a
> third "zImage" format. Doing so would at least require adding a lot of
> "case IMAGE_FORMAT_ZIMAGE" everywhere, but it'd probably be best to add
> some kind of vtable for image formats to move all the image-format
> knowledge into format-specific files, leaving the users of the images
> with much smaller code.
>
> I didn't feel like making such a large change. Hence, I chose to make a
> small change to the existing uImage support.
>
> Now admittedly I did say I didn't find any traces of zImage support, which
> isn't what I'm saying here; I guess I forgot about the stuff I did find
> soon after I chose the path of modifying the uImage formats.
>
FYI, this exact topic has been discussed on the Linaro weekly
boot-architecture call and list. The discussion has been more general in
terms of what can be done to make installing and updating kernels
easier/work for distros.
https://wiki.linaro.org/OfficeofCTO/BootArchitecture
Rob
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robherring2@gmail.com>
To: Stephen Warren <swarren@nvidia.com>
Cc: Nicolas Pitre <nico@fluxnic.net>,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
Erik Gilling <konkers@android.com>, Arnd Bergmann <arnd@arndb.de>,
Peter De Schrijver <pdeschrijver@nvidia.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
"Colin Cross (ccross@android.com)" <ccross@android.com>,
Olof Johansson <olof@lixom.net>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] arm/tegra: select AUTO_ZRELADDR by default
Date: Fri, 14 Oct 2011 17:06:02 -0500 [thread overview]
Message-ID: <4E98B24A.2070408@gmail.com> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF173BE1A3A6@HQMAIL01.nvidia.com>
On 10/14/2011 04:01 PM, Stephen Warren wrote:
> Nicolas Pitre wrote at Friday, October 14, 2011 2:45 PM:
>> On Fri, 14 Oct 2011, Stephen Warren wrote:
> ...
>>> I did originally briefly look into getting U-Boot to boot a zImage, but
>>> that looked like a far more invasive patch. There were rumours of some
>>> chip's custom U-Boot already having such support, but I couldn't find
>>> it, nor any evidence of such support in mainline U-Boot.
>>
>> FRom my clone of the u-Boot repo:
>>
>> $ git grep -l zImage
>> README
>> arch/sh/lib/zimageboot.c
>> arch/x86/lib/zimage.c
>> ...
>>
>> Even the name of some of those files clearly hints zImage support.
>>
>> In any case, loading zImage should be even simpler than loading uImage.
>> It is the same as loading uImage except that you just have to skip the
>> checking and relocating steps.
>
> Just by way of background in case anyone is wondering why I wrote the
> patch I did:
>
> Those files both implement custom commands "zimageboot" and "zboot". I
> was looking for integration with the existing "bootm" command.
>
> The advantage of re-using "bootm" for this is that it already supports
> all the stuff like setting up kernel command-lines, initrds, knowing how
> to pass the FDT to the kernel, and whatever other OS-specific setup might
> be required.
>
> The disadvantage of adding zImage support to bootm is that I'd have to
> teach a bunch of U-Boot image handling code about a new image format; it
> already knows about "legacy uImage", "FIT" images, and I'd have to add a
> third "zImage" format. Doing so would at least require adding a lot of
> "case IMAGE_FORMAT_ZIMAGE" everywhere, but it'd probably be best to add
> some kind of vtable for image formats to move all the image-format
> knowledge into format-specific files, leaving the users of the images
> with much smaller code.
>
> I didn't feel like making such a large change. Hence, I chose to make a
> small change to the existing uImage support.
>
> Now admittedly I did say I didn't find any traces of zImage support, which
> isn't what I'm saying here; I guess I forgot about the stuff I did find
> soon after I chose the path of modifying the uImage formats.
>
FYI, this exact topic has been discussed on the Linaro weekly
boot-architecture call and list. The discussion has been more general in
terms of what can be done to make installing and updating kernels
easier/work for distros.
https://wiki.linaro.org/OfficeofCTO/BootArchitecture
Rob
next prev parent reply other threads:[~2011-10-14 22:06 UTC|newest]
Thread overview: 117+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-28 1:07 [PATCH] arm/tegra: select AUTO_ZRELADDR by default Peter De Schrijver
2011-09-28 1:07 ` Peter De Schrijver
[not found] ` <1317172068-14872-1-git-send-email-pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2011-09-28 17:50 ` Stephen Warren
2011-09-28 17:50 ` Stephen Warren
2011-09-28 17:50 ` Stephen Warren
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF173955580F-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-10-13 23:38 ` Olof Johansson
2011-10-13 23:38 ` Olof Johansson
2011-10-13 23:38 ` Olof Johansson
[not found] ` <CAOesGMiuzF47kdmaaFny9Fg1ieox6a75tMFfLRAptvDxz_QPMg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-14 7:15 ` Russell King - ARM Linux
2011-10-14 7:15 ` Russell King - ARM Linux
2011-10-14 7:15 ` Russell King - ARM Linux
2011-10-14 14:45 ` Stephen Warren
2011-10-14 14:45 ` Stephen Warren
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF173BE1A260-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-10-14 15:29 ` Arnd Bergmann
2011-10-14 15:29 ` Arnd Bergmann
2011-10-14 15:29 ` Arnd Bergmann
[not found] ` <201110141729.41515.arnd-r2nGTMty4D4@public.gmane.org>
2011-10-14 16:12 ` Stephen Warren
2011-10-14 16:12 ` Stephen Warren
2011-10-14 16:12 ` Stephen Warren
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF173BE1A283-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-10-14 16:27 ` Arnd Bergmann
2011-10-14 16:27 ` Arnd Bergmann
2011-10-14 16:27 ` Arnd Bergmann
[not found] ` <201110141827.53906.arnd-r2nGTMty4D4@public.gmane.org>
2011-10-14 16:44 ` Olof Johansson
2011-10-14 16:44 ` Olof Johansson
2011-10-14 16:44 ` Olof Johansson
[not found] ` <CAOesGMgDwMeVmwo1WJ3tU+ZdjCSJ6F8mY42e_KwTmzAkGQsqmg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-14 18:03 ` Nicolas Pitre
2011-10-14 18:03 ` Nicolas Pitre
2011-10-14 18:03 ` Nicolas Pitre
2011-10-14 19:20 ` Russell King - ARM Linux
2011-10-14 19:20 ` Russell King - ARM Linux
[not found] ` <20111014192011.GS21648-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-10-14 20:06 ` Nicolas Pitre
2011-10-14 20:06 ` Nicolas Pitre
2011-10-14 20:06 ` Nicolas Pitre
2011-10-14 20:12 ` Russell King - ARM Linux
2011-10-14 20:12 ` Russell King - ARM Linux
[not found] ` <20111014201223.GV21648-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-10-14 20:16 ` Stephen Warren
2011-10-14 20:16 ` Stephen Warren
2011-10-14 20:16 ` Stephen Warren
2011-10-14 20:19 ` Russell King - ARM Linux
2011-10-14 20:19 ` Russell King - ARM Linux
2011-10-15 15:29 ` Tixy
2011-10-15 15:29 ` Tixy
2011-10-15 15:29 ` Tixy
2011-10-14 20:26 ` Nicolas Pitre
2011-10-14 20:26 ` Nicolas Pitre
2011-10-14 20:26 ` Nicolas Pitre
2011-10-14 20:14 ` Stephen Warren
2011-10-14 20:14 ` Stephen Warren
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF173BE1A362-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-10-14 20:45 ` Nicolas Pitre
2011-10-14 20:45 ` Nicolas Pitre
2011-10-14 20:45 ` Nicolas Pitre
[not found] ` <alpine.LFD.2.02.1110141631530.17040-QuJgVwGFrdf/9pzu0YdTqQ@public.gmane.org>
2011-10-14 21:01 ` Stephen Warren
2011-10-14 21:01 ` Stephen Warren
2011-10-14 21:01 ` Stephen Warren
2011-10-14 21:28 ` Nicolas Pitre
2011-10-14 21:28 ` Nicolas Pitre
2011-10-14 21:28 ` Nicolas Pitre
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF173BE1A3A6-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-10-14 22:06 ` Rob Herring [this message]
2011-10-14 22:06 ` Rob Herring
2011-10-14 22:06 ` Rob Herring
2011-10-14 18:01 ` Nicolas Pitre
2011-10-14 18:01 ` Nicolas Pitre
2011-10-14 18:01 ` Nicolas Pitre
2011-10-14 19:20 ` Russell King - ARM Linux
2011-10-14 19:20 ` Russell King - ARM Linux
2011-10-14 19:20 ` Russell King - ARM Linux
[not found] ` <20111014192057.GT21648-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-10-14 20:14 ` Nicolas Pitre
2011-10-14 20:14 ` Nicolas Pitre
2011-10-14 20:14 ` Nicolas Pitre
2011-10-14 20:17 ` Russell King - ARM Linux
2011-10-14 20:17 ` Russell King - ARM Linux
[not found] ` <20111014201737.GW21648-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-10-14 20:31 ` Nicolas Pitre
2011-10-14 20:31 ` Nicolas Pitre
2011-10-14 20:31 ` Nicolas Pitre
2011-10-14 21:13 ` Russell King - ARM Linux
2011-10-14 21:13 ` Russell King - ARM Linux
[not found] ` <20111014211311.GY21648-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-10-14 22:26 ` Nicolas Pitre
2011-10-14 22:26 ` Nicolas Pitre
2011-10-14 22:26 ` Nicolas Pitre
2011-10-14 17:53 ` Nicolas Pitre
2011-10-14 17:53 ` Nicolas Pitre
2011-10-14 17:53 ` Nicolas Pitre
[not found] ` <alpine.LFD.2.02.1110141348270.17040-QuJgVwGFrdf/9pzu0YdTqQ@public.gmane.org>
2011-10-14 17:58 ` Olof Johansson
2011-10-14 17:58 ` Olof Johansson
2011-10-14 17:58 ` Olof Johansson
[not found] ` <CAOesGMhm_dD=9dyMWQRfH5AXH4AE42+4ckNwhnMUGaBJEHsumg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-10-14 18:00 ` Olof Johansson
2011-10-14 18:00 ` Olof Johansson
2011-10-14 18:00 ` Olof Johansson
2011-10-14 15:59 ` [PATCH] ARM: mutually exclude ZBOOT_ROM and AUTO_ZRELADDR Olof Johansson
2011-10-14 15:59 ` Olof Johansson
[not found] ` <1318607945-6807-1-git-send-email-olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>
2011-10-14 16:29 ` Arnd Bergmann
2011-10-14 16:29 ` Arnd Bergmann
2011-10-14 16:29 ` Arnd Bergmann
[not found] ` <201110141829.48634.arnd-r2nGTMty4D4@public.gmane.org>
2011-10-14 18:07 ` Nicolas Pitre
2011-10-14 18:07 ` Nicolas Pitre
2011-10-14 18:07 ` Nicolas Pitre
2011-10-14 18:04 ` Nicolas Pitre
2011-10-14 18:04 ` Nicolas Pitre
2011-10-14 18:04 ` Nicolas Pitre
-- strict thread matches above, loose matches on Subject: below --
2011-10-03 13:06 [PATCH v6 0/3] Add support for tegra2 based ventana board Peter De Schrijver
2011-10-03 13:06 ` [PATCH] arm/tegra: select AUTO_ZRELADDR by default Peter De Schrijver
2011-10-03 13:06 ` Peter De Schrijver
2011-10-03 13:06 ` Peter De Schrijver
2011-10-03 13:13 ` Peter De Schrijver
2011-10-03 13:13 ` Peter De Schrijver
[not found] ` <20111003131352.GX21166-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2011-10-03 16:22 ` Stephen Warren
2011-10-03 16:22 ` Stephen Warren
2011-10-03 16:22 ` Stephen Warren
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF173A2C6CAA-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-10-04 8:51 ` Peter De Schrijver
2011-10-04 8:51 ` Peter De Schrijver
2011-10-04 8:51 ` Peter De Schrijver
2011-09-22 13:27 Peter De Schrijver
[not found] ` <1316698045-23190-1-git-send-email-pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2011-09-22 16:26 ` Stephen Warren
2011-09-22 16:26 ` Stephen Warren
[not found] ` <74CDBE0F657A3D45AFBB94109FB122FF1739554CB9-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-09-22 17:58 ` Peter De Schrijver
2011-09-22 17:58 ` Peter De Schrijver
[not found] ` <20110922175800.GC21166-Rysk9IDjsxmJz7etNGeUX8VPkgjIgRvpAL8bYrjMMd8@public.gmane.org>
2011-09-22 18:06 ` Colin Cross
2011-09-22 18:06 ` Colin Cross
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=4E98B24A.2070408@gmail.com \
--to=robherring2-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=arnd-r2nGTMty4D4@public.gmane.org \
--cc=ccross-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org \
--cc=konkers-z5hGa2qSFaRBDgjK7y7TUQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=nico-vtqb6HGKxmzR7s880joybQ@public.gmane.org \
--cc=olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org \
--cc=pdeschrijver-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.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.