From: Pedro Alves Date: Wed, 10 Jun 2009 17:26:36 +0000 (+0000) Subject: * infrun.c (handle_inferior_event): Update comment around trying X-Git-Tag: cgen-1_1-branchpoint~80 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=66852e9c4de0fbfa8be916df494f98e6bcc9eed8;p=thirdparty%2Fbinutils-gdb.git * infrun.c (handle_inferior_event): Update comment around trying to revert back to a stepping thread that has exited. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 537fea8bdc6..3d97324ccde 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2009-06-10 Pedro Alves + + * infrun.c (handle_inferior_event): Update comment around trying + to revert back to a stepping thread that has exited. + 2009-06-10 Cary Coutant * dwarf2read.c (dwarf_decode_lines): Recognize and ignore diff --git a/gdb/infrun.c b/gdb/infrun.c index 4b101519508..885e3257db7 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -3496,9 +3496,25 @@ infrun: BPSTAT_WHAT_SET_LONGJMP_RESUME (!gdbarch_get_longjmp_target)\n"); return; } - /* If the stepping thread exited, then don't try reverting - back to it, just keep going. We need to query the target - in case it doesn't support thread exit events. */ + /* If the stepping thread exited, then don't try to switch + back and resume it, which could fail in several different + ways depending on the target. Instead, just keep going. + + We can find a stepping dead thread in the thread list in + two cases: + + - The target supports thread exit events, and when the + target tries to delete the thread from the thread list, + inferior_ptid pointed at the exiting thread. In such + case, calling delete_thread does not really remove the + thread from the list; instead, the thread is left listed, + with 'exited' state. + + - The target's debug interface does not support thread + exit events, and so we have no idea whatsoever if the + previously stepping thread is still alive. For that + reason, we need to synchronously query the target + now. */ if (is_exited (tp->ptid) || !target_thread_alive (tp->ptid)) {