This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: x86 watchpoints bug (Re: ping: Re: PATCH : allow to set length of hw watchpoints (e.g. for Valgrind gdbserver))
- From: Pedro Alves <pedro at codesourcery dot com>
- To: gdb-patches at sourceware dot org
- Cc: "Philippe Waroquiers" <philippe dot waroquiers at skynet dot be>, yao at codesourcery dot com
- Date: Fri, 22 Jul 2011 18:00:39 +0100
- Subject: Re: x86 watchpoints bug (Re: ping: Re: PATCH : allow to set length of hw watchpoints (e.g. for Valgrind gdbserver))
- References: <CDA9C6B129F5458D9301BA5289052C97@soleil> <201107211712.26443.pedro@codesourcery.com> <BB85C0C7E4DE4A759F5FCA4D62B30B55@soleil>
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