gdb: make find_thread_ptid a process_stratum_target method
authorSimon Marchi <simon.marchi@polymtl.ca>
Mon, 27 Mar 2023 16:53:55 +0000 (12:53 -0400)
committerSimon Marchi <simon.marchi@efficios.com>
Wed, 5 Apr 2023 01:05:31 +0000 (21:05 -0400)
Make find_thread_ptid (the overload that takes a process_stratum_target)
a method of process_stratum_target.

Change-Id: Ib190a925a83c6b93e9c585dc7c6ab65efbdd8629
Reviewed-By: Tom Tromey <tom@tromey.com>
25 files changed:
gdb/aarch64-tdep.c
gdb/aix-thread.c
gdb/amd-dbgapi-target.c
gdb/bsd-uthread.c
gdb/darwin-nat.c
gdb/fbsd-nat.c
gdb/gdbthread.h
gdb/gnu-nat.c
gdb/infcmd.c
gdb/infrun.c
gdb/linux-fork.c
gdb/linux-nat.c
gdb/linux-thread-db.c
gdb/mi/mi-interp.c
gdb/netbsd-nat.c
gdb/nto-procfs.c
gdb/process-stratum-target.c
gdb/process-stratum-target.h
gdb/procfs.c
gdb/ravenscar-thread.c
gdb/record-btrace.c
gdb/remote.c
gdb/thread-iter.c
gdb/thread.c
gdb/windows-nat.c

