qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Alexander Graf <agraf@suse.de>
Cc: qemu-devel <qemu-devel@nongnu.org>,
	"Fabien Chouteau" <chouteau@adacore.com>,
	"Hervé Poussineau" <hpoussin@reactos.org>,
	qemu-ppc@nongnu.org, "Andreas Färber" <afaerber@suse.de>,
	"Artyom Tarasenko" <atar4qemu@gmail.com>
Subject: Re: [Qemu-devel] [PATCH 3/3] PPC PReP: can run without bios image
Date: Thu, 4 Apr 2013 17:41:53 +0100	[thread overview]
Message-ID: <CAFEAcA-WZ35PzoJPjpFpbcyYwdcz=jhg0RMU1kyv6BWAjHVppA@mail.gmail.com> (raw)
In-Reply-To: <1E98DD2E-DE7A-4E7F-865B-A77549839B70@suse.de>

On 4 April 2013 17:34, Alexander Graf <agraf@suse.de> wrote:
> On 04.04.2013, at 18:30, Peter Maydell wrote:
>> * -kernel (if it means anything at all) has to mean "boot in
>> the way a Linux kernel expects and defines its boot protocol"
>
> This is what -kernel does. If more OSs than Linux end up happy with
> that interface, great. Examples for that are multiboot kernels on
> x86. But in general, kernels want to talk to firmware.

Agreed.

>> * "just load a binary and run it" needs to be some other option,
>> because that's not the same thing
>
> This is what -bios does. Maybe we should add an alias and call it
> "-firmware", but the idea stays the same. That's the initial blob
> executed when a VM gets up.

No, in general this isn't what -bios does. Usually -bios means
"take a blob and put it wherever this board expects to have
a ROM or flash firmware image". So on highbank it goes in the
sysram at 0xfff88000, on shix it's 0x4000 bytes loaded at address 0,
on mips_malta it's an image to load into a flash device, and
so on. And on some boards it does nothing at all, because we
ignore the parameter.

What is being asked for (and what ARM's -kernel does at the
moment if you pass it an ELF image) is "just load the ELF
image where the ELF image says to put it, and jump to the
ELF image's entrypoint".

-- PMM

  reply	other threads:[~2013-04-04 16:42 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-03 16:40 [Qemu-devel] [PATCH 0/3] PPC PReP: Use ElF kernel on PReP Fabien Chouteau
2013-04-03 16:40 ` [Qemu-devel] [PATCH 1/3] PPC PReP: Load ELF kernel Fabien Chouteau
2013-04-03 16:40 ` [Qemu-devel] [PATCH 2/3] PPC PReP: Use kernel entry to set nip at reset Fabien Chouteau
2013-04-03 16:40 ` [Qemu-devel] [PATCH 3/3] PPC PReP: can run without bios image Fabien Chouteau
2013-04-03 16:59   ` Alexander Graf
2013-04-04  8:37     ` Fabien Chouteau
2013-04-04  9:26       ` Alexander Graf
2013-04-04 16:19         ` Fabien Chouteau
2013-04-04  9:46       ` Artyom Tarasenko
2013-04-04  9:50         ` Alexander Graf
2013-04-04  9:57           ` Artyom Tarasenko
2013-04-04 11:53             ` Andreas Färber
2013-04-04 11:59               ` Alexander Graf
2013-04-05 23:00                 ` Scott Wood
2013-04-06  9:01                   ` Alexander Graf
2013-04-08 17:52                     ` Scott Wood
2013-07-04  8:35                       ` Julio Guerra
2013-07-10  9:16                         ` Julio Guerra
2013-04-06  9:07                   ` Alexander Graf
2013-04-06 11:27                     ` Peter Maydell
2013-04-06 11:38                       ` Alexander Graf
2013-04-06 13:08                         ` Peter Maydell
2013-04-06 20:07                         ` [Qemu-devel] [Qemu-ppc] " Edgar E. Iglesias
2013-04-06 20:12                           ` Alexander Graf
2013-04-07  0:52                             ` Edgar E. Iglesias
2013-04-04 12:43               ` [Qemu-devel] " Peter Maydell
2013-04-04 16:17                 ` Fabien Chouteau
2013-04-04 16:20                   ` Peter Maydell
2013-04-04 16:26                     ` Artyom Tarasenko
2013-04-04 16:30                       ` Peter Maydell
2013-04-04 16:34                         ` Alexander Graf
2013-04-04 16:41                           ` Peter Maydell [this message]
2013-04-04 16:51                             ` Alexander Graf
2013-04-04 16:52                               ` Peter Maydell
2013-04-04 22:32                                 ` Alexander Graf
2013-04-04 22:35                                   ` Peter Maydell
2013-04-04 16:46                         ` Artyom Tarasenko
2013-04-04 16:51                           ` Peter Maydell
2013-04-04 17:08                             ` Artyom Tarasenko
2013-04-04 17:22                       ` Andreas Färber
2013-04-05  9:19                         ` Fabien Chouteau
2013-04-05  9:30                           ` Peter Maydell
2013-04-04 17:05                   ` Andreas Färber
2013-04-05  2:32             ` Rob Landley
2013-04-04 11:16       ` Andreas Färber
2013-04-04 16:18         ` Fabien Chouteau
2013-04-03 16:47 ` [Qemu-devel] [PATCH 0/3] PPC PReP: Use ElF kernel on PReP Alexander Graf
2013-04-03 17:32   ` Andreas Färber
2013-04-04  8:17     ` Fabien Chouteau

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='CAFEAcA-WZ35PzoJPjpFpbcyYwdcz=jhg0RMU1kyv6BWAjHVppA@mail.gmail.com' \
    --to=peter.maydell@linaro.org \
    --cc=afaerber@suse.de \
    --cc=agraf@suse.de \
    --cc=atar4qemu@gmail.com \
    --cc=chouteau@adacore.com \
    --cc=hpoussin@reactos.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.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).