OpenE2K/qemu-e2k
13
4
Fork 0
Code Issues 4 Pull Requests Projects 2 Releases Activity

target/xtensa: Use probe_access for itlb_hit_test

Browse Source 
We don't actually need the result of the read, only to probe that the
memory mapping exists.  This is exactly what probe_access does.

This is also the only user of any cpu_ld*_code_ra function.
Removing this allows the interface to be removed shortly.

Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
Richard Henderson 2019-12-09 15:21:52 -08:00
parent 3a63b24a1b
commit ecd3571eb1
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
target/xtensa/mmu_helper.c
View File

@ -63,10 +63,11 @@
void HELPER(itlb_hit_test)(CPUXtensaState *env, uint32_t vaddr) void HELPER(itlb_hit_test)(CPUXtensaState *env, uint32_t vaddr)
{ {
/* /*
* Attempt the memory load; we don't care about the result but * Probe the memory; we don't care about the result but
* only the side-effects (ie any MMU or other exception) * only the side-effects (ie any MMU or other exception)
*/ */
cpu_ldub_code_ra(env, vaddr, GETPC()); probe_access(env, vaddr, 1, MMU_INST_FETCH,
cpu_mmu_index(env, true), GETPC());
} }
void HELPER(wsr_rasid)(CPUXtensaState *env, uint32_t v) void HELPER(wsr_rasid)(CPUXtensaState *env, uint32_t v)