qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [5949] target-ppc: memory load/store rework


From: Aurelien Jarno
Subject: Re: [Qemu-devel] [5949] target-ppc: memory load/store rework
Date: Sat, 13 Dec 2008 12:17:32 +0100
User-agent: Mutt/1.5.18 (2008-05-17)

On Sat, Dec 13, 2008 at 09:25:54AM +0200, Blue Swirl wrote:
> On 12/13/08, Stuart Brady <address@hidden> wrote:
> > On Mon, Dec 08, 2008 at 06:11:22PM +0000, Aurelien Jarno wrote:
> >  > - Don't swap values twice for bit-reverse load/store functions
> >  >   in little endian mode.
> >
> >  If I've read this correctly, it seems to me that we would still swap
> >  twice on little endian hosts when when emulating little endian mode or
> >  performing bit-reverse loads/stores.  Bit-reverse loads/stores under
> >  little endian emulation on a little endian host would have actually
> >  resulted in *three* byteswaps!
> >
> >  I'm not sure what the performance impact of this would be, but it's
> >  something that I've been wondering about while working on PA-RISC
> >  emulation, which will also need to provide a little endian mode.
> 
> Jocelyn sent a patch to address this. There were some (small) problems
> with it, so please read the full thread:
> http://lists.gnu.org/archive/html/qemu-devel/2007-10/msg00289.html
> 

That looks like a good start if someone wants to work again on that. The
switch to TCG probably implies some more changes.

-- 
  .''`.  Aurelien Jarno             | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   address@hidden         | address@hidden
   `-    people.debian.org/~aurel32 | www.aurel32.net




reply via email to

[Prev in Thread] Current Thread [Next in Thread]