On Fri, Feb 8, 2019 at 11:09 AM Jim Wilson <address@hidden> wrote:
On Fri, Feb 8, 2019 at 10:17 AM Alistair Francis <address@hidden> 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.