Make linux_nat_detach/thread_db_detach use the inferior parameter
authorSimon Marchi <simon.marchi@ericsson.com>
Fri, 19 Jan 2018 16:48:11 +0000 (11:48 -0500)
committerSimon Marchi <simon.marchi@ericsson.com>
Fri, 19 Jan 2018 16:48:11 +0000 (11:48 -0500)
commitbc09b0c14fb713a9aec25e09b78499f3bc2441b5
tree1be85b1497ab8383a20262964e0c9610988c42b5
parent6e1e1966bac965c5a26b5e5cae69cb0ed21be4cc
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.
gdb/ChangeLog
gdb/inf-ptrace.c
gdb/inf-ptrace.h
gdb/inferior.c
gdb/inferior.h
gdb/linux-nat.c
gdb/linux-thread-db.c