[v1] target/arm: Remove access_el3_aa32ns()

[PATCH v1 0/1] target/arm: Remove access_el3_aa32ns()

Edgar E. Iglesias posted 1 patch 4 years ago

Diff against v2
Download series mbox

Test docker-quick@centos7 passed

Test docker-mingw@fedora passed

Test checkpatch passed

Test asan passed

Test FreeBSD passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200428160350.10030-1-edgar.iglesias@gmail.com

Maintainers: Peter Maydell <peter.maydell@linaro.org>

There is a newer version of this series

target/arm/helper.c | 34 ++++++++++------------------------
1 file changed, 10 insertions(+), 24 deletions(-)

Expand all Fold all

[PATCH v1 0/1] target/arm: Remove access_el3_aa32ns()

Posted by Edgar E. Iglesias 4 years ago

From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>

Hi,

Laurent reported hitting the assert in access_el3_aa32ns()
when accessing 32-bit versions of some of the virtualization
regs when EL3 is 64-bit.

I think we got this wrong back then and it seems to me like
we should remove direct usage of access_el3_aa32ns() and
always call access_el3_aa32ns_aa64_any() to handle both
the aa32-only cases and the mixed aa32/aa64.

Cheers,
Edgar

Edgar E. Iglesias (1):
  target/arm: Drop access_el3_aa32ns()

 target/arm/helper.c | 34 ++++++++++------------------------
 1 file changed, 10 insertions(+), 24 deletions(-)

-- 
2.20.1