On Fri, Feb 8, 2019 at 11:09 AM Jim Wilson <j...@sifive.com> wrote:
>
> On Fri, Feb 8, 2019 at 10:17 AM Alistair Francis <alistai...@gmail.com> wrote:
> > Can we just write a wrapper function then that sets and unsets the variable?
> > Something like this:
> >
> > riscv_csrrw_debug(...) {
> > #if !defined(CONFIG_USER_ONLY)
> > env->debugger = true;
> > #endif
> > result = riscv_csrrw(env, ...);
> > #if !defined(CONFIG_USER_ONLY)
> > env->debugger = false;
> > #endif
> > }
>
> Yes, that would work. Do you want me to resubmit a fixed part 5 patch?
It's probably best to send out a new version of the whole series.
Include all the Reviewed-by tags and then that should be the last
version and Palmer can apply that series directly.
Alistair
>
> Jim
