tcg: Factor out probe_write() logic into probe_access()

Let's also allow to probe other access types.

Backports commit c25c283df0f08582df29f1d5d7be1516b851532d from qemu
This commit is contained in:
David Hildenbrand 2020-01-14 07:07:11 -05:00 committed by Lioncash
parent 53c3c47efa
commit d9d91c1db6
18 changed files with 56 additions and 13 deletions

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_aarch64
#define print_type_size print_type_size_aarch64
#define print_type_str print_type_str_aarch64
#define probe_access probe_access_aarch64
#define probe_write probe_write_aarch64
#define propagateFloat128NaN propagateFloat128NaN_aarch64
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_aarch64

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_aarch64eb
#define print_type_size print_type_size_aarch64eb
#define print_type_str print_type_str_aarch64eb
#define probe_access probe_access_aarch64eb
#define probe_write probe_write_aarch64eb
#define propagateFloat128NaN propagateFloat128NaN_aarch64eb
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_aarch64eb

View file

@ -703,30 +703,51 @@ static void io_writex(CPUArchState *env, CPUIOTLBEntry *iotlbentry,
}
}
/* Probe for whether the specified guest write access is permitted.
* If it is not permitted then an exception will be taken in the same
* way as if this were a real write access (and we will not return).
/*
* Probe for whether the specified guest access is permitted. If it is not
* permitted then an exception will be taken in the same way as if this
* were a real access (and we will not return).
* If the size is 0 or the page requires I/O access, returns NULL; otherwise,
* returns the address of the host page similar to tlb_vaddr_to_host().
*/
void *probe_write(CPUArchState *env, target_ulong addr, int size, int mmu_idx,
uintptr_t retaddr)
void *probe_access(CPUArchState *env, target_ulong addr, int size,
MMUAccessType access_type, int mmu_idx, uintptr_t retaddr)
{
uintptr_t index = tlb_index(env, mmu_idx, addr);
CPUTLBEntry *entry = tlb_entry(env, mmu_idx, addr);
target_ulong tlb_addr = tlb_addr_write(entry);
target_ulong tlb_addr;
size_t elt_ofs;
int wp_access;
g_assert(-(addr | TARGET_PAGE_MASK) >= size);
switch (access_type) {
case MMU_DATA_LOAD:
elt_ofs = offsetof(CPUTLBEntry, addr_read);
wp_access = BP_MEM_READ;
break;
case MMU_DATA_STORE:
elt_ofs = offsetof(CPUTLBEntry, addr_write);
wp_access = BP_MEM_WRITE;
break;
case MMU_INST_FETCH:
elt_ofs = offsetof(CPUTLBEntry, addr_code);
wp_access = BP_MEM_READ;
break;
default:
g_assert_not_reached();
}
tlb_addr = tlb_read_ofs(entry, elt_ofs);
if (unlikely(!tlb_hit(tlb_addr, addr))) {
if (!VICTIM_TLB_HIT(addr_write, addr)) {
tlb_fill(env_cpu(env), addr, size, MMU_DATA_STORE,
mmu_idx, retaddr);
if (!victim_tlb_hit(env, mmu_idx, index, elt_ofs,
addr & TARGET_PAGE_MASK)) {
tlb_fill(env_cpu(env), addr, size, access_type, mmu_idx, retaddr);
/* TLB resize via tlb_fill may have moved the entry. */
index = tlb_index(env, mmu_idx, addr);
entry = tlb_entry(env, mmu_idx, addr);
}
tlb_addr = tlb_addr_write(entry);
tlb_addr = tlb_read_ofs(entry, elt_ofs);
}
if (!size) {
@ -737,7 +758,7 @@ void *probe_write(CPUArchState *env, target_ulong addr, int size, int mmu_idx,
if (tlb_addr & TLB_WATCHPOINT) {
cpu_check_watchpoint(env_cpu(env), addr, size,
env->iotlb[mmu_idx][index].attrs,
BP_MEM_WRITE, retaddr);
wp_access, retaddr);
}
if (tlb_addr & (TLB_NOTDIRTY | TLB_MMIO)) {

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_arm
#define print_type_size print_type_size_arm
#define print_type_str print_type_str_arm
#define probe_access probe_access_arm
#define probe_write probe_write_arm
#define propagateFloat128NaN propagateFloat128NaN_arm
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_arm

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_armeb
#define print_type_size print_type_size_armeb
#define print_type_str print_type_str_armeb
#define probe_access probe_access_armeb
#define probe_write probe_write_armeb
#define propagateFloat128NaN propagateFloat128NaN_armeb
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_armeb

View file

@ -2263,6 +2263,7 @@ symbols = (
'print_type_number',
'print_type_size',
'print_type_str',
'probe_access',
'probe_write',
'propagateFloat128NaN',
'propagateFloat32MulAddNaN',

View file

@ -213,8 +213,14 @@ static inline void tb_invalidate_phys_addr(AddressSpace *as, hwaddr addr)
}
#endif
void *probe_write(CPUArchState *env, target_ulong addr, int size, int mmu_idx,
uintptr_t retaddr);
void *probe_access(CPUArchState *env, target_ulong addr, int size,
MMUAccessType access_type, int mmu_idx, uintptr_t retaddr);
static inline void *probe_write(CPUArchState *env, target_ulong addr, int size,
int mmu_idx, uintptr_t retaddr)
{
return probe_access(env, addr, size, MMU_DATA_STORE, mmu_idx, retaddr);
}
#define CODE_GEN_ALIGN 16 /* must be >= of the size of a icache line */

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_m68k
#define print_type_size print_type_size_m68k
#define print_type_str print_type_str_m68k
#define probe_access probe_access_m68k
#define probe_write probe_write_m68k
#define propagateFloat128NaN propagateFloat128NaN_m68k
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_m68k

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_mips
#define print_type_size print_type_size_mips
#define print_type_str print_type_str_mips
#define probe_access probe_access_mips
#define probe_write probe_write_mips
#define propagateFloat128NaN propagateFloat128NaN_mips
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_mips

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_mips64
#define print_type_size print_type_size_mips64
#define print_type_str print_type_str_mips64
#define probe_access probe_access_mips64
#define probe_write probe_write_mips64
#define propagateFloat128NaN propagateFloat128NaN_mips64
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_mips64

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_mips64el
#define print_type_size print_type_size_mips64el
#define print_type_str print_type_str_mips64el
#define probe_access probe_access_mips64el
#define probe_write probe_write_mips64el
#define propagateFloat128NaN propagateFloat128NaN_mips64el
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_mips64el

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_mipsel
#define print_type_size print_type_size_mipsel
#define print_type_str print_type_str_mipsel
#define probe_access probe_access_mipsel
#define probe_write probe_write_mipsel
#define propagateFloat128NaN propagateFloat128NaN_mipsel
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_mipsel

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_powerpc
#define print_type_size print_type_size_powerpc
#define print_type_str print_type_str_powerpc
#define probe_access probe_access_powerpc
#define probe_write probe_write_powerpc
#define propagateFloat128NaN propagateFloat128NaN_powerpc
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_powerpc

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_riscv32
#define print_type_size print_type_size_riscv32
#define print_type_str print_type_str_riscv32
#define probe_access probe_access_riscv32
#define probe_write probe_write_riscv32
#define propagateFloat128NaN propagateFloat128NaN_riscv32
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_riscv32

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_riscv64
#define print_type_size print_type_size_riscv64
#define print_type_str print_type_str_riscv64
#define probe_access probe_access_riscv64
#define probe_write probe_write_riscv64
#define propagateFloat128NaN propagateFloat128NaN_riscv64
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_riscv64

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_sparc
#define print_type_size print_type_size_sparc
#define print_type_str print_type_str_sparc
#define probe_access probe_access_sparc
#define probe_write probe_write_sparc
#define propagateFloat128NaN propagateFloat128NaN_sparc
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_sparc

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_sparc64
#define print_type_size print_type_size_sparc64
#define print_type_str print_type_str_sparc64
#define probe_access probe_access_sparc64
#define probe_write probe_write_sparc64
#define propagateFloat128NaN propagateFloat128NaN_sparc64
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_sparc64

View file

@ -2257,6 +2257,7 @@
#define print_type_number print_type_number_x86_64
#define print_type_size print_type_size_x86_64
#define print_type_str print_type_str_x86_64
#define probe_access probe_access_x86_64
#define probe_write probe_write_x86_64
#define propagateFloat128NaN propagateFloat128NaN_x86_64
#define propagateFloat32MulAddNaN propagateFloat32MulAddNaN_x86_64