index d11d8320799cf26be2186b87c07945cc766c8748..a1daa9a17884d8048b6d4f1ca037dab643cb7076 100644 (file)
@@ -3576,9 +3576,7 @@ aarch64_remove_non_address_bits (struct gdbarch *gdbarch, CORE_ADDR pointer)
     {
       /* If we do have an inferior, attempt to fetch its thread's thread_info
         struct.  */
-      thread_info *thread
-       = find_thread_ptid (current_inferior ()->process_target (),
-                           inferior_ptid);
+      thread_info *thread = current_inferior ()->find_thread (inferior_ptid);
 
       /* If the thread is running, we will not be able to fetch the mask
         registers.  */
index f4ccfb2b364dc055ddf60c3237426a0c84398971..c587027fb6dc8eda4d133c92bd11974ac0c81e23 100644 (file)
@@ -941,7 +941,7 @@ sync_threadlists (pid_t pid)
 
              if (gptid.is_pid ())
                {
-                 tp = find_thread_ptid (proc_target, gptid);
+                 tp = proc_target->find_thread (gptid);
                  thread_change_ptid (proc_target, gptid, pptid);
                  aix_thread_info *priv = new aix_thread_info;
                  priv->pdtid = pbuf[pi].pdtid;
index f5161038c51d56227d30b929139232e45e46029f..f1ef37d17967f6520b02e5140ed2776ff3022395 100644 (file)
@@ -718,7 +718,7 @@ amd_dbgapi_target::stop (ptid_t ptid)
     {
       /* No need to iterate all non-exited threads if the request is to stop a
         specific thread.  */
-      stop_one_thread (find_thread_ptid (proc_target, ptid));
+      stop_one_thread (proc_target->find_thread (ptid));
       return;
     }
 
@@ -998,7 +998,7 @@ process_one_event (amd_dbgapi_event_id_t event_id,
            else
              ws.set_stopped (GDB_SIGNAL_0);
 
-           thread_info *thread = find_thread_ptid (proc_target, event_ptid);
+           thread_info *thread = proc_target->find_thread (event_ptid);
            if (thread == nullptr)
              {
                /* Silently create new GPU threads to avoid spamming the
index 8b08de8075f40dc78f2c19fc6b044d72c8eceeb6..d0094bdc2ca1d929743db2f9a7ef3fa9148fdf7f 100644 (file)
@@ -415,7 +415,7 @@ bsd_uthread_target::wait (ptid_t ptid, struct target_waitstatus *status,
     thread_change_ptid (beneath, inferior_ptid, ptid);
 
   /* Don't let the core see a ptid without a corresponding thread.  */
-  thread_info *thread = find_thread_ptid (beneath, ptid);
+  thread_info *thread = beneath->find_thread (ptid);
   if (thread == NULL || thread->state == THREAD_EXITED)
     add_thread (beneath, ptid);
 
@@ -466,7 +466,7 @@ bsd_uthread_target::update_thread_list ()
 
       process_stratum_target *proc_target
        = as_process_stratum_target (this->beneath ());
-      thread_info *thread = find_thread_ptid (proc_target, ptid);
+      thread_info *thread = proc_target->find_thread (ptid);
       if (thread == nullptr || thread->state == THREAD_EXITED)
        {
          /* If INFERIOR_PTID doesn't have a tid member yet, then ptid
index e864c35620ea57bacda34f7dad95e5248e2f21f8..4cf2d9f387a18914a320be292250d332b84f065c 100644 (file)
@@ -360,7 +360,7 @@ darwin_nat_target::check_new_threads (inferior *inf)
        {
          /* A thread was removed.  */
          struct thread_info *thr
-           = find_thread_ptid (this, ptid_t (inf->pid, 0, old_id));
+           = this->find_thread (ptid_t (inf->pid, 0, old_id));
          delete_thread (thr);
          kret = mach_port_deallocate (gdb_task, old_id);
          MACH_CHECK_ERROR (kret);
index 04d67fc5278d2861e1a36aa3e5be86f87ddeaf78..b0ca8b0069910d324ea5a9357b9e04be85cb0473 100644 (file)
@@ -1295,7 +1295,7 @@ fbsd_nat_target::wait_1 (ptid_t ptid, struct target_waitstatus *ourstatus,
                 threads might be skipped during post_attach that
                 have not yet reported their PL_FLAG_EXITED event.
                 Ignore EXITED events for an unknown LWP.  */
-             thread_info *thr = find_thread_ptid (this, wptid);
+             thread_info *thr = this->find_thread (wptid);
              if (thr != nullptr)
                {
                  fbsd_lwp_debug_printf ("deleting thread for LWP %u",
index 81e4afd24c25749caa8c3c38d23675a9b0884c7f..65aab9d12c32ca376a519e8322a7fbf3c9d16218 100644 (file)
@@ -672,10 +672,6 @@ extern bool in_thread_list (process_stratum_target *targ, ptid_t ptid);
    global id, not the system's).  */
 extern int valid_global_thread_id (int global_id);
 
-/* Search function to lookup a (non-exited) thread by 'ptid'.  */
-extern struct thread_info *find_thread_ptid (process_stratum_target *targ,
-                                            ptid_t ptid);
-
 /* Find thread by GDB global thread ID.  */
 struct thread_info *find_thread_global_id (int global_id);
 
index afaec46e3c15b1136ca9a8d127640a426bfedb2e..138905395813837da248813c49b0263469de7c75 100644 (file)
@@ -2186,8 +2186,7 @@ gnu_nat_target::attach (const char *args, int from_tty)
 
   inf_update_procs (inf);
 
-  thread_info *thr
-    = find_thread_ptid (this, ptid_t (pid, inf_pick_first_thread ()));
+  thread_info *thr = this->find_thread (ptid_t (pid, inf_pick_first_thread ()));
   switch_to_thread (thr);
 
   /* We have to initialize the terminal settings now, since the code
index a68611538f21884e9ff7a16457c59d3831c3f85f..103899432f765a76a861cc95390d84a37ba27c1b 100644 (file)
@@ -710,7 +710,7 @@ continue_command (const char *args, int from_tty)
          ptid_t last_ptid;
 
          get_last_target_status (&last_target, &last_ptid, nullptr);
-         tp = find_thread_ptid (last_target, last_ptid);
+         tp = last_target->find_thread (last_ptid);
        }
       if (tp != nullptr)
        bs = tp->control.stop_bpstat;
index 461855b2a7bd93804d6642cb19c9010be0f54eff..ba56fe37a789be1bf813e000cc4d5b7cdba24a2c 100644 (file)
@@ -876,7 +876,7 @@ follow_fork ()
            /* If we followed the child, switch to it...  */
            if (follow_child)
              {
-               tp = find_thread_ptid (parent_targ, child);
+               tp = parent_targ->find_thread (child);
                switch_to_thread (tp);
 
                /* ... and preserve the stepping state, in case the
@@ -5116,7 +5116,7 @@ handle_one (const wait_one_event &event)
        }
       else
        {
-         t = find_thread_ptid (event.target, event.ptid);
+         t = event.target->find_thread (event.ptid);
          /* Check if this is the first time we see this thread.
             Don't bother adding if it individually exited.  */
          if (t == nullptr
@@ -5137,7 +5137,7 @@ handle_one (const wait_one_event &event)
     }
   else
     {
-      thread_info *t = find_thread_ptid (event.target, event.ptid);
+      thread_info *t = event.target->find_thread (event.ptid);
       if (t == nullptr)
        t = add_thread (event.target, event.ptid);
 
@@ -5538,7 +5538,7 @@ handle_inferior_event (struct execution_control_state *ecs)
   if (ecs->ws.kind () != TARGET_WAITKIND_EXITED
       && ecs->ws.kind () != TARGET_WAITKIND_SIGNALLED)
     {
-      ecs->event_thread = find_thread_ptid (ecs->target, ecs->ptid);
+      ecs->event_thread = ecs->target->find_thread (ecs->ptid);
       /* If it's a new thread, add it to the thread database.  */
       if (ecs->event_thread == nullptr)
        ecs->event_thread = add_thread (ecs->target, ecs->ptid);
@@ -5687,7 +5687,7 @@ handle_inferior_event (struct execution_control_state *ecs)
           need to have access to the just-exited thread.  That is the
           case of GNU/Linux's "checkpoint" support, for example.
           Call the switch_to_xxx routine as appropriate.  */
-       thread_info *thr = find_thread_ptid (ecs->target, ecs->ptid);
+       thread_info *thr = ecs->target->find_thread (ecs->ptid);
        if (thr != nullptr)
          switch_to_thread (thr);
        else
@@ -5874,7 +5874,7 @@ handle_inferior_event (struct execution_control_state *ecs)
          /* Note that one of these may be an invalid pointer,
             depending on detach_fork.  */
          thread_info *parent = ecs->event_thread;
-         thread_info *child = find_thread_ptid (targ, ecs->ws.child_ptid ());
+         thread_info *child = targ->find_thread (ecs->ws.child_ptid ());
 
          /* At this point, the parent is marked running, and the
             child is marked stopped.  */
index a6e74c0cb529d6a707e4e763b905f3f051647e67..52e385411c7e656b51fca270c649e21620ebb98f 100644 (file)
@@ -529,7 +529,7 @@ Please switch to another checkpoint before deleting the current one"));
      list, waitpid the ptid.
      If fi->parent_ptid is a part of lwp and it is stopped, waitpid the
      ptid.  */
-  thread_info *parent = find_thread_ptid (linux_target, pptid);
+  thread_info *parent = linux_target->find_thread (pptid);
   if ((parent == NULL && find_fork_ptid (pptid))
       || (parent != NULL && parent->state == THREAD_STOPPED))
     {
index fd80fd975c144be9bcbb39971a711b00742f3ac4..fba08973e28390b44acc06755b6ebaf81af6cdc9 100644 (file)
@@ -903,7 +903,7 @@ linux_nat_switch_fork (ptid_t new_ptid)
 static void
 exit_lwp (struct lwp_info *lp)
 {
-  struct thread_info *th = find_thread_ptid (linux_target, lp->ptid);
+  struct thread_info *th = linux_target->find_thread (lp->ptid);
 
   if (th)
     {
@@ -1232,7 +1232,7 @@ get_detach_signal (struct lwp_info *lp)
     signo = gdb_signal_from_host (WSTOPSIG (lp->status));
   else
     {
-      struct thread_info *tp = find_thread_ptid (linux_target, lp->ptid);
+      thread_info *tp = linux_target->find_thread (lp->ptid);
 
       if (target_is_non_stop_p () && !tp->executing ())
        {
@@ -1325,7 +1325,7 @@ detach_one_lwp (struct lwp_info *lp, int *signo_p)
 
 
   /* Check in thread_info::pending_waitstatus.  */
-  thread_info *tp = find_thread_ptid (linux_target, lp->ptid);
+  thread_info *tp = linux_target->find_thread (lp->ptid);
   if (tp->has_pending_waitstatus ())
     {
       const target_waitstatus &ws = tp->pending_waitstatus ();
@@ -1579,7 +1579,7 @@ linux_nat_resume_callback (struct lwp_info *lp, struct lwp_info *except)
     {
       struct thread_info *thread;
 
-      thread = find_thread_ptid (linux_target, lp->ptid);
+      thread = linux_target->find_thread (lp->ptid);
       if (thread != NULL)
        {
          signo = thread->stop_signal ();
@@ -1717,7 +1717,7 @@ linux_handle_syscall_trap (struct lwp_info *lp, int stopping)
 {
   struct target_waitstatus *ourstatus = &lp->waitstatus;
   struct gdbarch *gdbarch = target_thread_architecture (lp->ptid);
-  thread_info *thread = find_thread_ptid (linux_target, lp->ptid);
+  thread_info *thread = linux_target->find_thread (lp->ptid);
   int syscall_number = (int) gdbarch_get_syscall_number (gdbarch, thread);
 
   if (stopping)
@@ -2996,7 +2996,7 @@ linux_nat_filter_event (int lwpid, int status)
       if (!lp->step
          && WSTOPSIG (status) && sigismember (&pass_mask, WSTOPSIG (status))
          && (WSTOPSIG (status) != SIGSTOP
-             || !find_thread_ptid (linux_target, lp->ptid)->stop_requested)
+             || !linux_target->find_thread (lp->ptid)->stop_requested)
          && !linux_wstatus_maybe_breakpoint (status))
        {
          linux_resume_one_lwp (lp, lp->step, signo);
@@ -4278,7 +4278,7 @@ linux_nat_stop_lwp (struct lwp_info *lwp)
 
       if (debug_linux_nat)
        {
-         if (find_thread_ptid (linux_target, lwp->ptid)->stop_requested)
+         if (linux_target->find_thread (lwp->ptid)->stop_requested)
            linux_nat_debug_printf ("already stopped/stop_requested %s",
                                    lwp->ptid.to_string ().c_str ());
          else
index ef1c2819229d9c0568768414d56ff5e9ea86f1ee..0e190f9426688588241f871b4b6078fa96e77303 100644 (file)
@@ -422,7 +422,7 @@ thread_from_lwp (thread_info *stopped, ptid_t ptid)
           thread_db_err_str (err));
 
   /* Fill the cache.  */
-  tp = find_thread_ptid (stopped->inf->process_target (), ptid);
+  tp = stopped->inf->process_target ()->find_thread (ptid);
   return record_thread (info, tp, ptid, &th, &ti);
 }
 \f
@@ -439,7 +439,7 @@ thread_db_notice_clone (ptid_t parent, ptid_t child)
   if (info == NULL)
     return 0;
 
-  thread_info *stopped = find_thread_ptid (linux_target, parent);
+  thread_info *stopped = linux_target->find_thread (parent);
 
   thread_from_lwp (stopped, child);
 
@@ -685,7 +685,7 @@ check_thread_db_callback (const td_thrhandle_t *th, void *arg)
      to how GDB accesses TLS could result in this passing
      without exercising the calls it's supposed to.  */
   ptid_t ptid = ptid_t (tdb_testinfo->info->pid, ti.ti_lid);
-  thread_info *thread_info = find_thread_ptid (linux_target, ptid);
+  thread_info *thread_info = linux_target->find_thread (ptid);
   if (thread_info != NULL && thread_info->priv != NULL)
     {
       LOG ("; errno");
@@ -1417,7 +1417,7 @@ thread_db_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
     return ptid;
 
   /* Fill in the thread's user-level thread id and status.  */
-  thread_from_lwp (find_thread_ptid (beneath, ptid), ptid);
+  thread_from_lwp (beneath->find_thread (ptid), ptid);
 
   return ptid;
 }
@@ -1511,7 +1511,7 @@ find_new_threads_callback (const td_thrhandle_t *th_p, void *data)
     }
 
   ptid_t ptid (info->pid, ti.ti_lid);
-  tp = find_thread_ptid (info->process_target, ptid);
+  tp = info->process_target->find_thread (ptid);
   if (tp == NULL || tp->priv == NULL)
     record_thread (info, tp, ptid, th_p, &ti);
 
@@ -1751,7 +1751,7 @@ thread_db_target::get_thread_local_address (ptid_t ptid,
   process_stratum_target *beneath
     = as_process_stratum_target (this->beneath ());
   /* Find the matching thread.  */
-  thread_info = find_thread_ptid (beneath, ptid);
+  thread_info = beneath->find_thread (ptid);
 
   /* We may not have discovered the thread yet.  */
   if (thread_info != NULL && thread_info->priv == NULL)
index e1244f3df43871832430c34592c7af4793b3bcaf..4f0bbd40f9d7ca30f0e0e774c99f3a5941c83e3d 100644 (file)
@@ -1015,7 +1015,7 @@ mi_on_resume (ptid_t ptid)
   if (ptid == minus_one_ptid || ptid.is_pid ())
     tp = inferior_thread ();
   else
-    tp = find_thread_ptid (target, ptid);
+    tp = target->find_thread (ptid);
 
   /* Suppress output while calling an inferior function.  */
   if (tp->control.in_infcall)
index 64f74c9405c98c72c3606587c27369ab6dccc748..86df0c8fe6a011ff18696e0e923cfd07326adc96 100644 (file)
@@ -618,7 +618,7 @@ nbsd_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
         threads might be skipped during post_attach that
         have not yet reported their PTRACE_LWP_EXIT event.
         Ignore exited events for an unknown LWP.  */
-      thread_info *thr = find_thread_ptid (this, wptid);
+      thread_info *thr = this->find_thread (wptid);
       if (thr == nullptr)
          ourstatus->set_spurious ();
       else
index 8e40efa31fcf0e85923ab2eb529ec34654acd915..61c18fdc8254440b66cd04d2f592b9498e257f5f 100644 (file)
@@ -409,7 +409,7 @@ nto_procfs_target::update_thread_list ()
           (e.g. thread exited).  */
        continue;
       ptid = ptid_t (pid, 0, tid);
-      new_thread = find_thread_ptid (this, ptid);
+      new_thread = this->find_thread (ptid);
       if (!new_thread)
        new_thread = add_thread (ptid);
       update_thread_private_data (new_thread, tid, status.state, 0);
@@ -713,7 +713,7 @@ nto_procfs_target::attach (const char *args, int from_tty)
 
   update_thread_list ();
 
-  switch_to_thread (find_thread_ptid (this, ptid));
+  switch_to_thread (this->find_thread (ptid));
 }
 
 void
@@ -1282,7 +1282,7 @@ nto_procfs_target::create_inferior (const char *exec_file,
 
   ptid_t ptid = do_attach (ptid_t (pid));
   update_thread_list ();
-  switch_to_thread (find_thread_ptid (this, ptid));
+  switch_to_thread (this->find_thread (ptid));
 
   inf = current_inferior ();
   inferior_appeared (inf, pid);
index e1d41c86b9c30b110b2a92af8da4513f8178f129..5c031203e89e2c7b842c50966f9a9cba70dfdace 100644 (file)
@@ -199,6 +199,17 @@ process_stratum_target::random_resumed_with_pending_wait_status
 
 /* See process-stratum-target.h.  */
 
+thread_info *
+process_stratum_target::find_thread (ptid_t ptid)
+{
+  inferior *inf = find_inferior_ptid (this, ptid);
+  if (inf == NULL)
+    return NULL;
+  return inf->find_thread (ptid);
+}
+
+/* See process-stratum-target.h.  */
+
 std::set<process_stratum_target *>
 all_non_exited_process_targets ()
 {
index dcbca5a9c6adc52fd7ecb0b2f0186dd118e639c4..35cfd9c1cca6a8d0c4f84d59c26ade6a2f16fbe0 100644 (file)
@@ -108,6 +108,9 @@ public:
   thread_info *random_resumed_with_pending_wait_status
     (inferior *inf, ptid_t filter_ptid);
 
+  /* Search function to lookup a (non-exited) thread by 'ptid'.  */
+  thread_info *find_thread (ptid_t ptid);
+
   /* The connection number.  Visible in "info connections".  */
   int connection_number = 0;
 
index 741e62a24026e5fb75e40931ba1760229f116ecf..7990640963b7486c74744c19f6c1fb929adae479 100644 (file)
@@ -2118,7 +2118,7 @@ wait_again:
                    if (print_thread_events)
                      gdb_printf (_("[%s exited]\n"),
                                  target_pid_to_str (retval).c_str ());
-                   delete_thread (find_thread_ptid (this, retval));
+                   delete_thread (this->find_thread (retval));
                    target_continue_no_signal (ptid);
                    goto wait_again;
                  }
@@ -2225,7 +2225,7 @@ wait_again:
                    if (print_thread_events)
                      gdb_printf (_("[%s exited]\n"),
                                  target_pid_to_str (retval).c_str ());
-                   delete_thread (find_thread_ptid (this, retval));
+                   delete_thread (this->find_thread (retval));
                    status->set_spurious ();
                    return retval;
                  }
@@ -2854,7 +2854,7 @@ procfs_notice_thread (procinfo *pi, procinfo *thread, void *ptr)
 {
   ptid_t gdb_threadid = ptid_t (pi->pid, thread->tid, 0);
 
-  thread_info *thr = find_thread_ptid (&the_procfs_target, gdb_threadid);
+  thread_info *thr = the_procfs_target.find_thread (gdb_threadid);
   if (thr == NULL || thr->state == THREAD_EXITED)
     add_thread (&the_procfs_target, gdb_threadid);
 
index ad357352eca86551cb4d856f34a7715e6c604f0a..520161338895bc5f9a1b7f27cdd5c9a013f1ecf4 100644 (file)
@@ -127,7 +127,7 @@ struct ravenscar_thread_target final : public target_ops
     process_stratum_target *proc_target
       = as_process_stratum_target (this->beneath ());
     ptid_t underlying = get_base_thread_from_ravenscar_task (tp->ptid);
-    tp = find_thread_ptid (proc_target, underlying);
+    tp = proc_target->find_thread (underlying);
 
     return beneath ()->enable_btrace (tp, conf);
   }
@@ -161,7 +161,7 @@ private:
     process_stratum_target *proc_target
       = as_process_stratum_target (this->beneath ());
     ptid_t underlying = get_base_thread_from_ravenscar_task (ptid);
-    switch_to_thread (find_thread_ptid (proc_target, underlying));
+    switch_to_thread (proc_target->find_thread (underlying));
   }
 
   /* Some targets use lazy FPU initialization.  On these, the FP
@@ -312,7 +312,7 @@ ravenscar_thread_target::add_active_thread ()
   /* The running thread may not have been added to
      system.tasking.debug's list yet; so ravenscar_update_thread_list
      may not always add it to the thread list.  Add it here.  */
-  thread_info *active_thr = find_thread_ptid (proc_target, active_ptid);
+  thread_info *active_thr = proc_target->find_thread (active_ptid);
   if (active_thr == nullptr)
     {
       active_thr = ::add_thread (proc_target, active_ptid);
index 2d88e4d20bf6fb61bdbdfcaacd93fc55fc57941a..25ae66311520598eaa5534beeaff190c90913958 100644 (file)
@@ -1396,7 +1396,7 @@ enum record_method
 record_btrace_target::record_method (ptid_t ptid)
 {
   process_stratum_target *proc_target = current_inferior ()->process_target ();
-  thread_info *const tp = find_thread_ptid (proc_target, ptid);
+  thread_info *const tp = proc_target->find_thread (ptid);
 
   if (tp == NULL)
     error (_("No thread."));
@@ -1548,7 +1548,7 @@ record_btrace_target::fetch_registers (struct regcache *regcache, int regno)
   /* Thread-db may ask for a thread's registers before GDB knows about the
      thread.  We forward the request to the target beneath in this
      case.  */
-  thread_info *tp = find_thread_ptid (regcache->target (), regcache->ptid ());
+  thread_info *tp = regcache->target ()->find_thread (regcache->ptid ());
   if (tp != nullptr)
     replay =  tp->btrace.replay;
 
index 526df313ea73b419fa450fed1f06e22e6fe75bd3..73deea3690b31157dc04b1463bae0a2a041ecfe3 100644 (file)
@@ -2783,7 +2783,7 @@ remote_target::remote_notice_new_inferior (ptid_t currthread, bool executing)
   /* If this is a new thread, add it to GDB's thread list.
      If we leave it up to WFI to do this, bad things will happen.  */
 
-  thread_info *tp = find_thread_ptid (this, currthread);
+  thread_info *tp = this->find_thread (currthread);
   if (tp != NULL && tp->state == THREAD_EXITED)
     {
       /* We're seeing an event on a thread id we knew had exited.
@@ -2875,7 +2875,7 @@ get_remote_thread_info (thread_info *thread)
 static remote_thread_info *
 get_remote_thread_info (remote_target *target, ptid_t ptid)
 {
-  thread_info *thr = find_thread_ptid (target, ptid);
+  thread_info *thr = target->find_thread (ptid);
   return get_remote_thread_info (thr);
 }
 
@@ -4175,7 +4175,7 @@ remote_target::update_thread_list ()
 
              remote_notice_new_inferior (item.ptid, executing);
 
-             thread_info *tp = find_thread_ptid (this, item.ptid);
+             thread_info *tp = this->find_thread (item.ptid);
              remote_thread_info *info = get_remote_thread_info (tp);
              info->core = item.core;
              info->extra = std::move (item.extra);
@@ -4771,7 +4771,7 @@ remote_target::process_initial_stop_replies (int from_tty)
       if (ignore_event)
        continue;
 
-      thread_info *evthread = find_thread_ptid (this, event_ptid);
+      thread_info *evthread = this->find_thread (event_ptid);
 
       if (ws.kind () == TARGET_WAITKIND_STOPPED)
        {
@@ -5123,7 +5123,7 @@ remote_target::start_remote_1 (int from_tty, int extended_p)
                }
            }
          else
-           switch_to_thread (find_thread_ptid (this, curr_thread));
+           switch_to_thread (this->find_thread (curr_thread));
        }
 
       /* init_wait_for_inferior should be called before get_offsets in order
@@ -6192,7 +6192,7 @@ remote_target::remote_detach_1 (inferior *inf, int from_tty)
       remote_detach_pid (reply->ws.child_ptid ().pid ());
     }
 
-  thread_info *tp = find_thread_ptid (this, inferior_ptid);
+  thread_info *tp = this->find_thread (inferior_ptid);
 
   /* Check to see if we are detaching a fork parent.  Note that if we
      are detaching a fork child, tp == NULL.  */
@@ -6502,10 +6502,10 @@ remote_target::append_resumption (char *p, char *endp,
        {
          /* If we don't know about the target thread's tid, then
             we're resuming magic_null_ptid (see caller).  */
-         tp = find_thread_ptid (this, magic_null_ptid);
+         tp = this->find_thread (magic_null_ptid);
        }
       else
-       tp = find_thread_ptid (this, ptid);
+       tp = this->find_thread (ptid);
       gdb_assert (tp != NULL);
 
       if (tp->control.may_range_step)
@@ -13964,7 +13964,7 @@ remote_target::set_disconnected_tracing (int val)
 int
 remote_target::core_of_thread (ptid_t ptid)
 {
-  thread_info *info = find_thread_ptid (this, ptid);
+  thread_info *info = this->find_thread (ptid);
 
   if (info != NULL && info->priv != NULL)
     return get_remote_thread_info (info)->core;
index 9fc8c8a1b13665f6bbf32c8a09ee2a20d19dd1dd..f61ce7fb33ce97360d3afb4c8b4c46b4458ce6dc 100644 (file)
@@ -122,7 +122,7 @@ all_matching_threads_iterator::all_matching_threads_iterator
          /* Iterate on a single thread.  */
          m_mode = mode::SINGLE_THREAD;
 
-         m_thr = find_thread_ptid (filter_target, filter_ptid);
+         m_thr = filter_target->find_thread (filter_ptid);
        }
     }
 }
index 38afdff616a2a6b691882a951f2e910c3a63a6df..1c664bc3ab55b952cf3d484c789721ec387c3b75 100644 (file)
@@ -514,17 +514,6 @@ find_thread_id (struct inferior *inf, int thr_num)
 
 /* See gdbthread.h.  */
 
-struct thread_info *
-find_thread_ptid (process_stratum_target *targ, ptid_t ptid)
-{
-  inferior *inf = find_inferior_ptid (targ, ptid);
-  if (inf == NULL)
-    return NULL;
-  return inf->find_thread (ptid);
-}
-
-/* See gdbthread.h.  */
-
 struct thread_info *
 find_thread_by_handle (gdb::array_view<const gdb_byte> handle,
                       struct inferior *inf)
@@ -598,7 +587,7 @@ valid_global_thread_id (int global_id)
 bool
 in_thread_list (process_stratum_target *targ, ptid_t ptid)
 {
-  return find_thread_ptid (targ, ptid) != nullptr;
+  return targ->find_thread (ptid) != nullptr;
 }
 
 /* Finds the first thread of the inferior.  */
@@ -1333,7 +1322,7 @@ switch_to_thread (thread_info *thr)
 void
 switch_to_thread (process_stratum_target *proc_target, ptid_t ptid)
 {
-  thread_info *thr = find_thread_ptid (proc_target, ptid);
+  thread_info *thr = proc_target->find_thread (ptid);
   switch_to_thread (thr);
 }
 
index 26ad04b27be3415e9c7878cf2663a50eb883f509..c09e459a80197a3a35c5607c723912434f826cfa 100644 (file)
@@ -626,7 +626,7 @@ windows_nat_target::delete_thread (ptid_t ptid, DWORD exit_code,
                target_pid_to_str (ptid).c_str (),
                (unsigned) exit_code);
 
-  ::delete_thread (find_thread_ptid (this, ptid));
+  ::delete_thread (this->find_thread (ptid));
 
   auto iter = std::find_if (windows_process.thread_list.begin (),
                            windows_process.thread_list.end (),
@@ -1944,7 +1944,7 @@ windows_nat_target::do_initial_windows_stuff (DWORD pid, bool attaching)
       this->resume (minus_one_ptid, 0, GDB_SIGNAL_0);
     }
 
-  switch_to_thread (find_thread_ptid (this, last_ptid));
+  switch_to_thread (this->find_thread (last_ptid));
 
   /* Now that the inferior has been started and all DLLs have been mapped,
      we can iterate over all DLLs and load them in.