[Qemu-devel] [SPARC][PATCH] Generate cp_disabled trap for Cpop instructions

[Aurelien Jarno]

Hi all,

Both of the SPARC CPU currently emulated in QEMU do not have a 
coprocessor. In such case executing a Cpop1 or a Cpop2 instruction 
should generate a cp_disabled trap, but the current implementation
generates a illegal_instruction instead.

The patch below fixes that.

Bye,
Aurelien

Index: target-sparc/cpu.h
===================================================================
RCS file: /sources/qemu/qemu/target-sparc/cpu.h,v
retrieving revision 1.30
diff -u -d -p -r1.30 cpu.h
--- target-sparc/cpu.h  25 Mar 2007 07:55:52 -0000      1.30
+++ target-sparc/cpu.h  30 Mar 2007 17:09:46 -0000
@@ -40,6 +40,7 @@
 #define TT_TOVF     0x0a
 #define TT_EXTINT   0x10
 #define TT_DIV_ZERO 0x2a
+#define TT_NCP_INSN 0x24
 #define TT_TRAP     0x80
 #else
 #define TT_TFAULT   0x08
Index: target-sparc/translate.c
===================================================================
RCS file: /sources/qemu/qemu/target-sparc/translate.c,v
retrieving revision 1.38
diff -u -d -p -r1.38 translate.c
--- target-sparc/translate.c    25 Mar 2007 07:55:52 -0000      1.38
+++ target-sparc/translate.c    30 Mar 2007 17:09:47 -0000
@@ -1736,7 +1739,7 @@ static void disas_sparc_insn(DisasContex
                    gen_op_sra();
                gen_movl_T0_reg(rd);
 #endif
-           } else if (xop < 0x38) {
+           } else if (xop < 0x36) {
                 rs1 = GET_FIELD(insn, 13, 17);
                gen_movl_reg_T0(rs1);
                if (IS_IMM) {   /* immediate */
@@ -2142,6 +2145,10 @@ static void disas_sparc_insn(DisasContex
                        goto illegal_insn;
                    }
                }
+           } else if (xop == 0x36 || xop == 0x37) { /* CPop1 & CPop2 */
+               save_state(dc);
+               gen_op_exception(TT_NCP_INSN);
+               dc->is_br = 1;
 #ifdef TARGET_SPARC64
            } else if (xop == 0x39) { /* V9 return */
                 rs1 = GET_FIELD(insn, 13, 17);

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