[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