From 3c9c4b83df84dc1fb2c8b4d7024049d5e0f53249 Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Thu, 16 Oct 2008 14:39:54 +0000 Subject: [PATCH] * remote.c (set_general_process): New. (remote_check_symbols): Use it. --- gdb/ChangeLog | 5 +++++ gdb/remote.c | 27 +++++++++++++++++++++++++++ 2 files changed, 32 insertions(+) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 94b3178..025c4a3 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2008-10-16 Pedro Alves + * remote.c (set_general_process): New. + (remote_check_symbols): Use it. + +2008-10-16 Pedro Alves + * remote.c (push_remote_target): Delete. * target.h (push_remote_target): Delete declaration. diff --git a/gdb/remote.c b/gdb/remote.c index 9a473f2..f9249f6 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -1246,6 +1246,30 @@ set_continue_thread (struct ptid ptid) set_thread (ptid, 0); } +/* Change the remote current process. Which thread within the process + ends up selected isn't important, as long as it is the same process + as what INFERIOR_PTID points to. + + This comes from that fact that there is no explicit notion of + "selected process" in the protocol. The selected process for + general operations is the process the selected general thread + belongs to. */ + +static void +set_general_process (void) +{ + struct remote_state *rs = get_remote_state (); + + /* If the remote can't handle multiple processes, don't bother. */ + if (!remote_multi_process_p (rs)) + return; + + /* We only need to change the remote current thread if it's pointing + at some other process. */ + if (ptid_get_pid (general_thread) != ptid_get_pid (inferior_ptid)) + set_general_thread (inferior_ptid); +} + /* Return nonzero if the thread PTID is still alive on the remote system. */ @@ -2553,6 +2577,9 @@ remote_check_symbols (struct objfile *objfile) if (remote_protocol_packets[PACKET_qSymbol].support == PACKET_DISABLE) return; + /* Make sure the remote is pointing at the right process. */ + set_general_process (); + /* Allocate a message buffer. We can't reuse the input buffer in RS, because we need both at the same time. */ msg = alloca (get_remote_packet_size ()); -- 2.7.4