From: Simon Marchi Date: Fri, 19 Jan 2018 16:48:11 +0000 (-0500) Subject: Make linux_nat_detach/thread_db_detach use the inferior parameter X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=bc09b0c1;p=binutils-gdb.git Make linux_nat_detach/thread_db_detach use the inferior parameter This patch makes these two functions actually use the inferior parameter added by the previous patch, instead of reading inferior_ptid. I chose these two, because they are the one actually used when I detach on my GNU/Linux system, so they were easy to test. I took the opportunity to pass the inferior being detached to inf_ptrace_detach_success, so it could use it too. From there, it made sense to add an overload of detach_inferior that takes the inferior directly rather than the pid, to avoid having to pass inf->pid only for the callee to look up the inferior structure by pid. gdb/ChangeLog: * inf-ptrace.c (inf_ptrace_detach): Adjust call to inf_ptrace_detach_success. (inf_ptrace_detach_success): Add inferior parameter, use it instead of inferior_ptid, pass it to detach_inferior. * inf-ptrace.h (inf_ptrace_detach_success): Add inferior parameter. * inferior.c (detach_inferior): Add overload that takes an inferior object. * inferior.h (detach_inferior): Likewise. * linux-nat.c (linux_nat_detach): Use the inf parameter, don't use inferior_ptid, adjust call to inf_ptrace_detach_success. * linux-thread-db.c (thread_db_detach): Use inf parameter. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 4a994b411c7..5467e89c604 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,18 @@ +2018-01-19 Simon Marchi + + * inf-ptrace.c (inf_ptrace_detach): Adjust call to + inf_ptrace_detach_success. + (inf_ptrace_detach_success): Add inferior parameter, use it + instead of inferior_ptid, pass it to detach_inferior. + * inf-ptrace.h (inf_ptrace_detach_success): Add inferior + parameter. + * inferior.c (detach_inferior): Add overload that takes an + inferior object. + * inferior.h (detach_inferior): Likewise. + * linux-nat.c (linux_nat_detach): Use the inf parameter, don't + use inferior_ptid, adjust call to inf_ptrace_detach_success. + * linux-thread-db.c (thread_db_detach): Use inf parameter. + 2018-01-19 Simon Marchi * target.h (struct target_ops) : Add inferior diff --git a/gdb/inf-ptrace.c b/gdb/inf-ptrace.c index aa8b17f41a5..72aa33480f4 100644 --- a/gdb/inf-ptrace.c +++ b/gdb/inf-ptrace.c @@ -263,18 +263,16 @@ inf_ptrace_detach (struct target_ops *ops, inferior *inf, int from_tty) error (_("This system does not support detaching from a process")); #endif - inf_ptrace_detach_success (ops); + inf_ptrace_detach_success (ops, inf); } /* See inf-ptrace.h. */ void -inf_ptrace_detach_success (struct target_ops *ops) +inf_ptrace_detach_success (struct target_ops *ops, inferior *inf) { - pid_t pid = ptid_get_pid (inferior_ptid); - inferior_ptid = null_ptid; - detach_inferior (pid); + detach_inferior (inf); inf_child_maybe_unpush_target (ops); } diff --git a/gdb/inf-ptrace.h b/gdb/inf-ptrace.h index c5bd7573603..d10f64ae56c 100644 --- a/gdb/inf-ptrace.h +++ b/gdb/inf-ptrace.h @@ -40,6 +40,6 @@ extern pid_t get_ptrace_pid (ptid_t); /* Cleanup the inferior after a successful ptrace detach. */ -extern void inf_ptrace_detach_success (struct target_ops *ops); +extern void inf_ptrace_detach_success (struct target_ops *ops, inferior *inf); #endif diff --git a/gdb/inferior.c b/gdb/inferior.c index 0b8f340b63a..38b7369275b 100644 --- a/gdb/inferior.c +++ b/gdb/inferior.c @@ -253,10 +253,13 @@ exit_inferior_num_silent (int num) exit_inferior_1 (inf, 1); } +/* See inferior.h. */ + void -detach_inferior (int pid) +detach_inferior (inferior *inf) { - struct inferior *inf = find_inferior_pid (pid); + /* Save the pid, since exit_inferior_1 will reset it. */ + int pid = inf->pid; exit_inferior_1 (inf, 0); @@ -264,6 +267,14 @@ detach_inferior (int pid) printf_unfiltered (_("[Inferior %d detached]\n"), pid); } +/* See inferior.h. */ + +void +detach_inferior (int pid) +{ + detach_inferior (find_inferior_pid (pid)); +} + void inferior_appeared (struct inferior *inf, int pid) { diff --git a/gdb/inferior.h b/gdb/inferior.h index 01a12f2290b..a87ffe00546 100644 --- a/gdb/inferior.h +++ b/gdb/inferior.h @@ -458,6 +458,9 @@ extern struct inferior *add_inferior_silent (int pid); extern void delete_inferior (struct inferior *todel); /* Delete an existing inferior list entry, due to inferior detaching. */ +extern void detach_inferior (inferior *inf); + +/* Same as the above, but with the inferior specified by PID. */ extern void detach_inferior (int pid); extern void exit_inferior (int pid); diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 70f4c084dc6..b83066cb040 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -1499,10 +1499,8 @@ detach_callback (struct lwp_info *lp, void *data) static void linux_nat_detach (struct target_ops *ops, inferior *inf, int from_tty) { - int pid; struct lwp_info *main_lwp; - - pid = ptid_get_pid (inferior_ptid); + int pid = inf->pid; /* Don't unregister from the event loop, as there may be other inferiors running. */ @@ -1517,7 +1515,7 @@ linux_nat_detach (struct target_ops *ops, inferior *inf, int from_tty) iterate_over_lwps (pid_to_ptid (pid), detach_callback, NULL); /* Only the initial process should be left right now. */ - gdb_assert (num_lwps (ptid_get_pid (inferior_ptid)) == 1); + gdb_assert (num_lwps (pid) == 1); main_lwp = find_lwp_pid (pid_to_ptid (pid)); @@ -1538,7 +1536,7 @@ linux_nat_detach (struct target_ops *ops, inferior *inf, int from_tty) detach_one_lwp (main_lwp, &signo); - inf_ptrace_detach_success (ops); + inf_ptrace_detach_success (ops, inf); } } diff --git a/gdb/linux-thread-db.c b/gdb/linux-thread-db.c index 873f929144b..794c97b48ac 100644 --- a/gdb/linux-thread-db.c +++ b/gdb/linux-thread-db.c @@ -1094,7 +1094,7 @@ thread_db_detach (struct target_ops *ops, inferior *inf, int from_tty) { struct target_ops *target_beneath = find_target_beneath (ops); - delete_thread_db_info (ptid_get_pid (inferior_ptid)); + delete_thread_db_info (inf->pid); target_beneath->to_detach (target_beneath, inf, from_tty);