On 1/11/22 11:10, Philippe Mathieu-Daudé wrote:
On 1/11/22 00:14, Philippe Mathieu-Daudé wrote:
On 25/10/22 18:39, Peter Maydell wrote:
From: Richard Henderson <richard.hender...@linaro.org>

Hoist the computation of the mmu_idx for the ptw up to
get_phys_addr_with_struct and get_phys_addr_twostage.
This removes the duplicate check for stage2 disabled
from the middle of the walk, performing it only once.

Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
Reviewed-by: Alex Bennée <alex.ben...@linaro.org>
Tested-by: Alex Bennée <alex.ben...@linaro.org>
Message-id: 20221024051851.3074715-3-richard.hender...@linaro.org
Signed-off-by: Peter Maydell <peter.mayd...@linaro.org>
---
  target/arm/ptw.c | 71 ++++++++++++++++++++++++++++++++++++------------
  1 file changed, 54 insertions(+), 17 deletions(-)

Since this commit I can not boot Trusted Firmware on the SBSA-ref machine.

Do we need to set in_ptw_idx in get_phys_addr_with_secure()?

I opened https://gitlab.com/qemu-project/qemu/-/issues/1293 to track.

Reply via email to