[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 7/7] target/riscv: Honour -semihosting-config userspace=on an
Re: [PATCH 7/7] target/riscv: Honour -semihosting-config userspace=on and enable=on
Thu, 18 Aug 2022 14:57:48 +0100
On Thu, 18 Aug 2022 at 05:20, Alistair Francis <firstname.lastname@example.org> wrote:
> On Tue, Aug 16, 2022 at 5:11 AM Peter Maydell <email@example.com>
> > The riscv target incorrectly enabled semihosting always, whether the
> > user asked for it or not. Call semihosting_enabled() passing the
> > correct value to the is_userspace argument, which fixes this and also
> > handles the userspace=on argument.
> > Note that this is a behaviour change: we used to default to
> > semihosting being enabled, and now the user must pass
> > "-semihosting-config enable=on" if they want it.
> > Signed-off-by: Peter Maydell <firstname.lastname@example.org>
> I agree with Richard that a check in translate would be better, but
> this is also an improvement on the broken implementation we have now
Do you have an opinion on whether there are likely to be many
users who are using riscv semihosting without explicitly enabling it
on the command line ?
Re: [PATCH 0/7] Allow semihosting from user mode, Richard Henderson, 2022/08/15
- Re: [PATCH 4/7] target/mips: Honour -semihosting-config userspace=on, (continued)