From: "Piotr Król" <pietrushnic@gmail.com>
To: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] vexpress-a9: coreboot is unable to push any data on stack
Date: Fri, 15 Aug 2014 20:02:24 +0200 [thread overview]
Message-ID: <20140815180221.GA25591@echad> (raw)
In-Reply-To: <CAFEAcA9-KPQFCDpsiWozmgikR9eib3UEoxmTH0N0w4PLj5kzZQ@mail.gmail.com>
On Fri, Aug 15, 2014 at 05:10:04PM +0100, Peter Maydell wrote:
> For your stack issues, it looks like your code is trying to
> use the area which is the flash as the stack. Since flash
> isn't writeable, we ignore the writes and it's not very
> useful for stack. It looks like your code is assuming that
> the low memory is RAM, not flash -- so how does your
> code work on real hardware? Do you try to use the
> software controllable remapping to copy from the flash
> into RAM before using the stack, or something else?
Peter thanks for your reply. I'm not coreboot developer so cannot
advocate for their decision. Bootblock that I sent is from build
targeted on qemu and developer who initially wrote the code suggest
using it with '-kernel' parameter. I would like to fix this code
according to correct memory map.
Comment in hw/arm/vexpress.c say that, as you wrote below, Versatile
Express got two possible memory maps. Can you point me to exact
documentation that you use as reference for vexpress implementation ?
>
> In terms of where we go from here, we have two
> choices:
> (1) leave address 0 as RAM, not flash; this means
> legacy guest binaries that work only on QEMU and
> not on real hardware will still work, but the -bios
> option won't be of much use. (This is more or less
> reverting to the 2.0 situation.)
> (2) bring it in to line with vexpress-a15 (which is
> effectively how 2.1 shipped), so 0 is always flash
> and never RAM. This is consistent but (as you've
> found) binaries assuming that 0 is a RAM alias
> will stop working.
>
Assuming option (2) contain strategy for future releases it should be
priority. I will try to fix coreboot binary according to this advice.
One more time thanks for explanation.
Thanks,
Piotr Król
next prev parent reply other threads:[~2014-08-15 18:02 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-15 10:07 [Qemu-devel] vexpress-a9: coreboot is unable to push any data on stack Piotr Król
2014-08-15 16:10 ` Peter Maydell
2014-08-15 18:02 ` Piotr Król [this message]
2014-08-15 22:54 ` Peter Maydell
2014-08-16 9:50 ` Piotr Król
2014-08-16 11:19 ` Peter Maydell
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=20140815180221.GA25591@echad \
--to=pietrushnic@gmail.com \
--cc=qemu-devel@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).