Re: [PATCH v1 2/2] target/riscv: Set mtval and stval support

[Bin Meng]

On Sat, Sep 4, 2021 at 1:06 AM Richard Henderson
<richard.hender...@linaro.org> wrote:
>
> On 9/3/21 1:23 AM, Alistair Francis wrote:
> > +    DEFINE_PROP_BOOL("mtval_inst", RISCVCPU, cfg.mtval_inst, true),
>
> Dash not underscore for the prop name, I think.
>

But we also have "priv_spec" :)

The name "mtval_inst" sounds like only for M-mode. Maybe omitting 'm',
and just "tval"?

Regards,
Bin