From: Doug Evans <dje@google.com>
To: gdb-patches@sourceware.org
Subject: Re: [RFA] rename find_thread_pid to find_thread_ptid
Date: Mon, 18 May 2009 23:21:00 -0000 [thread overview]
Message-ID: <e394668d0905181621m2654c31flba1256fb8fd26256@mail.gmail.com> (raw)
In-Reply-To: <20090501225121.1C38B84890@localhost>
Ping.
Joel's comment was to rename to find_thread_from_ptid.
I think that's great, but there's a lot more to be renamed along those
lines than what this patch does, so I went with consistency with the
current usage in gdb and just fixed the pid->ptid bug.
Ok to check in?
On Fri, May 1, 2009 at 3:51 PM, Doug Evans <dje@google.com> wrote:
> Hi.
>
> Anyone mind a global renaming of find_thread_pid to find_thread_ptid?
>
> Tested on x86_64-linux only, but the global search-and-replace
> was mechanical, not hand-done.
>
>
> 2009-05-01 Doug Evans <dje@google.com>
>
> Global renaming of find_thread_pid to find_thread_ptid.
> * gdbthread.h (find_thread_ptid): Renamed from find_thread_pid.
> * thread.c (find_thread_ptid): Renamed from find_thread_pid.
> (inferior_thread, add_thread_silent, thread_change_ptid): Update.
> (set_running, is_thread_state, is_executing): Update.
> (set_executing, set_stop_requested, finish_thread_state): Update.
> (do_restore_current_thread_cleanup): Update.
> (restore_current_thread_cleanup_dtor): Update.
> (make_cleanup_restore_current_thread): Update.
> * aix-thread.c (aix_thread_resume): Update.
> (aix_thread_fetch_registers, aix_thread_store_registers): Update.
> * exceptions.c (throw_exception): Update.
> * inf-ptrace.c (inf_ptrace_follow_fork): Update.
> * inf-ttrace.c (inf_ttrace_follow_fork): Update.
> (inf_ttrace_resume, inf_ttrace_wait): Update.
> * infcall.c (run_inferior_call): Update.
> * infcmd.c (continue_command): Update.
> (program_info): Update.
> * infrun.c (proceed): Update.
> (infrun_thread_stop_requested_callback): Update.
> (handle_inferior_event): Update.
> * linux-nat.c (linux_child_follow_fork): Update.
> (linux_nat_switch_fork, get_pending_status): Update.
> (linux_nat_stop_lwp): Update.
> * linux-thread-db.c (thread_get_info_callback): Update.
> (attach_thread, detach_thread, find_new_threads_callback): Update.
> (thread_db_find_new_threads): Update.
> (thread_db_get_thread_local_address): Update.
> * mi/mi-interp.c (mi_on_resume): Update.
>
> In gdbserver.
> Global renaming of find_thread_pid to find_thread_ptid.
> * server.h (find_thread_ptid): Renamed from find_thread_pid.
> * inferiors.c (find_thread_ptid): Renamed from find_thread_pid.
> (gdb_id_to_thread_id): Update.
> * remote-utils.c (prepare_resume_reply): Update.
> * server.c (handle_query): Update.
> * target.c (set_desired_inferior): Update.
>
> Index: aix-thread.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/aix-thread.c,v
> retrieving revision 1.63
> diff -u -p -r1.63 aix-thread.c
> --- aix-thread.c 15 Mar 2009 21:05:41 -0000 1.63
> +++ aix-thread.c 1 May 2009 22:24:06 -0000
> @@ -972,7 +972,7 @@ aix_thread_resume (struct target_ops *op
> }
> else
> {
> - thread = find_thread_pid (ptid);
> + thread = find_thread_ptid (ptid);
> if (!thread)
> error (_("aix-thread resume: unknown pthread %ld"),
> TIDGET (ptid));
> @@ -1288,7 +1288,7 @@ aix_thread_fetch_registers (struct targe
> beneath->to_fetch_registers (beneath, regcache, regno);
> else
> {
> - thread = find_thread_pid (inferior_ptid);
> + thread = find_thread_ptid (inferior_ptid);
> tid = thread->private->tid;
>
> if (tid == PTHDB_INVALID_TID)
> @@ -1629,7 +1629,7 @@ aix_thread_store_registers (struct targe
> beneath->to_store_registers (beneath, regcache, regno);
> else
> {
> - thread = find_thread_pid (inferior_ptid);
> + thread = find_thread_ptid (inferior_ptid);
> tid = thread->private->tid;
>
> if (tid == PTHDB_INVALID_TID)
> Index: exceptions.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/exceptions.c,v
> retrieving revision 1.33
> diff -u -p -r1.33 exceptions.c
> --- exceptions.c 13 Mar 2009 17:26:59 -0000 1.33
> +++ exceptions.c 1 May 2009 22:24:06 -0000
> @@ -219,7 +219,7 @@ throw_exception (struct gdb_exception ex
> immediate_quit = 0;
>
> if (!ptid_equal (inferior_ptid, null_ptid))
> - tp = find_thread_pid (inferior_ptid);
> + tp = find_thread_ptid (inferior_ptid);
>
> /* Perhaps it would be cleaner to do this via the cleanup chain (not sure
> I can think of a reason why that is vital, though). */
> Index: gdbthread.h
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbthread.h,v
> retrieving revision 1.47
> diff -u -p -r1.47 gdbthread.h
> --- gdbthread.h 31 Mar 2009 15:23:57 -0000 1.47
> +++ gdbthread.h 1 May 2009 22:24:07 -0000
> @@ -224,7 +224,7 @@ extern int in_thread_list (ptid_t ptid);
> extern int valid_thread_id (int thread);
>
> /* Search function to lookup a thread by 'pid'. */
> -extern struct thread_info *find_thread_pid (ptid_t ptid);
> +extern struct thread_info *find_thread_ptid (ptid_t ptid);
>
> /* Find thread by GDB user-visible thread number. */
> struct thread_info *find_thread_id (int num);
> Index: inf-ptrace.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/inf-ptrace.c,v
> retrieving revision 1.59
> diff -u -p -r1.59 inf-ptrace.c
> --- inf-ptrace.c 17 Mar 2009 19:28:09 -0000 1.59
> +++ inf-ptrace.c 1 May 2009 22:24:07 -0000
> @@ -57,7 +57,7 @@ inf_ptrace_follow_fork (struct target_op
> gdb_assert (status.kind == TARGET_WAITKIND_FORKED);
>
> pid = ptid_get_pid (ptid);
> - last_tp = find_thread_pid (ptid);
> + last_tp = find_thread_ptid (ptid);
> }
>
> if (ptrace (PT_GET_PROCESS_STATE, pid,
> Index: inf-ttrace.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/inf-ttrace.c,v
> retrieving revision 1.44
> diff -u -p -r1.44 inf-ttrace.c
> --- inf-ttrace.c 17 Mar 2009 19:28:09 -0000 1.44
> +++ inf-ttrace.c 1 May 2009 22:24:07 -0000
> @@ -428,7 +428,7 @@ inf_ttrace_follow_fork (struct target_op
>
> pid = ptid_get_pid (ptid);
> lwpid = ptid_get_lwp (ptid);
> - last_tp = find_thread_pid (ptid);
> + last_tp = find_thread_ptid (ptid);
> }
>
> /* Get all important details that core GDB doesn't (and shouldn't)
> @@ -909,7 +909,7 @@ inf_ttrace_resume (struct target_ops *op
> if (resume_all)
> ptid = inferior_ptid;
>
> - info = find_thread_pid (ptid);
> + info = find_thread_ptid (ptid);
> inf_ttrace_resume_lwp (info, request, sig);
>
> if (resume_all)
> @@ -982,7 +982,7 @@ inf_ttrace_wait (struct target_ops *ops,
>
> /* We haven't set the private member on the main thread yet. Do
> it now. */
> - ti = find_thread_pid (inferior_ptid);
> + ti = find_thread_ptid (inferior_ptid);
> gdb_assert (ti != NULL && ti->private == NULL);
> ti->private =
> xmalloc (sizeof (struct inf_ttrace_private_thread_info));
> @@ -1083,7 +1083,7 @@ inf_ttrace_wait (struct target_ops *ops,
> case TTEVT_LWP_EXIT:
> if (print_thread_events)
> printf_unfiltered (_("[%s exited]\n"), target_pid_to_str (ptid));
> - ti = find_thread_pid (ptid);
> + ti = find_thread_ptid (ptid);
> gdb_assert (ti != NULL);
> ((struct inf_ttrace_private_thread_info *)ti->private)->dying = 1;
> inf_ttrace_num_lwps--;
> @@ -1100,7 +1100,7 @@ inf_ttrace_wait (struct target_ops *ops,
> if (print_thread_events)
> printf_unfiltered(_("[%s has been terminated]\n"),
> target_pid_to_str (ptid));
> - ti = find_thread_pid (ptid);
> + ti = find_thread_ptid (ptid);
> gdb_assert (ti != NULL);
> ((struct inf_ttrace_private_thread_info *)ti->private)->dying = 1;
> inf_ttrace_num_lwps--;
> Index: infcall.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/infcall.c,v
> retrieving revision 1.111
> diff -u -p -r1.111 infcall.c
> --- infcall.c 22 Mar 2009 17:59:59 -0000 1.111
> +++ infcall.c 1 May 2009 22:24:07 -0000
> @@ -349,7 +349,7 @@ run_inferior_call (struct thread_info *c
>
> /* At this point the current thread may have changed. Refresh
> CALL_THREAD as it could be invalid if its thread has exited. */
> - call_thread = find_thread_pid (call_thread_ptid);
> + call_thread = find_thread_ptid (call_thread_ptid);
>
> /* Don't restore the async mask if the target has changed,
> saved_async is for the original target. */
> Index: infcmd.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/infcmd.c,v
> retrieving revision 1.238
> diff -u -p -r1.238 infcmd.c
> --- infcmd.c 25 Mar 2009 21:42:34 -0000 1.238
> +++ infcmd.c 1 May 2009 22:24:07 -0000
> @@ -706,14 +706,14 @@ Can't resume all threads and specify pro
> struct thread_info *tp;
>
> if (non_stop)
> - tp = find_thread_pid (inferior_ptid);
> + tp = find_thread_ptid (inferior_ptid);
> else
> {
> ptid_t last_ptid;
> struct target_waitstatus ws;
>
> get_last_target_status (&last_ptid, &ws);
> - tp = find_thread_pid (last_ptid);
> + tp = find_thread_ptid (last_ptid);
> }
> if (tp != NULL)
> bs = tp->stop_bpstat;
> @@ -1580,7 +1580,7 @@ program_info (char *args, int from_tty)
> else if (is_running (ptid))
> error (_("Selected thread is running."));
>
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> bs = tp->stop_bpstat;
> stat = bpstat_num (&bs, &num);
>
> Index: infrun.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/infrun.c,v
> retrieving revision 1.370
> diff -u -p -r1.370 infrun.c
> --- infrun.c 30 Apr 2009 03:09:59 -0000 1.370
> +++ infrun.c 1 May 2009 22:24:07 -0000
> @@ -1413,7 +1413,7 @@ proceed (CORE_ADDR addr, enum target_sig
> && !ptid_equal (last_ptid, null_ptid)
> && !ptid_equal (last_ptid, minus_one_ptid))
> {
> - last_thread = find_thread_pid (last_ptid);
> + last_thread = find_thread_ptid (last_ptid);
> if (last_thread)
> {
> tp->stop_signal = last_thread->stop_signal;
> @@ -1645,7 +1645,7 @@ infrun_thread_stop_requested_callback (s
> have consistent output as if the stop event had been
> reported. */
> ecs->ptid = info->ptid;
> - ecs->event_thread = find_thread_pid (info->ptid);
> + ecs->event_thread = find_thread_ptid (info->ptid);
> ecs->ws.kind = TARGET_WAITKIND_STOPPED;
> ecs->ws.value.sig = TARGET_SIGNAL_0;
>
> @@ -2227,7 +2227,7 @@ handle_inferior_event (struct execution_
> && ecs->ws.kind != TARGET_WAITKIND_SIGNALLED && ecs->new_thread_event)
> add_thread (ecs->ptid);
>
> - ecs->event_thread = find_thread_pid (ecs->ptid);
> + ecs->event_thread = find_thread_ptid (ecs->ptid);
>
> /* Dependent on valid ECS->EVENT_THREAD. */
> adjust_pc_after_break (ecs);
> @@ -2720,7 +2720,7 @@ targets should add new threads to the th
> stop_signal = ecs->event_thread->stop_signal;
> ecs->event_thread->stop_signal = TARGET_SIGNAL_0;
> ecs->ptid = singlestep_ptid;
> - ecs->event_thread = find_thread_pid (ecs->ptid);
> + ecs->event_thread = find_thread_ptid (ecs->ptid);
> ecs->event_thread->stop_signal = stop_signal;
> stop_pc = new_singlestep_pc;
> }
> Index: linux-nat.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/linux-nat.c,v
> retrieving revision 1.128
> diff -u -p -r1.128 linux-nat.c
> --- linux-nat.c 1 Apr 2009 19:07:50 -0000 1.128
> +++ linux-nat.c 1 May 2009 22:24:07 -0000
> @@ -809,7 +809,7 @@ linux_child_follow_fork (struct target_o
> }
> else
> {
> - struct thread_info *last_tp = find_thread_pid (last_ptid);
> + struct thread_info *last_tp = find_thread_ptid (last_ptid);
> struct thread_info *tp;
> char child_pid_spelling[40];
> struct inferior *parent_inf, *child_inf;
> @@ -1154,7 +1154,7 @@ linux_nat_switch_fork (ptid_t new_ptid)
> static void
> exit_lwp (struct lwp_info *lp)
> {
> - struct thread_info *th = find_thread_pid (lp->ptid);
> + struct thread_info *th = find_thread_ptid (lp->ptid);
>
> if (th)
> {
> @@ -1505,7 +1505,7 @@ get_pending_status (struct lwp_info *lp,
> have the last signal recorded in
> thread_info->stop_signal. */
>
> - struct thread_info *tp = find_thread_pid (lp->ptid);
> + struct thread_info *tp = find_thread_ptid (lp->ptid);
> signo = tp->stop_signal;
> }
>
> @@ -1534,7 +1534,7 @@ GPT: lwp %s had signal %s, but it is in
> {
> if (GET_LWP (lp->ptid) == GET_LWP (last_ptid))
> {
> - struct thread_info *tp = find_thread_pid (lp->ptid);
> + struct thread_info *tp = find_thread_ptid (lp->ptid);
> if (tp->stop_signal != TARGET_SIGNAL_0
> && signal_pass_state (tp->stop_signal))
> *status = W_STOPCODE (target_signal_to_host (tp->stop_signal));
> @@ -4694,7 +4694,7 @@ linux_nat_stop_lwp (struct lwp_info *lwp
>
> if (debug_linux_nat)
> {
> - if (find_thread_pid (lwp->ptid)->stop_requested)
> + if (find_thread_ptid (lwp->ptid)->stop_requested)
> fprintf_unfiltered (gdb_stdlog, "\
> LNSL: already stopped/stop_requested %s\n",
> target_pid_to_str (lwp->ptid));
> Index: linux-thread-db.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/linux-thread-db.c,v
> retrieving revision 1.55
> diff -u -p -r1.55 linux-thread-db.c
> --- linux-thread-db.c 1 May 2009 22:05:46 -0000 1.55
> +++ linux-thread-db.c 1 May 2009 22:24:07 -0000
> @@ -266,7 +266,7 @@ thread_get_info_callback (const td_thrha
>
> /* Fill the cache. */
> thread_ptid = ptid_build (GET_PID (proc_handle.ptid), ti.ti_lid, 0);
> - thread_info = find_thread_pid (thread_ptid);
> + thread_info = find_thread_ptid (thread_ptid);
>
> /* In the case of a zombie thread, don't continue. We don't want to
> attach to it thinking it is a new thread. */
> @@ -284,7 +284,7 @@ thread_get_info_callback (const td_thrha
> thread_db_find_new_threads_1 ();
> else
> attach_thread (thread_ptid, thp, &ti);
> - thread_info = find_thread_pid (thread_ptid);
> + thread_info = find_thread_ptid (thread_ptid);
> gdb_assert (thread_info != NULL);
> }
>
> @@ -696,7 +696,7 @@ attach_thread (ptid_t ptid, const td_thr
> thread and attach to the new one. */
> if (in_thread_list (ptid))
> {
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp != NULL);
>
> /* If tp->private is NULL, then GDB is already attached to this
> @@ -765,7 +765,7 @@ detach_thread (ptid_t ptid)
> until we notice that it's dead (via prune_threads), or until
> something re-uses its thread ID. We'll report the thread exit
> when the underlying LWP dies. */
> - thread_info = find_thread_pid (ptid);
> + thread_info = find_thread_ptid (ptid);
> gdb_assert (thread_info != NULL && thread_info->private != NULL);
> thread_info->private->dying = 1;
> }
> @@ -981,7 +981,7 @@ find_new_threads_callback (const td_thrh
> return 0;
> }
>
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> if (tp == NULL || tp->private == NULL)
> attach_thread (ptid, th_p, &ti);
>
> @@ -1026,7 +1026,7 @@ thread_db_find_new_threads (struct targe
> static char *
> thread_db_pid_to_str (struct target_ops *ops, ptid_t ptid)
> {
> - struct thread_info *thread_info = find_thread_pid (ptid);
> + struct thread_info *thread_info = find_thread_ptid (ptid);
> struct target_ops *beneath;
>
> if (thread_info != NULL && thread_info->private != NULL)
> @@ -1080,7 +1080,7 @@ thread_db_get_thread_local_address (stru
> thread_db_find_new_threads_1 ();
>
> /* Find the matching thread. */
> - thread_info = find_thread_pid (ptid);
> + thread_info = find_thread_ptid (ptid);
>
> if (thread_info != NULL && thread_info->private != NULL)
> {
> Index: thread.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/thread.c,v
> retrieving revision 1.109
> diff -u -p -r1.109 thread.c
> --- thread.c 31 Mar 2009 15:23:57 -0000 1.109
> +++ thread.c 1 May 2009 22:24:07 -0000
> @@ -75,7 +75,7 @@ enum thread_state
> struct thread_info*
> inferior_thread (void)
> {
> - struct thread_info *tp = find_thread_pid (inferior_ptid);
> + struct thread_info *tp = find_thread_ptid (inferior_ptid);
> gdb_assert (tp);
> return tp;
> }
> @@ -146,7 +146,7 @@ add_thread_silent (ptid_t ptid)
> {
> struct thread_info *tp;
>
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> if (tp)
> /* Found an old thread with the same id. It has to be dead,
> otherwise we wouldn't be adding a new thread with the same id.
> @@ -302,7 +302,7 @@ find_thread_id (int num)
>
> /* Find a thread_info by matching PTID. */
> struct thread_info *
> -find_thread_pid (ptid_t ptid)
> +find_thread_ptid (ptid_t ptid)
> {
> struct thread_info *tp;
>
> @@ -499,7 +499,7 @@ thread_change_ptid (ptid_t old_ptid, pti
> inf = find_inferior_pid (ptid_get_pid (old_ptid));
> inf->pid = ptid_get_pid (new_ptid);
>
> - tp = find_thread_pid (old_ptid);
> + tp = find_thread_ptid (old_ptid);
> tp->ptid = new_ptid;
>
> observer_notify_thread_ptid_changed (old_ptid, new_ptid);
> @@ -530,7 +530,7 @@ set_running (ptid_t ptid, int running)
> else
> {
> int started = 0;
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp);
> gdb_assert (tp->state_ != THREAD_EXITED);
> if (running && tp->state_ == THREAD_STOPPED)
> @@ -549,7 +549,7 @@ is_thread_state (ptid_t ptid, enum threa
> if (!target_has_execution)
> return 0;
>
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp);
> return tp->state_ == state;
> }
> @@ -607,7 +607,7 @@ is_executing (ptid_t ptid)
> if (!target_has_execution)
> return 0;
>
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp);
> return tp->executing_;
> }
> @@ -624,7 +624,7 @@ set_executing (ptid_t ptid, int executin
> }
> else
> {
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp);
> tp->executing_ = executing;
> }
> @@ -644,7 +644,7 @@ set_stop_requested (ptid_t ptid, int sto
> }
> else
> {
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp);
> tp->stop_requested = stop;
> }
> @@ -680,7 +680,7 @@ finish_thread_state (ptid_t ptid)
> }
> else
> {
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
> gdb_assert (tp);
> if (tp->state_ != THREAD_EXITED)
> {
> @@ -933,7 +933,7 @@ do_restore_current_thread_cleanup (void
> struct thread_info *tp;
> struct current_thread_cleanup *old = arg;
>
> - tp = find_thread_pid (old->inferior_ptid);
> + tp = find_thread_ptid (old->inferior_ptid);
>
> /* If the previously selected thread belonged to a process that has
> in the mean time been deleted (due to normal exit, detach, etc.),
> @@ -962,7 +962,7 @@ restore_current_thread_cleanup_dtor (voi
> {
> struct current_thread_cleanup *old = arg;
> struct thread_info *tp;
> - tp = find_thread_pid (old->inferior_ptid);
> + tp = find_thread_ptid (old->inferior_ptid);
> if (tp)
> tp->refcount--;
> xfree (old);
> @@ -992,7 +992,7 @@ make_cleanup_restore_current_thread (voi
> old->selected_frame_id = get_frame_id (frame);
> old->selected_frame_level = frame_relative_level (frame);
>
> - tp = find_thread_pid (inferior_ptid);
> + tp = find_thread_ptid (inferior_ptid);
> if (tp)
> tp->refcount++;
> }
> Index: mi/mi-interp.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/mi/mi-interp.c,v
> retrieving revision 1.46
> diff -u -p -r1.46 mi-interp.c
> --- mi/mi-interp.c 25 Mar 2009 21:53:11 -0000 1.46
> +++ mi/mi-interp.c 1 May 2009 22:24:07 -0000
> @@ -397,7 +397,7 @@ mi_on_resume (ptid_t ptid)
> if (ptid_equal (ptid, minus_one_ptid))
> tp = inferior_thread ();
> else
> - tp = find_thread_pid (ptid);
> + tp = find_thread_ptid (ptid);
>
> /* Suppress output while calling an inferior function. */
> if (tp->in_infcall)
> @@ -428,7 +428,7 @@ mi_on_resume (ptid_t ptid)
> }
> else
> {
> - struct thread_info *ti = find_thread_pid (ptid);
> + struct thread_info *ti = find_thread_ptid (ptid);
> gdb_assert (ti);
> fprintf_unfiltered (raw_stdout, "*running,thread-id=\"%d\"\n", ti->num);
> }
> Index: gdbserver/inferiors.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/inferiors.c,v
> retrieving revision 1.22
> diff -u -p -r1.22 inferiors.c
> --- gdbserver/inferiors.c 30 Apr 2009 22:21:06 -0000 1.22
> +++ gdbserver/inferiors.c 1 May 2009 22:24:07 -0000
> @@ -210,7 +210,7 @@ thread_to_gdb_id (struct thread_info *th
> }
>
> struct thread_info *
> -find_thread_pid (ptid_t ptid)
> +find_thread_ptid (ptid_t ptid)
> {
> struct inferior_list_entry *inf = all_threads.head;
>
> @@ -228,7 +228,7 @@ find_thread_pid (ptid_t ptid)
> ptid_t
> gdb_id_to_thread_id (ptid_t gdb_id)
> {
> - struct thread_info *thread = find_thread_pid (gdb_id);
> + struct thread_info *thread = find_thread_ptid (gdb_id);
>
> return thread ? thread->entry.id : null_ptid;
> }
> Index: gdbserver/remote-utils.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/remote-utils.c,v
> retrieving revision 1.66
> diff -u -p -r1.66 remote-utils.c
> --- gdbserver/remote-utils.c 3 Apr 2009 14:38:38 -0000 1.66
> +++ gdbserver/remote-utils.c 1 May 2009 22:24:07 -0000
> @@ -1091,7 +1091,7 @@ prepare_resume_reply (char *buf, ptid_t
>
> saved_inferior = current_inferior;
>
> - current_inferior = find_thread_pid (ptid);
> + current_inferior = find_thread_ptid (ptid);
>
> if (the_target->stopped_by_watchpoint != NULL
> && (*the_target->stopped_by_watchpoint) ())
> Index: gdbserver/server.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/server.c,v
> retrieving revision 1.96
> diff -u -p -r1.96 server.c
> --- gdbserver/server.c 30 Apr 2009 22:21:06 -0000 1.96
> +++ gdbserver/server.c 1 May 2009 22:24:07 -0000
> @@ -1156,7 +1156,7 @@ handle_query (char *own_buf, int packet_
> err = 1;
> else
> {
> - struct thread_info *thread = find_thread_pid (ptid);
> + struct thread_info *thread = find_thread_ptid (ptid);
>
> if (thread == NULL)
> err = 2;
> Index: gdbserver/server.h
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/server.h,v
> retrieving revision 1.57
> diff -u -p -r1.57 server.h
> --- gdbserver/server.h 30 Apr 2009 22:21:06 -0000 1.57
> +++ gdbserver/server.h 1 May 2009 22:24:07 -0000
> @@ -239,7 +239,7 @@ struct process_info *find_process_pid (i
> int have_started_inferiors_p (void);
> int have_attached_inferiors_p (void);
>
> -struct thread_info *find_thread_pid (ptid_t ptid);
> +struct thread_info *find_thread_ptid (ptid_t ptid);
>
> ptid_t thread_id_to_gdb_id (ptid_t);
> ptid_t thread_to_gdb_id (struct thread_info *);
> Index: gdbserver/target.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/gdbserver/target.c,v
> retrieving revision 1.15
> diff -u -p -r1.15 target.c
> --- gdbserver/target.c 1 Apr 2009 22:50:24 -0000 1.15
> +++ gdbserver/target.c 1 May 2009 22:24:07 -0000
> @@ -29,7 +29,7 @@ set_desired_inferior (int use_general)
> struct thread_info *found;
>
> if (use_general == 1)
> - found = find_thread_pid (general_thread);
> + found = find_thread_ptid (general_thread);
> else
> {
> found = NULL;
> @@ -41,10 +41,10 @@ set_desired_inferior (int use_general)
> && !ptid_equal (step_thread, minus_one_ptid))
> && (ptid_equal (cont_thread, null_ptid)
> || ptid_equal (cont_thread, minus_one_ptid)))
> - found = find_thread_pid (step_thread);
> + found = find_thread_ptid (step_thread);
>
> if (found == NULL)
> - found = find_thread_pid (cont_thread);
> + found = find_thread_ptid (cont_thread);
> }
>
> if (found == NULL)
>
next prev parent reply other threads:[~2009-05-18 23:21 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-01 22:51 Doug Evans
2009-05-06 19:02 ` Joel Brobecker
2009-05-06 20:25 ` Doug Evans
2009-05-18 23:21 ` Doug Evans [this message]
2009-05-22 15:20 ` Joel Brobecker
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e394668d0905181621m2654c31flba1256fb8fd26256@mail.gmail.com \
--to=dje@google.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox