Re: [PATCH v3 17/51] gdbstub: Report actual watchpoint trigger address

Philippe Mathieu-Daudé <[email protected]> Fri, 10 Jul 2026 23:52:43 +0200

Newsgroups	org.nongnu.qemu-riscv,org.nongnu.qemu-arm,org.nongnu.qemu-devel
Message-ID	<[email protected]>

On 10/7/26 22:53, Richard Henderson wrote:
> While gdb doesn't care either way, it seems more correct to
> report the address of the watchpoint hit, rather than the
> beginning of the watchpoint region.
> 
> Adjust all virtualization hosts which construct a dummy
> CPUWatchpoint structure.
> 
> Signed-off-by: Richard Henderson <[email protected]>
> ---
>   gdbstub/system.c       | 4 ++--
>   target/arm/hvf/hvf.c   | 1 +
>   target/arm/kvm.c       | 1 +
>   target/i386/kvm/kvm.c  | 4 ++--
>   target/ppc/kvm.c       | 4 ++--
>   target/s390x/kvm/kvm.c | 2 +-
>   6 files changed, 9 insertions(+), 7 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <[email protected]>