On Wed, Oct 05, 2022 at 08:44:07PM -0700, Richard Henderson wrote:
Masking after the fact in s390x_tr_init_disas_context
provides incorrect information to tb_lookup.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
target/s390x/cpu.h | 13 +++++++------
target/s390x/tcg/translate.c | 6 ------
2 files changed, 7 insertions(+), 12 deletions(-)
How can we end up in a situation where this matters? E.g. if we are in
64-bit mode and execute
0xa12345678: sam31
we will get a specification exception, and cpu_get_tb_cpu_state() will
not run. And for valid 31-bit addresses masking should be a no-op.