This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Z packet change
- To: gdb-patches at sourceware dot cygnus dot com
- Subject: Z packet change
- From: Mark Salter <msalter at redhat dot com>
- Date: Fri, 27 Oct 2000 09:17:31 -0400
The following patch changes the Z packet support for hw breakpoints to
match the documentation. Its not clear to me if we should do this or
change the documentation to make make the 'len' field optional. Some
hw lets you specify a range for hw breakpoints and although gdb doesn't
support that right now, it may in the future.
--Mark
2000-10-27 Mark Salter <msalter@redhat.com>
* remote.c (remote_remove_hw_breakpoint): Add 'len' field to Z packet.
(remote_insert_hw_breakpoint): Ditto.
Index: remote.c
===================================================================
RCS file: /cvs/src/src/gdb/remote.c,v
retrieving revision 1.26
diff -p -c -r1.26 remote.c
*** remote.c 2000/10/23 22:49:28 1.26
--- remote.c 2000/10/27 13:06:52
*************** remote_insert_hw_breakpoint (CORE_ADDR a
*** 4433,4439 ****
addr = remote_address_masked (addr);
p += hexnumstr (p, (ULONGEST) addr);
! *p = '\0';
putpkt (buf);
getpkt (buf, PBUFSIZ, 0);
--- 4433,4439 ----
addr = remote_address_masked (addr);
p += hexnumstr (p, (ULONGEST) addr);
! sprintf (p, ",%x", len);
putpkt (buf);
getpkt (buf, PBUFSIZ, 0);
*************** remote_remove_hw_breakpoint (CORE_ADDR a
*** 4469,4475 ****
addr = remote_address_masked (addr);
p += hexnumstr (p, (ULONGEST) addr);
! *p = '\0';
putpkt(buf);
getpkt (buf, PBUFSIZ, 0);
--- 4469,4475 ----
addr = remote_address_masked (addr);
p += hexnumstr (p, (ULONGEST) addr);
! sprintf (p, ",%x", len);
putpkt(buf);
getpkt (buf, PBUFSIZ, 0);