diff options
author | Pedro Alves <palves@redhat.com> | 2009-05-24 18:00:08 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2009-05-24 18:00:08 +0000 |
commit | e58b0e63bb0a42e99f6fb1e6d697a1b29caa02c2 (patch) | |
tree | 0885c611c83e59c16a996ecc2aecb21f823bf028 /gdb/inf-ttrace.c | |
parent | e27d73f6b0fa7a7bb175b7b52b9fce5ff535e239 (diff) | |
download | gdb-e58b0e63bb0a42e99f6fb1e6d697a1b29caa02c2.zip gdb-e58b0e63bb0a42e99f6fb1e6d697a1b29caa02c2.tar.gz gdb-e58b0e63bb0a42e99f6fb1e6d697a1b29caa02c2.tar.bz2 |
gdb/
* gdbthread.h (struct thread_info): New `pending_follow' field.
* thread.c (new_thread): New function.
(add_thread_silent): Use it.
* breakpoint.c (internal_breakpoint_number): New global, moved
from inside...
(create_internal_breakpoint): ... this.
(clone_momentary_breakpoint): New.
* breakpoint.h (clone_momentary_breakpoint): Declare.
* infrun.c (nullify_last_target_wait_ptid): Move declaration
higher.
(pending_follow): Delete.
(follow_fork): Handle pending follow fork event here. Moved the
preserving of thread stepping state here.
(resume): Don't handle pending follow fork events here. Only
install the inferior's terminal modes if we're about to resume it.
(proceed): Handle possible pending follow fork events here.
(init_wait_for_inferior): No need to clear pending_follow anymore,
it's gone.
(handle_inferior_event): Adjust to per-thread `pending_follow'.
Call `follow_fork' to handle following the fork. If the
follow-fork is cancelled, stop stepping.
* linux-nat.c (linux_child_follow_fork): Adjust to per-thread
`pending_follow' events. Remove code that handled preserving the
thread stepping state.
* inf-ptrace.c (inf_ptrace_follow_fork): Ditto.
* inf-ttrace.c (inf_ttrace_follow_fork): Ditto.
gdb/testsuite/
* gdb.threads/fork-thread-pending.c: New.
* gdb.threads/fork-thread-pending.exp: New.
Diffstat (limited to 'gdb/inf-ttrace.c')
-rw-r--r-- | gdb/inf-ttrace.c | 44 |
1 files changed, 7 insertions, 37 deletions
diff --git a/gdb/inf-ttrace.c b/gdb/inf-ttrace.c index a4c4b99..ffec586 100644 --- a/gdb/inf-ttrace.c +++ b/gdb/inf-ttrace.c @@ -412,25 +412,13 @@ inf_ttrace_follow_fork (struct target_ops *ops, int follow_child) pid_t pid, fpid; lwpid_t lwpid, flwpid; ttstate_t tts; - struct thread_info *last_tp = NULL; - struct breakpoint *step_resume_breakpoint = NULL; - CORE_ADDR step_range_start = 0, step_range_end = 0; - struct frame_id step_frame_id = null_frame_id; - - /* FIXME: kettenis/20050720: This stuff should really be passed as - an argument by our caller. */ - { - ptid_t ptid; - struct target_waitstatus status; - - get_last_target_status (&ptid, &status); - gdb_assert (status.kind == TARGET_WAITKIND_FORKED - || status.kind == TARGET_WAITKIND_VFORKED); - - pid = ptid_get_pid (ptid); - lwpid = ptid_get_lwp (ptid); - last_tp = find_thread_pid (ptid); - } + struct thread_info *tp = inferior_thread (); + + gdb_assert (tp->pending_follow.kind == TARGET_WAITKIND_FORKED + || tp->pending_follow.kind == TARGET_WAITKIND_VFORKED); + + pid = ptid_get_pid (inferior_ptid); + lwpid = ptid_get_lwp (inferior_ptid); /* Get all important details that core GDB doesn't (and shouldn't) know about. */ @@ -462,16 +450,6 @@ inf_ttrace_follow_fork (struct target_ops *ops, int follow_child) parent_inf = find_inferior_pid (pid); - /* Copy user stepping state to the new inferior thread. */ - step_resume_breakpoint = last_tp->step_resume_breakpoint; - step_range_start = last_tp->step_range_start; - step_range_end = last_tp->step_range_end; - step_frame_id = last_tp->step_frame_id; - - /* Otherwise, deleting the parent would get rid of this - breakpoint. */ - last_tp->step_resume_breakpoint = NULL; - inferior_ptid = ptid_build (fpid, flwpid, 0); inf = add_inferior (fpid); inf->attach_flag = parent_inf->attach_flag; @@ -553,14 +531,6 @@ Detaching after fork from child process %ld.\n"), (long)fpid); xmalloc (sizeof (struct inf_ttrace_private_thread_info)); memset (ti->private, 0, sizeof (struct inf_ttrace_private_thread_info)); - - ti->step_resume_breakpoint = step_resume_breakpoint; - ti->step_range_start = step_range_start; - ti->step_range_end = step_range_end; - ti->step_frame_id = step_frame_id; - - /* Reset breakpoints in the child as appropriate. */ - follow_inferior_reset_breakpoints (); } return 0; |