Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-18 Thread J. Mayer
On Wed, 2005-05-18 at 11:06 +0200, Herbert Poetzl wrote: > On Tue, May 17, 2005 at 11:10:52PM +0200, J. Mayer wrote: > > > > I will implement the new form, so it most crf transfers can be > > optimized. > > > > The latest PowerPC specification is to be found here: [...] > you sure about that? N

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-18 Thread Herbert Poetzl
On Tue, May 17, 2005 at 11:10:52PM +0200, J. Mayer wrote: > > I will implement the new form, so it most crf transfers can be > optimized. > > The latest PowerPC specification is to be found here: > >

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-18 Thread Pierre d'Herbemont
On 16 mai 05, at 09:54, Benjamin Herrenschmidt wrote: OK, I did put this in my working repository and I'll submit this to Fabrice. Please try to do this change to check if other bits need to be relax or not: Index: target-ppc/translate.c ==

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-17 Thread J. Mayer
On Mon, 2005-05-16 at 17:54 +1000, Benjamin Herrenschmidt wrote: > > OK, I did put this in my working repository and I'll submit this to > > Fabrice. > > Please try to do this change to check if other bits need to be relax or > > not: > > > > Index: target-ppc/translate.c > > =

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-17 Thread J. Mayer
On Mon, 2005-05-16 at 19:06 +0100, Piotras wrote: > You may also want to take a look at mtocrf (with "o" for "one") in the > document: > ftp://www6.software.ibm.com/software/developer/library/es-ppcbook1.zip > All right, there is a new (optional) form of this instruction. However, what is curr

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-17 Thread Benjamin Herrenschmidt
> OK, I did put this in my working repository and I'll submit this to > Fabrice. > Please try to do this change to check if other bits need to be relax or > not: > > Index: target-ppc/translate.c > === > RCS file: /cvsroot/qemu/qemu/

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-16 Thread J. Mayer
On Sat, 2005-05-14 at 21:03 +0200, Pierre d'Herbemont wrote: > On 14 mai 05, at 20:15, J. Mayer wrote: > > > Qemu assume all reserved fields are set to zero, not 1. > > > PowerPC specifications says: (section 8.1 of PEM) [...] > > In the case of mtcrf, the PowerPC specification says the bits 11,

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-14 Thread Pierre d'Herbemont
On 14 mai 05, at 20:15, J. Mayer wrote: Qemu assume all reserved fields are set to zero, not 1. PowerPC specifications says: (section 8.1 of PEM) "Some instructions fields are reserved or must contain a predefined value as shown in the individual instruction layouts. If a reserved field does not h

Re: [Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-14 Thread J. Mayer
On Sat, 2005-05-14 at 19:20 +0200, Pierre d'Herbemont wrote: > Hi, > > I have been playing with ppc-darwin-user. And I have to say that qemu- > ppc is too strict with ppc opcode validity: > On Mac OS X binary release called Tiger (or 10.4), the __bzero > function [1] contains a mtcrf, which has

[Qemu-devel] [Patch] target-ppc mtcrf instruction not recognized

2005-05-14 Thread Pierre d'Herbemont
Hi, I have been playing with ppc-darwin-user. And I have to say that qemu- ppc is too strict with ppc opcode validity: On Mac OS X binary release called Tiger (or 10.4), the __bzero function [1] contains a mtcrf, which has been translated to 0x7c901120, by Apple's as. Current qemu rejects it,