This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: Fixes for a couple of infrun bugs (thread hop, revert to step thread).
- From: Pedro Alves <pedro at codesourcery dot com>
- To: gdb-patches at sourceware dot org
- Cc: "Ulrich Weigand" <uweigand at de dot ibm dot com>
- Date: Mon, 16 Aug 2010 19:52:52 +0100
- Subject: Re: Fixes for a couple of infrun bugs (thread hop, revert to step thread).
- References: <201008161840.o7GIel4W024223@d12av02.megacenter.de.ibm.com>
On Monday 16 August 2010 19:40:47, Ulrich Weigand wrote:
> Pedro Alves wrote:
>
> > Replacing the "next" by a "continue" should work. I've looked over the
> > original description of the problem this is covering, and, that
> > would still exercise the problem (which is gdb trying to step
> > the other (main) thread with inferior_ptid still pointing at
> > the thread that was being "next"ed, and in the process failing
> > to remove breakpoints from memory because inferior_ptid pointed
> > at an inferior thread.
>
> But isn't the code your patch changes under an if that's only true
> if another thread is currently being stepped or nexted? If we just
> do "continue" here, that's no longer the case, and the code wouldn't
> be exercised at all ...
That's also exercised by the other test my original patch added,
IIRC (gdb.thread/thread-execl.exp). There were two bugs fixed by
that patch. The specific bug the threxit-hop-specific.exp test is
covering is the "failing to remove breakpoints" one.
--
Pedro Alves