[RFA] Fix hw watchpoint count in update_watchpoint
Thiago Jung Bauermann
bauerman@br.ibm.com
Wed Dec 30 20:14:00 GMT 2009
On Wed 30 Dec 2009 17:06:26 Joel Brobecker wrote:
> > + /* Mark as software watchpoint to ensure that this watchpoint
> > + will not be counted in the hw_watchpoint_used_count call
> > + below. */
>
> /* We need to determine how many resources are already used
> for all other hardware watchpoints to see if we still have
> enough resources to also fit this watchpoint in as well.
> To avoid the hw_watchpoint_used_count call below from
> counting this watchpoint, make sure that it is marked as a software
> watchpoint. */
>
> This is only a suggestion, so if you don't like it, please commit your
> version, and I'll see if I can suggest an update separately.
Your version is much better. I used it.
> Thanks for submitting the patch! (and welcome back, it's been a while)
Thanks! It's nice to be back. :-)
I committed the attached.
--
[]'s
Thiago Jung Bauermann
IBM Linux Technology Center
20090-12-30 Jan Kratochvil <jan.kratochvil@redhat.com>
Thiago Jung Bauermann <bauerman@br.ibm.com>
* breakpoint.c (update_watchpoint): Change b->type to
bp_watchpoint before calling hw_watchpoint_used_count.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: watchpoint-count-fix.diff
Type: text/x-patch
Size: 874 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20091230/84805a7c/attachment.bin>
More information about the Gdb-patches
mailing list