On 15/8/26 22:37, Richard Henderson wrote:
Signed-off-by: Richard Henderson <[email protected]>
---
  include/accel/tcg/probe.h     |  2 +-
  accel/tcg/cputlb.c            | 17 +++++++++++------
  target/arm/tcg/helper-a64.c   |  5 ++++-
  target/arm/tcg/mte_helper.c   | 21 ++++++++++++---------
  target/arm/tcg/sve_helper.c   |  5 ++++-
  target/mips/tcg/ldst_helper.c |  4 ++--
  6 files changed, 34 insertions(+), 20 deletions(-)

diff --git a/include/accel/tcg/probe.h b/include/accel/tcg/probe.h
index 0b788901ba..e3068a79de 100644
--- a/include/accel/tcg/probe.h
+++ b/include/accel/tcg/probe.h
@@ -86,7 +86,7 @@ int probe_access_flags(CPUArchState *env, vaddr addr, int 
size,
   * This function will return TLB_MMIO in order to force the access
   * to be handled out-of-line if plugins wish to instrument the access.
   */
-int probe_access_full(CPUArchState *env, vaddr addr, int size,
+int probe_access_full(CPUArchState *env, vaddr addr, vaddr first, vaddr last,

I prefer this clearer first/last range API, it feels safer.

Missing updating the docstring, otherwise:
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>

                        MMUAccessType access_type, int mmu_idx,
                        bool nonfault, void **phost,
                        CPUTLBEntryFull **pfull, uintptr_t retaddr);

Reply via email to