diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 3a6bffdcf3..abad94504a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2014-10-15 Pedro Alves + + * gdbthread.h (ALL_NON_EXITED_THREADS_SAFE): New macro. + * remote.c (remote_update_thread_list): Skip calling prune_threads + if any thread listing method is supported, and instead walk over + the set of remote threads listed, deleting those that are not + found in GDB's thread list. + 2014-10-15 Pedro Alves * ada-tasks.c (print_ada_task_info, task_command_1): Adjust. diff --git a/gdb/gdbthread.h b/gdb/gdbthread.h index 1ed98a23bb..937bd55cd6 100644 --- a/gdb/gdbthread.h +++ b/gdb/gdbthread.h @@ -354,6 +354,14 @@ extern struct thread_info *iterate_over_threads (thread_callback_func, void *); for (T = thread_list; T; T = T->next) \ if ((T)->state != THREAD_EXITED) +/* Like ALL_NON_EXITED_THREADS, but allows deleting the currently + iterated thread. */ +#define ALL_NON_EXITED_THREADS_SAFE(T, TMP) \ + for ((T) = thread_list; \ + (T) != NULL ? ((TMP) = (T)->next, 1): 0; \ + (T) = (TMP)) \ + if ((T)->state != THREAD_EXITED) + extern int thread_count (void); /* Switch from one thread to another. */ diff --git a/gdb/remote.c b/gdb/remote.c index cd85fd5986..98d3f2a57f 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -2762,9 +2762,7 @@ remote_update_thread_list (struct target_ops *ops) struct remote_state *rs = get_remote_state (); struct threads_listing_context context; struct cleanup *old_chain; - - /* Delete GDB-side threads no longer found on the target. */ - prune_threads (); + int got_list = 0; context.items = NULL; old_chain = make_cleanup (clear_threads_listing_context, &context); @@ -2778,8 +2776,31 @@ remote_update_thread_list (struct target_ops *ops) { int i; struct thread_item *item; + struct thread_info *tp, *tmp; - /* Now add threads we don't know about yet to our list. */ + got_list = 1; + + /* CONTEXT now holds the current thread list on the remote + target end. Delete GDB-side threads no longer found on the + target. */ + ALL_NON_EXITED_THREADS_SAFE (tp, tmp) + { + for (i = 0; + VEC_iterate (thread_item_t, context.items, i, item); + ++i) + { + if (ptid_equal (item->ptid, tp->ptid)) + break; + } + + if (i == VEC_length (thread_item_t, context.items)) + { + /* Not found. */ + delete_thread (tp->ptid); + } + } + + /* And now add threads we don't know about yet to our list. */ for (i = 0; VEC_iterate (thread_item_t, context.items, i, item); ++i) @@ -2803,6 +2824,15 @@ remote_update_thread_list (struct target_ops *ops) } } + if (!got_list) + { + /* If no thread listing method is supported, then query whether + each known thread is alive, one by one, with the T packet. + If the target doesn't support threads at all, then this is a + no-op. See remote_thread_alive. */ + prune_threads (); + } + do_cleanups (old_chain); }