Re: x86 watchpoints bug (Re: ping: Re: PATCH : allow to set length of hw watchpoints (e.g. for Valgrind gdbserver))

[Pedro Alves <pedro at codesourcery dot com>]

On Friday 22 July 2011 17:02:42, Philippe Waroquiers wrote:
> > I think nothing else changed in the patch.
> 
> I looked at the new patch and re-tested on f12/x86 and debian5/amd64, using 7.3.
> Behaviour looks ok to me regarding the handling of debug registers.

Awesome.  I've applied it now.

> So, in summary:

> The following strange behaviours/bugs have still to be fixed or looked at:
>   * handling of duplicate locations across disabled breakpoints
>      (resulting in wrongly duplicated z packets and/or missing active debug registers in native)
>   * watch s1000 then run then disable then enable impossible
> 
>  There was also a 'nice to have' which could be looked at:
>   * ensure that the insertion of watchpoint is done using the order of breakpoints
>      (so as to not have a new watchpoint causing an error/rejection on a previously accepted
>       watchpoint).

For avoidance of doubt, I'm not working on any of
those issues at the moment (and I don't know when/if I will).
If you don't plan to, it'll be a good idea to record them in bugzilla.
Actually, it'll be a good idea in any case.

> Thanks for all the work

Thank you!  

-- 
Pedro Alves