Fix inconsistent breakpoint kinds between breakpoints and tracepoints in GDBServer.
authorAntoine Tremblay <antoine.tremblay@ericsson.com>
Thu, 22 Oct 2015 16:21:51 +0000 (12:21 -0400)
committerAntoine Tremblay <antoine.tremblay@ericsson.com>
Thu, 22 Oct 2015 16:23:58 +0000 (12:23 -0400)
commit1652a98612a20443c0e17fd90e7f6941b7bc3ad8
tree0778ff303a88ec35ba3f49d91d8e2dcd924500ab
parentdb0f0d0c1d0541b6c2f81d655cd5420e059c91d4
Fix inconsistent breakpoint kinds between breakpoints and tracepoints in GDBServer.

This patch fixes a regression introduced by :
https://sourceware.org/ml/gdb-patches/2015-10/msg00369.html

Tests : gdb.trace/trace-break.exp and gdb.trace/trace-mt.exp would fail on x86
with gdbserver-{native,extended}.

Before this patch, the breakpoint kind set by GDB with a Z packet and the one
set in the case of a tracepoint would be inconsistent on targets that did not
implement breakpoint_kind_from_pc. On x86 for example a breakpoint set by GDB
would have a kind of 1 but a breakpoint set by a tracepoint would have a kind of
0.

This created a missmatch when trying to insert a tracepoint and a breakpoint at
the same location. One of the two breakpoints would be removed with debug
message : "Inconsistent breakpoint kind".

This patch fixes the issue by changing the default 0 breakpoint kind to be
the size of the breakpoint according to sw_breakpoint_from_kind.

The default breakpoint kind must be the breakpoint length to keep consistency
between breakpoints set via GDB and the ones set internally by GDBServer.

No regression on Ubuntu 14.04 x86-64 with gdbserver-{native-extended}

gdb/gdbserver/ChangeLog:

* linux-low.c (default_breakpoint_kind_from_pc): New function.
(linux_breakpoint_kind_from_pc): Use default_breakpoint_kind_from_pc for
the default breakpoint kind.
gdb/gdbserver/ChangeLog
gdb/gdbserver/linux-low.c