Message ID | 20190823100741.9621-10-david@redhat.com |
---|---|
State | New |
Headers | show |
Series | [v1,1/9] s390x/tcg: Use guest_addr_valid() instead of h2g_valid() in probe_write_access() | expand |
On 8/23/19 3:07 AM, David Hildenbrand wrote: > Let's check for write watchpoints. We'll want to do something similar > for probe_read() in the future (once we introduce that). > > Suggested-by: Richard Henderson <richard.henderson@linaro.org> > Signed-off-by: David Hildenbrand <david@redhat.com> > --- > accel/tcg/cputlb.c | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) Reviewed-by: Richard Henderson <richard.henderson@linaro.org> r~
On 8/23/19 3:07 AM, David Hildenbrand wrote: > @@ -1071,8 +1072,23 @@ void probe_write(CPUArchState *env, target_ulong addr, > if (!VICTIM_TLB_HIT(addr_write, addr)) { > tlb_fill(env_cpu(env), addr, size, MMU_DATA_STORE, > mmu_idx, retaddr); > + /* TLB resize via tlb_fill may have moved the entry. */ > + entry = tlb_entry(env, mmu_idx, addr); > } > } > + > + if (!size) { > + return; > + } > + tlb_addr = tlb_addr_write(entry); > + > + /* Watchpoints for this entry only apply if TLB_MMIO was set. */ > + if (tlb_addr & TLB_MMIO) { > + MemTxAttrs attrs = env_tlb(env)->d[mmu_idx].iotlb[index].attrs; We need to recompute index above as well, since we use it here. Fixed up and applied to tcg-next. r~
diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c index 4b49ccb58a..8382ac2fc2 100644 --- a/accel/tcg/cputlb.c +++ b/accel/tcg/cputlb.c @@ -1063,6 +1063,7 @@ void probe_write(CPUArchState *env, target_ulong addr, int size, int mmu_idx, { uintptr_t index = tlb_index(env, mmu_idx, addr); CPUTLBEntry *entry = tlb_entry(env, mmu_idx, addr); + target_ulong tlb_addr; g_assert(-(addr | TARGET_PAGE_MASK) >= size); @@ -1071,8 +1072,23 @@ void probe_write(CPUArchState *env, target_ulong addr, int size, int mmu_idx, if (!VICTIM_TLB_HIT(addr_write, addr)) { tlb_fill(env_cpu(env), addr, size, MMU_DATA_STORE, mmu_idx, retaddr); + /* TLB resize via tlb_fill may have moved the entry. */ + entry = tlb_entry(env, mmu_idx, addr); } } + + if (!size) { + return; + } + tlb_addr = tlb_addr_write(entry); + + /* Watchpoints for this entry only apply if TLB_MMIO was set. */ + if (tlb_addr & TLB_MMIO) { + MemTxAttrs attrs = env_tlb(env)->d[mmu_idx].iotlb[index].attrs; + + cpu_check_watchpoint(env_cpu(env), addr, size, attrs, BP_MEM_WRITE, + retaddr); + } } void *tlb_vaddr_to_host(CPUArchState *env, abi_ptr addr,
Let's check for write watchpoints. We'll want to do something similar for probe_read() in the future (once we introduce that). Suggested-by: Richard Henderson <richard.henderson@linaro.org> Signed-off-by: David Hildenbrand <david@redhat.com> --- accel/tcg/cputlb.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)