Re: [Qemu-devel] [PATCH v5 2/5] target/arm: handle M-profile semihosting at translate time

2019-09-13 Thread Alex Bennée
Peter Maydell writes: > On Fri, 13 Sep 2019 at 14:28, Alex Bennée wrote: >> >> >> Peter Maydell writes: >> > This is still disabling semihosting for the linux-user-mode >> > build for M-profile, isn't it ? >> >> Sure - as rth suggested. But m-profile doesn't run Linux so why would we >>

Re: [Qemu-devel] [PATCH v5 2/5] target/arm: handle M-profile semihosting at translate time

2019-09-13 Thread Peter Maydell
On Fri, 13 Sep 2019 at 14:31, Peter Maydell wrote: > > On Fri, 13 Sep 2019 at 14:28, Alex Bennée wrote: > > > > > > Peter Maydell writes: > > > This is still disabling semihosting for the linux-user-mode > > > build for M-profile, isn't it ? > > > > Sure - as rth suggested. But m-profile

Re: [Qemu-devel] [PATCH v5 2/5] target/arm: handle M-profile semihosting at translate time

2019-09-13 Thread Peter Maydell
On Fri, 13 Sep 2019 at 14:28, Alex Bennée wrote: > > > Peter Maydell writes: > > This is still disabling semihosting for the linux-user-mode > > build for M-profile, isn't it ? > > Sure - as rth suggested. But m-profile doesn't run Linux so why would we > support it in linux-user? (a) Linux

Re: [Qemu-devel] [PATCH v5 2/5] target/arm: handle M-profile semihosting at translate time

2019-09-13 Thread Alex Bennée
Peter Maydell writes: > On Wed, 11 Sep 2019 at 17:50, Alex Bennée wrote: >> >> We do this for other semihosting calls so we might as well do it for >> M-profile as well. >> >> Signed-off-by: Alex Bennée >> Reviewed-by: Richard Henderson >> >> --- >> v2 >> - update for change to

Re: [Qemu-devel] [PATCH v5 2/5] target/arm: handle M-profile semihosting at translate time

2019-09-13 Thread Peter Maydell
On Wed, 11 Sep 2019 at 17:50, Alex Bennée wrote: > > We do this for other semihosting calls so we might as well do it for > M-profile as well. > > Signed-off-by: Alex Bennée > Reviewed-by: Richard Henderson > > --- > v2 > - update for change to gen_exception_internal_insn API > v3 > -

[Qemu-devel] [PATCH v5 2/5] target/arm: handle M-profile semihosting at translate time

2019-09-11 Thread Alex Bennée
We do this for other semihosting calls so we might as well do it for M-profile as well. Signed-off-by: Alex Bennée Reviewed-by: Richard Henderson --- v2 - update for change to gen_exception_internal_insn API v3 - update for decode tree v4 - use !IS_USER --- target/arm/m_helper.c | 18