[patch] gdbserver: Add support for Z0/Z1 packets
Aleksandar Ristovski
aristovski@qnx.com
Tue Jun 23 15:18:00 GMT 2009
Pedro Alves wrote:
> On Monday 22 June 2009 20:38:50, Aleksandar Ristovski wrote:
>
>>> Z0 and Z1 breakpoints also take a 'len' argument, just
>>> like Z2-Z4. You should also pass those down.
>>>
>>> But, Let's take a step back --- why not just rename the
>>> insert_watchpoint|remove_watchpoint functions to insert_point,remove_point,
>>> and relax the type checks in server.c:
>> That was my initial implementation, prior to proposing the
>> change. Then I looked at target ops in gdb; there we have
>> two different functions for breakpoint and watchpoint so I
>> followed that logic (even though the logic there seems to be
>> incomplete: there is a pair for hw and non-hw breakponts but
>> only one pair for watchpoints).
>
> That's because software watchpoints aren't "inserted".
Yes, silly me.
>
>> But either way is fine with me - just let me know.
>
> I'd prefer the approach I suggested, and worry about splitting
> the breakpoints from watchpoints API if/when we actually need it.
>
Ok, then that version is committed.
I attached what I committed.
ChangeLog:
* server.c (process_serial_event): Add support for Z0 and Z1
packet.
* target.h: Comment for *_watchpoint to make it clear the
functions
can get types '0' and '1'.
Thanks,
--
Aleksandar Ristovski
QNX Software Systems
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gdbserver-Z0Z1support-20090623.diff
Type: text/x-patch
Size: 3699 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20090623/4e2d4217/attachment.bin>
More information about the Gdb-patches
mailing list