Re: [Qemu-devel] [patch] Bugfix for 64-bit targets

[Fabrice Bellard <fabrice@bellard.org>]

Hi,

I am afraid to say that sizeof(int) = 4 is hardcoded in QEMU. QEMU 
assumes that sizeof(long) = sizeof(void *) = 4 or 8 bytes depending on 
the host.

Fabrice.

Paul Brook wrote:
> The attached patch fixes a bug in arm target emulation on hosts where 
> sizeof(int) > 4. On these targets the offset wouldn't be properly sign 
> extended.
> 
> Paul
> 
> 
> ------------------------------------------------------------------------
> 
> Index: translate.c
> ===================================================================
> RCS file: /cvsroot/qemu/qemu/target-arm/translate.c,v
> retrieving revision 1.10
> diff -u -p -r1.10 translate.c
> --- translate.c	22 Jun 2004 10:55:49 -0000	1.10
> +++ translate.c	11 Aug 2004 00:17:59 -0000
> @@ -690,15 +867,15 @@ static void disas_arm_insn(DisasContext 
>          case 0xa:
>          case 0xb:
>              {
> -                int offset;
> +                int32_t offset;
>                  
>                  /* branch (and link) */
> -                val = (int)s->pc;
> +                val = (int32_t)s->pc;
>                  if (insn & (1 << 24)) {
>                      gen_op_movl_T0_im(val);
>                      gen_op_movl_reg_TN[0][14]();
>                  }
> -                offset = (((int)insn << 8) >> 8);
> +                offset = (((int32_t)insn << 8) >> 8);
>                  val += (offset << 2) + 4;
>                  gen_op_jmp((long)s->tb, val);
>                  s->is_jmp = DISAS_TB_JUMP;
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel