On 12/12/24 10:53, Philippe Mathieu-Daudé wrote:
> Match with the page_protect() prototype, use a tb_page_addr_t
> argument instead of target_ulong.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> include/exec/translate-all.h | 2 +-
> accel/tcg/user-exec.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/exec/translate-all.h b/include/exec/translate-all.h
> index 85c9460c7ca..c50661a05d5 100644
> --- a/include/exec/translate-all.h
> +++ b/include/exec/translate-all.h
> @@ -27,7 +27,7 @@ void tb_check_watchpoint(CPUState *cpu, uintptr_t retaddr);
>
> #ifdef CONFIG_USER_ONLY
> void page_protect(tb_page_addr_t page_addr);
> -int page_unprotect(target_ulong address, uintptr_t pc);
> +int page_unprotect(tb_page_addr_t address, uintptr_t pc);
> #endif
>
> #endif /* TRANSLATE_ALL_H */
> diff --git a/accel/tcg/user-exec.c b/accel/tcg/user-exec.c
> index 06016eb0301..4ed6dd19f30 100644
> --- a/accel/tcg/user-exec.c
> +++ b/accel/tcg/user-exec.c
> @@ -696,7 +696,7 @@ void page_protect(tb_page_addr_t address)
> * immediately exited. (We can only return 2 if the 'pc' argument is
> * non-zero.)
> */
> -int page_unprotect(target_ulong address, uintptr_t pc)
> +int page_unprotect(tb_page_addr_t address, uintptr_t pc)
> {
> PageFlagsNode *p;
> bool current_tb_invalidated;
Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>