[3/7] Adjust the bsd-uthread target

Pedro Alves pedro@codesourcery.com
Mon Aug 11 13:33:00 GMT 2008


On Saturday 09 August 2008 02:31:30, Daniel Jacobowitz wrote:
> On Fri, Aug 08, 2008 at 11:41:23PM +0100, Pedro Alves wrote:
> > The "exited" state is what you get *after* you delete_thread the current
> > thread (inferior_ptid).  We don't delete the thread from the thread list
> > in that case, but instead tag it as "exited".  It means the thread list
> > is still holding reference to a thread that has already exited.
> > If you're seeing an event with a ptid equal to an "exited" thread,
> > this is the OS reusing the ptid, but, it's a new thread, which
> > should get a new gdb thread id, so things like thread
> > specific breakpoints don't think this was the same thread, both the
> > CLI user or MI see a "new thread event", etc. add_thread handles
> > that case internally.  This means it is now safe to delete_thread
> > (inferior_ptid), regarding context-switching, and infrun state.  It
> > wasn't a couple of weeks ago.

> How many of the call sites for in_thread_list want to see exited
> threads?  Maybe there should be another predicate (I'd suggest
> thread_alive except target_thread_alive would confuse things...)

I actually started out using a new predicate, but then got rid
of it, as it looked more confusing to me.

There are a few calls than want to see all threads.  context_switch (while
it still exists :-) ) and handle_inferior_event want to.  The
breakpoints.c and infcmd.c calls don't care currently.

Can I ask you to not do that now, please?  I can come back to it
as an follow-up cleanup.  I've got several patches that touch
contexts where this form was used, and it would be a lot of work
to rebase and retest them all for not much gain.  A single patch
to cleanup and introduce a new predicate can be made in one sweep.

-- 
Pedro Alves



More information about the Gdb-patches mailing list