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

Pedro Alves
2011-07-22T17:01:02Z

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