linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Paul Mundt <lethal@linux-sh.org>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH] sh: include empty zero page in zImage
Date: Thu, 03 Dec 2009 21:09:20 +0000	[thread overview]
Message-ID: <20091203210920.GD11047@linux-sh.org> (raw)
In-Reply-To: <20091203122619.9975.58412.sendpatchset@rxone.opensource.se>

On Thu, Dec 03, 2009 at 09:22:04AM -0500, Magnus Damm wrote:
> Hi Stuart,
> 
> On Thu, Dec 3, 2009 at 8:48 AM, Stuart MENEFY <stuart.menefy@st.com> wrote:
> > If you do this won't you overwrite any parameters from the boot loader?
> 
> True, so we probably want to do this in a different way.
> 
> I don't want to break anything of course, but I'm not sure if u-boot
> supports passing parameters to zImages. It's not a u-boot centric
> world though.
> 
> I remember adding command line passing code to kexec-tools for either
> vmlinux or zImage, but I doubt that both image types are properly
> supported in kexec today.
> 
> Any ideas? How are things for ST? I need to fix up romImage somehow at
> least. Maybe zImage should be left alone as-is and work should be put
> into fixing the boot loaders.
> 
Yes, it is the boot loader that is broken, as it doesen't comply with the
boot ABI. We will not be applying any patches to the kernel to work
around boot loader laziness. The romImage case is a special one, similar
to XIP, and that's something that the current argument passing scheme
wasn't intended for. On the other hand, you can easily do what all of the
other boot loaders do and wrap the zImage while inserting the argument
page immediately following it.

I'm quite frankly amazed at the number of patches that have shown up to
work around this "problem" in multiple ways. It's called an ABI for a
reason folks, if you don't want to support it properly, don't be
surprised when things blow up.

      parent reply	other threads:[~2009-12-03 21:09 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-03 12:26 [PATCH] sh: include empty zero page in zImage Magnus Damm
2009-12-03 13:48 ` Stuart MENEFY
2009-12-03 14:22 ` Magnus Damm
2009-12-03 21:09 ` Paul Mundt [this message]

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=20091203210920.GD11047@linux-sh.org \
    --to=lethal@linux-sh.org \
    --cc=linux-sh@vger.kernel.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).