From bbf550d50e4c85666877456f014421089503e83b Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Sun, 19 Nov 2017 22:23:22 -0500 Subject: [PATCH] Remove usage of find_inferior in reset_lwp_ptrace_options_callback Replace with for_each_thread, and inline code from reset_lwp_ptrace_options_callback. gdb/gdbserver/ChangeLog: * linux-low.c (reset_lwp_ptrace_options_callback): Remove. (linux_handle_new_gdb_connection): Use for_each_thread, inline code from reset_lwp_ptrace_options_callback. --- gdb/gdbserver/ChangeLog | 6 +++++ gdb/gdbserver/linux-low.c | 50 ++++++++++++++++----------------------- 2 files changed, 26 insertions(+), 30 deletions(-) diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index c9f5966b502..3a233bbcd72 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,9 @@ +2017-11-19 Simon Marchi + + * linux-low.c (reset_lwp_ptrace_options_callback): Remove. + (linux_handle_new_gdb_connection): Use for_each_thread, inline + code from reset_lwp_ptrace_options_callback. + 2017-11-19 Simon Marchi * linux-arm-low.c (struct update_registers_data): Remove. diff --git a/gdb/gdbserver/linux-low.c b/gdb/gdbserver/linux-low.c index b267c70f059..628135a49bc 100644 --- a/gdb/gdbserver/linux-low.c +++ b/gdb/gdbserver/linux-low.c @@ -6434,33 +6434,6 @@ linux_supports_exec_events (void) return linux_supports_traceexec (); } -/* Callback for 'find_inferior'. Set the (possibly changed) ptrace - options for the specified lwp. */ - -static int -reset_lwp_ptrace_options_callback (thread_info *thread, void *args) -{ - struct lwp_info *lwp = get_thread_lwp (thread); - - if (!lwp->stopped) - { - /* Stop the lwp so we can modify its ptrace options. */ - lwp->must_set_ptrace_flags = 1; - linux_stop_lwp (lwp); - } - else - { - /* Already stopped; go ahead and set the ptrace options. */ - struct process_info *proc = find_process_pid (pid_of (thread)); - int options = linux_low_ptrace_options (proc->attached); - - linux_enable_event_reporting (lwpid_of (thread), options); - lwp->must_set_ptrace_flags = 0; - } - - return 0; -} - /* Target hook for 'handle_new_gdb_connection'. Causes a reset of the ptrace flags for all inferiors. This is in case the new GDB connection doesn't support the same set of events that the previous one did. */ @@ -6468,10 +6441,27 @@ reset_lwp_ptrace_options_callback (thread_info *thread, void *args) static void linux_handle_new_gdb_connection (void) { - pid_t pid; - /* Request that all the lwps reset their ptrace options. */ - find_inferior (&all_threads, reset_lwp_ptrace_options_callback , &pid); + for_each_thread ([] (thread_info *thread) + { + struct lwp_info *lwp = get_thread_lwp (thread); + + if (!lwp->stopped) + { + /* Stop the lwp so we can modify its ptrace options. */ + lwp->must_set_ptrace_flags = 1; + linux_stop_lwp (lwp); + } + else + { + /* Already stopped; go ahead and set the ptrace options. */ + struct process_info *proc = find_process_pid (pid_of (thread)); + int options = linux_low_ptrace_options (proc->attached); + + linux_enable_event_reporting (lwpid_of (thread), options); + lwp->must_set_ptrace_flags = 0; + } + }); } static int -- 2.30.2