gdb/riscv: expect h/w watchpoints to trigger before the memory is written
authorJoel Brobecker <brobecker@adacore.com>
Tue, 23 Oct 2018 10:27:50 +0000 (11:27 +0100)
committerAndrew Burgess <andrew.burgess@embecosm.com>
Tue, 23 Oct 2018 10:31:27 +0000 (11:31 +0100)
commit5a77b1b49f49cc5cfdb30727d8fc1bf456cad429
treeacd4eafe55849313bcf7cbf15b51623de68aec20
parent0dbfcfffe9abbc5198bce95eb8c66b6bc9b364be
gdb/riscv: expect h/w watchpoints to trigger before the memory is written

When using QEMU as a RISCV simulator, hardware watchpoint events are
reported to GDB before the target memory gets written. GDB currently
expects the event to be reported after it is written. As a result of
this mismatch, upon receiving the event, GDB sees that the target
memory region has not changed, and therefore decides to ignore the
event. It therefore resumes the program's execution with a continue,
which is the start of an infinite loop between QEMU repeatedly
reporting the same watchpoint event over and over, and GDB repeatedly
ignoring it.

This patch fixes the issue by telling GDB to expect the watchpoint
event to be reported ahead of the memory region being modified.
Upon receiving the event, GDB then single-steps the program before
checking the watched memory value.

gdb/ChangeLog:

        * riscv-tdep.c (riscv_gdbarch_init): Set the gdbarch's
        have_nonsteppable_watchpoint attribute to 1.
gdb/ChangeLog
gdb/riscv-tdep.c