diff options
author | Richard Henderson | 2022-08-20 00:28:05 +0200 |
---|---|---|
committer | Richard Henderson | 2022-10-04 05:53:30 +0200 |
commit | c3c8bf579b431b6b305dce12d67ff6fe86bc79e8 (patch) | |
tree | e8485f7a378f9af3607ca35f02aa47912e032a68 /target/s390x | |
parent | accel/tcg: Drop addr member from SavedIOTLB (diff) | |
download | qemu-c3c8bf579b431b6b305dce12d67ff6fe86bc79e8.tar.gz qemu-c3c8bf579b431b6b305dce12d67ff6fe86bc79e8.tar.xz qemu-c3c8bf579b431b6b305dce12d67ff6fe86bc79e8.zip |
accel/tcg: Suppress auto-invalidate in probe_access_internal
When PAGE_WRITE_INV is set when calling tlb_set_page,
we immediately set TLB_INVALID_MASK in order to force
tlb_fill to be called on the next lookup. Here in
probe_access_internal, we have just called tlb_fill
and eliminated true misses, thus the lookup must be valid.
This allows us to remove a warning comment from s390x.
There doesn't seem to be a reason to change the code though.
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: David Hildenbrand <david@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'target/s390x')
-rw-r--r-- | target/s390x/tcg/mem_helper.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/target/s390x/tcg/mem_helper.c b/target/s390x/tcg/mem_helper.c index fc52aa128b..3758b9e688 100644 --- a/target/s390x/tcg/mem_helper.c +++ b/target/s390x/tcg/mem_helper.c @@ -148,10 +148,6 @@ static int s390_probe_access(CPUArchState *env, target_ulong addr, int size, #else int flags; - /* - * For !CONFIG_USER_ONLY, we cannot rely on TLB_INVALID_MASK or haddr==NULL - * to detect if there was an exception during tlb_fill(). - */ env->tlb_fill_exc = 0; flags = probe_access_flags(env, addr, access_type, mmu_idx, nonfault, phost, ra); |