Re: [Qemu-devel] [PATCH] target-arm: Handle 'smc' as an undefined instruction

2010-11-14 Thread Peter Maydell
On 4 November 2010 22:21, Adam Lackorzynski wrote: > > Refine check on bkpt so that smc and undefined instruction encodings are > handled as an undefined instruction and trap. > > Signed-off-by: Adam Lackorzynski This patch looks good to me. I note that there's actually an implementation of SMC

Re: [Qemu-devel] [PATCH] target-arm: Handle 'smc' as an undefined instruction

2010-09-02 Thread Adam Lackorzynski
On Thu Sep 02, 2010 at 23:14:23 +0100, Peter Maydell wrote: > On Thu, Sep 02, 2010 at 11:40:50PM +0200, Adam Lackorzynski wrote: > > +case 7: > > +/* SMC? */ > > +if ((insn & 0xfff0) == 0xe1600070) { > > +goto illegal_op; > > +} > > +

Re: [Qemu-devel] [PATCH] target-arm: Handle 'smc' as an undefined instruction

2010-09-02 Thread Peter Maydell
On Thu, Sep 02, 2010 at 11:40:50PM +0200, Adam Lackorzynski wrote: > +case 7: > +/* SMC? */ > +if ((insn & 0xfff0) == 0xe1600070) { > +goto illegal_op; > +} > +/* bkpt */ This doesn't look right to me. SMC in the ARM encod

Re: [Qemu-devel] [PATCH] target-arm: Handle 'smc' as an undefined instruction

2010-09-02 Thread Stefan Weil
Am 02.09.2010 22:27, schrieb Adam Lackorzynski: Handle smc as an undefined instruction instead of having it wrongly interpreted as some other one. Signed-off-by: Adam Lackorzynski --- target-arm/translate.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/target-a