diff options
| author | Richard Henderson <richard.henderson@linaro.org> | 2017-12-28 19:26:52 -0800 |
|---|---|---|
| committer | Richard Henderson <richard.henderson@linaro.org> | 2018-01-31 05:30:50 -0800 |
| commit | 46559e8184b23e539912bb78764e662b5eabb537 (patch) | |
| tree | 8088cc00325d0e06fab04ff329431cd83813d460 /target/hppa/mem_helper.c | |
| parent | 43e056522f3e653d0d7e412b4e4d54eb7bcae8b1 (diff) | |
| download | qemu-46559e8184b23e539912bb78764e662b5eabb537.zip qemu-46559e8184b23e539912bb78764e662b5eabb537.tar.gz qemu-46559e8184b23e539912bb78764e662b5eabb537.tar.bz2 | |
target/hppa: Only use EXCP_DTLB_MISS
Unknown why this works, but if we return EXCP_ITLB_MISS we
will triple-fault the first userland instruction fetch.
Is it something to do with having a combined I/DTLB?
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Diffstat (limited to 'target/hppa/mem_helper.c')
| -rw-r--r-- | target/hppa/mem_helper.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/target/hppa/mem_helper.c b/target/hppa/mem_helper.c index bf7559b..ab160c2 100644 --- a/target/hppa/mem_helper.c +++ b/target/hppa/mem_helper.c @@ -96,7 +96,9 @@ int hppa_get_physical_address(CPUHPPAState *env, vaddr addr, int mmu_idx, if (ent == NULL || !ent->entry_valid) { phys = 0; prot = 0; - ret = (type & PAGE_EXEC ? EXCP_ITLB_MISS : EXCP_DTLB_MISS); + /* ??? Unconditionally report data tlb miss, + even if this is an instruction fetch. */ + ret = EXCP_DTLB_MISS; goto egress; } |