On Fri, 21 Oct 2022 at 10:27, Philippe Mathieu-Daudé <phi...@linaro.org> wrote:
>
> On 21/10/22 00:35, Richard Henderson wrote:
> > Leave the upper and lower attributes in the place they originate
> > from in the descriptor. Shifting them around is confusing, since
> > one cannot read the bit numbers out of the manual. Also, new
> > attributes have been added which would alter the shifts.
> >
> > Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
> > Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
> > ---
> > target/arm/ptw.c | 31 +++++++++++++++----------------
> > 1 file changed, 15 insertions(+), 16 deletions(-)
>
> > if (regime_is_stage2(mmu_idx)) {
> > ns = mmu_idx == ARMMMUIdx_Stage2;
> > - xn = extract32(attrs, 11, 2);
> > + xn = extract64(attrs, 54, 2);
>
> 54 -> 53?
Yes. I spotted that in review of v3 but I guess we lost the fix
along the way somewhere.
-- PMM
