From b65dc60b233704405c334bbe16c4fc5689187b7b Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Fri, 21 Mar 2014 11:08:44 +0000 Subject: [PATCH] normal_stop: Extend and clarify comment. Explain better why we skip saying "Switching to ..." in non-stop mode. gdb/ 2014-03-21 Pedro Alves * infrun.c (normal_stop): Extend comment. --- gdb/ChangeLog | 4 ++++ gdb/infrun.c | 16 ++++++++++------ 2 files changed, 14 insertions(+), 6 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ba2ca31f3ce..0aec7b92304 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2014-03-21 Pedro Alves + + * infrun.c (normal_stop): Extend comment. + 2014-03-21 Hui Zhu Pedro Alves diff --git a/gdb/infrun.c b/gdb/infrun.c index d7de7d16339..8f9e820735a 100644 --- a/gdb/infrun.c +++ b/gdb/infrun.c @@ -6059,18 +6059,22 @@ normal_stop (void) && last.kind != TARGET_WAITKIND_NO_RESUMED) make_cleanup (finish_thread_state_cleanup, &inferior_ptid); - /* In non-stop mode, we don't want GDB to switch threads behind the - user's back, to avoid races where the user is typing a command to - apply to thread x, but GDB switches to thread y before the user - finishes entering the command. */ - /* As with the notification of thread events, we want to delay notifying the user that we've switched thread context until the inferior actually stops. There's no point in saying anything if the inferior has exited. Note that SIGNALLED here means "exited with a signal", not - "received a signal". */ + "received a signal". + + Also skip saying anything in non-stop mode. In that mode, as we + don't want GDB to switch threads behind the user's back, to avoid + races where the user is typing a command to apply to thread x, + but GDB switches to thread y before the user finishes entering + the command, fetch_inferior_event installs a cleanup to restore + the current thread back to the thread the user had selected right + after this event is handled, so we're not really switching, only + informing of a stop. */ if (!non_stop && !ptid_equal (previous_inferior_ptid, inferior_ptid) && target_has_execution -- 2.30.2