From: "Rodrigo Vivi" <rodrigo.vivi@gmail.com>
To: Paul Brook <paul@codesourcery.com>
Cc: Lauro Ramos Venancio <lauro.venancio@indt.org.br>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] qemu-arm: wrong execution of post-indexed loads when Rm and Rd are the same register
Date: Thu, 15 Mar 2007 18:19:02 -0300 [thread overview]
Message-ID: <bdb0025f0703151419h402585ay764f02328e415f13@mail.gmail.com> (raw)
In-Reply-To: <200703152110.39814.paul@codesourcery.com>
On 3/15/07, Paul Brook <paul@codesourcery.com> wrote:
> > > This is still wrong.
> >
> > So, is this a known bug?
>
> Still wrong implies it's a bug, and your patch does not fix it properly.
I know that...
I was not clear.. sorry...
what I mean is: do you agree that there was a bug in these instructions?
>
> > > The writeback must happen after the load.
> >
> > We code like this because
> > - we didn't find this restriction in arm reference manual
>
> It's the Abort model section you mention below.
>
> > - the LLVM uses this instruction expecting a result like this
>
> The compiler knows nothing about the abort behavior. The difference is only
> visible if the load faults.
>
> > - That was the result that we got running these instructions in an OMAP1710
>
> I suggest you check again. I'm fairly sure the arm926 implements the Base
> Restored abort model.
Actually we did not test the abort model...
So, Base Restored abort model is the model that qemu implements, isn't it?
then we will try to use that and recode the patch...
thanks for your help
>
> Paul
>
vivijim
next prev parent reply other threads:[~2007-03-15 21:20 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-15 19:35 [Qemu-devel] qemu-arm: wrong execution of post-indexed loads when Rm and Rd are the same register Lauro Ramos Venancio
2007-03-15 19:43 ` [Qemu-devel] " Lauro Ramos Venancio
2007-03-15 20:03 ` [Qemu-devel] " Paul Brook
2007-03-15 20:32 ` Rodrigo Vivi
2007-03-15 21:10 ` Paul Brook
2007-03-15 21:19 ` Rodrigo Vivi [this message]
2007-03-15 21:55 ` Laurent Desnogues
2007-03-15 22:04 ` Paul Brook
2007-03-16 20:42 ` Lauro Ramos Venancio
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=bdb0025f0703151419h402585ay764f02328e415f13@mail.gmail.com \
--to=rodrigo.vivi@gmail.com \
--cc=lauro.venancio@indt.org.br \
--cc=paul@codesourcery.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).