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 Thiago Jung Bauermann * breakpoint.c (update_watchpoint): Change b->type to bp_watchpoint before calling hw_watchpoint_used_count.