David Gibson <da...@gibson.dropbear.id.au> writes: > On Mon, Jan 10, 2022 at 03:15:46PM -0300, Fabiano Rosas wrote: >> There's no sc 1. >> >> We also only used env->nip because of the vhyp code, so change to >> 'vector' now. > > I don't think this is right. The point with the env->nip change is > changing the PC as it appeared *before* saving it to SRR0, so that > we'll eventually return to the right place. 'vector' is the address > for the interrupt vector itself.
Ugh, In the RFC version I introduced a 'nip' that holds env->nip. So it would be correct to use 'nip' instead of 'env->nip' here. I knew 'vector' was looking off... Thanks for catching that.
