Re: [Qemu-devel] [PATCH] target-arm: modifying pc in tcg code for load/store multiple

2015-03-10 Thread Peter Maydell
On 19 February 2015 at 16:04, Ildar Isaev wrote: > A bit clumsy, but something like that. Qemu never gets to the code past stmda. > > -bash-4.1$ cat add.s > > > .text > mov r0, #5 > mov r1, #4 > add r2, r1, r0 > stmda sp, {r1, r2, r5, sp, lr, pc}^ >

Re: [Qemu-devel] [PATCH] target-arm: modifying pc in tcg code for load/store multiple

2015-02-19 Thread Ildar Isaev
> On 19 February 2015 at 21:26, wrote: > > From: Ildar Isaev > > > > pc wasn't modified in tcg code for load/store multiple, > > causing translation block to be executed in infinite loop forever > > > > Signed-off-by: Ildar Isaev > > It would be helpful if you gave an example of guest > cod

Re: [Qemu-devel] [PATCH] target-arm: modifying pc in tcg code for load/store multiple

2015-02-19 Thread Peter Maydell
On 19 February 2015 at 21:26, wrote: > From: Ildar Isaev > > pc wasn't modified in tcg code for load/store multiple, > causing translation block to be executed in infinite loop forever > > Signed-off-by: Ildar Isaev It would be helpful if you gave an example of guest code which we mishandle. D

[Qemu-devel] [PATCH] target-arm: modifying pc in tcg code for load/store multiple

2015-02-19 Thread ild
From: Ildar Isaev pc wasn't modified in tcg code for load/store multiple, causing translation block to be executed in infinite loop forever Signed-off-by: Ildar Isaev --- target-arm/translate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target-arm/translate.c b/target-