[Qemu-devel] qemu/target-ppc helper.c op.c op_helper.c op_he...

[Qemu-devel] qemu/target-ppc helper.c op.c op_helper.c op_he...

[Jocelyn Mayer]

CVSROOT:	/sources/qemu
Module name:	qemu
Changes by:	Jocelyn Mayer <j_mayer>	07/09/19 05:44:04

Modified files:
	target-ppc     : helper.c op.c op_helper.c op_helper.h 
	                 translate.c 

Log message:
	Code provision for PowerPC BookE MMU model support.
	Better MSR flags initialisation.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/helper.c?cvsroot=qemu&r1=1.53&r2=1.54
http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op.c?cvsroot=qemu&r1=1.40&r2=1.41
http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op_helper.c?cvsroot=qemu&r1=1.37&r2=1.38
http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op_helper.h?cvsroot=qemu&r1=1.11&r2=1.12
http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/translate.c?cvsroot=qemu&r1=1.64&r2=1.65

[Qemu-devel] Re: qemu/target-ppc helper.c op.c op_helper.c op_he...

[Hollis Blanchard]

On Wed, 19 Sep 2007 05:44:04 +0000, Jocelyn Mayer wrote:

> CVSROOT:	/sources/qemu
> Module name:	qemu
> Changes by:	Jocelyn Mayer <j_mayer>	07/09/19 05:44:04
> 
> Modified files:
> 	target-ppc     : helper.c op.c op_helper.c op_helper.h 
> 	                 translate.c 
> 
> Log message:
> 	Code provision for PowerPC BookE MMU model support.
> 	Better MSR flags initialisation.
> 
> CVSWeb URLs:
> http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/helper.c?cvsroot=qemu&r1=1.53&r2=1.54
> http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op.c?cvsroot=qemu&r1=1.40&r2=1.41
> http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op_helper.c?cvsroot=qemu&r1=1.37&r2=1.38
> http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op_helper.h?cvsroot=qemu&r1=1.11&r2=1.12
> http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/translate.c?cvsroot=qemu&r1=1.64&r2=1.65

tlbwe and tlbre do not use PID; they use MMUCR[STID].

Also, IBM and Freescale Book E architectures are very different here, so I
would suggest naming these functions something more specific than
e.g. "do_booke_tlbwe". You're really implementing IBM's 440
architecture here.

-- 
Hollis Blanchard
IBM Linux Technology Center

Re: [Qemu-devel] Re: qemu/target-ppc helper.c op.c op_helper.c op_he...

[J. Mayer]

On Wed, 2007-09-19 at 16:35 +0000, Hollis Blanchard wrote:
> On Wed, 19 Sep 2007 05:44:04 +0000, Jocelyn Mayer wrote:
> 
> > CVSROOT:	/sources/qemu
> > Module name:	qemu
> > Changes by:	Jocelyn Mayer <j_mayer>	07/09/19 05:44:04
> > 
> > Modified files:
> > 	target-ppc     : helper.c op.c op_helper.c op_helper.h 
> > 	                 translate.c 
> > 
> > Log message:
> > 	Code provision for PowerPC BookE MMU model support.
> > 	Better MSR flags initialisation.
> > 
> > CVSWeb URLs:
> > http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/helper.c?cvsroot=qemu&r1=1.53&r2=1.54
> > http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op.c?cvsroot=qemu&r1=1.40&r2=1.41
> > http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op_helper.c?cvsroot=qemu&r1=1.37&r2=1.38
> > http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/op_helper.h?cvsroot=qemu&r1=1.11&r2=1.12
> > http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/translate.c?cvsroot=qemu&r1=1.64&r2=1.65
> 
> tlbwe and tlbre do not use PID; they use MMUCR[STID].
> 
> Also, IBM and Freescale Book E architectures are very different here, so I
> would suggest naming these functions something more specific than
> e.g. "do_booke_tlbwe". You're really implementing IBM's 440
> architecture here.

Yes, you're right, this is incorrect for most (or all) cases. I did not
notice what the Power specification says that tlbre and tlbwe are
implementation dependent. I just compared IBM 440 and Freescale e500 TLB
models and, you're absolutelly right, they actually use different
implementation specific registers.

Thanks for reporting this !
I will start adding big warnings in my code until I can recode it in a
better way ;-)

-- 
J. Mayer <l_indien@magic.fr>
Never organized