qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).