On 7/14/22 18:52, Peter Maydell wrote:
The regime_tcr() function returns a pointer to a struct TCR
corresponding to the TCR controlling a translation regime.  The
struct TCR has the raw value of the register, plus two fields mask
and base_mask which are used as a small optimization in the case of
32-bit short-descriptor lookups.  Almost all callers of regime_tcr()
only want the raw register value.  Define and use a new
regime_tcr_value() function which returns only the raw 64-bit
register value.

This is a preliminary to removing the 32-bit short descriptor
optimization -- it only saves a handful of bit operations, which is
tiny compared to the overhead of doing a page table walk at all, and
the TCR struct is awkward and makes fixing
https://gitlab.com/qemu-project/qemu/-/issues/1103  unnecessarily
difficult.

Signed-off-by: Peter Maydell<peter.mayd...@linaro.org>
---
  target/arm/internals.h  | 6 ++++++
  target/arm/helper.c     | 6 +++---
  target/arm/ptw.c        | 8 ++++----
  target/arm/tlb_helper.c | 2 +-
  4 files changed, 14 insertions(+), 8 deletions(-)

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

r~

Reply via email to