Message ID | alpine.DEB.1.10.1411031848030.2881@tp.orcam.me.uk |
---|---|
State | New |
Headers | show |
On 03/11/2014 18:51, Maciej W. Rozycki wrote: > CP1.FIR is read-only in hardware so gdbstub must respect it. We already > respect it for CTC1 instructions, so do it here too. > > Signed-off-by: Maciej W. Rozycki <macro@codesourcery.com> > --- > Not much to say about it here. Please apply. > > Maciej > > qemu-mips-fir.diff > Index: qemu-git-trunk/target-mips/gdbstub.c > =================================================================== > --- qemu-git-trunk.orig/target-mips/gdbstub.c 2014-11-02 17:51:33.458968203 +0000 > +++ qemu-git-trunk/target-mips/gdbstub.c 2014-11-02 17:51:35.958924223 +0000 > @@ -112,7 +112,7 @@ int mips_cpu_gdb_write_register(CPUState > RESTORE_ROUNDING_MODE; > break; > case 71: > - env->active_fpu.fcr0 = tmp; > + /* FIR is read-only. Ignore writes. */ > break; > } > return sizeof(target_ulong); > Reviewed-by: Leon Alrae <leon.alrae@imgtec.com>
Index: qemu-git-trunk/target-mips/gdbstub.c =================================================================== --- qemu-git-trunk.orig/target-mips/gdbstub.c 2014-11-02 17:51:33.458968203 +0000 +++ qemu-git-trunk/target-mips/gdbstub.c 2014-11-02 17:51:35.958924223 +0000 @@ -112,7 +112,7 @@ int mips_cpu_gdb_write_register(CPUState RESTORE_ROUNDING_MODE; break; case 71: - env->active_fpu.fcr0 = tmp; + /* FIR is read-only. Ignore writes. */ break; } return sizeof(target_ulong);
CP1.FIR is read-only in hardware so gdbstub must respect it. We already respect it for CTC1 instructions, so do it here too. Signed-off-by: Maciej W. Rozycki <macro@codesourcery.com> --- Not much to say about it here. Please apply. Maciej qemu-mips-fir.diff