Make target_wait options use enum flags
This changes TARGET_WNOHANG to be a member of an enum, rather than a define, and also adds a DEF_ENUM_FLAGS_TYPE for this type. Then, it changes target_wait and the various target wait methods to use this type rather than "int". This didn't catch any bugs, but it seems like a decent cleanup nevertheless. I did not change deprecated_target_wait_hook, since that's only used out-of-tree (by Insight), and there didn't seem to be a need. I can't build some of these targets, so I modified them on a best-effort basis. I don't think this patch should go in before the release branch is made. gdb/ChangeLog 2020-09-18 Tom Tromey <tromey@adacore.com> * windows-nat.c (struct windows_nat_target) <wait>: Update. (windows_nat_target::wait): Update. * target/wait.h (enum target_wait_flag): New. Use DEF_ENUM_FLAGS_TYPE. * target/target.h (target_wait): Change type of options. * target.h (target_options_to_string, default_target_wait): Update. (struct target_ops) <wait>: Change type of options. * target.c (target_wait, default_target_wait, do_option): Change type of "options". (target_options_to_string): Likewise. * target-delegates.c: Rebuild. * target-debug.h (target_debug_print_target_wait_flags): Rename from target_debug_print_options. * sol-thread.c (class sol_thread_target) <wait>: Update. (sol_thread_target::wait): Update. * rs6000-nat.c (class rs6000_nat_target) <wait>: Update. (rs6000_nat_target::wait): Update. * remote.c (class remote_target) <wait, wait_ns, wait_as>: Update. (remote_target::wait_ns, remote_target::wait_as): Change type of "options". (remote_target::wait): Update. * remote-sim.c (struct gdbsim_target) <wait>: Update. (gdbsim_target::wait): Update. * record-full.c (class record_full_base_target) <wait>: Update. (record_full_wait_1): Change type of "options". (record_full_base_target::wait): Update. * record-btrace.c (class record_btrace_target) <wait>: Update. (record_btrace_target::wait): Update. * ravenscar-thread.c (struct ravenscar_thread_target) <wait>: Update. (ravenscar_thread_target::wait): Update. * procfs.c (class procfs_target) <wait>: Update. (procfs_target::wait): Update. * obsd-nat.h (class obsd_nat_target) <wait>: Update. * obsd-nat.c (obsd_nat_target::wait): Update. * nto-procfs.c (struct nto_procfs_target) <wait>: Update. (nto_procfs_target::wait): Update. * nbsd-nat.h (struct nbsd_nat_target) <wait>: Update. * nbsd-nat.c (nbsd_wait): Change type of "options". (nbsd_nat_target::wait): Update. * linux-thread-db.c (class thread_db_target) <wait>: Update. (thread_db_target::wait): Update. * linux-nat.h (class linux_nat_target) <wait>: Update. * linux-nat.c (linux_nat_target::wait): Update. (linux_nat_wait_1): Update. * infrun.c (do_target_wait_1, do_target_wait): Change type of "options". * inf-ptrace.h (struct inf_ptrace_target) <wait>: Update. * inf-ptrace.c (inf_ptrace_target::wait): Update. * go32-nat.c (struct go32_nat_target) <wait>: Update. (go32_nat_target::wait): Update. * gnu-nat.h (struct gnu_nat_target) <wait>: Update. * gnu-nat.c (gnu_nat_target::wait): Update. * fbsd-nat.h (class fbsd_nat_target) <wait>: Update. * fbsd-nat.c (fbsd_nat_target::wait): Update. * darwin-nat.h (class darwin_nat_target) <wait>: Update. * darwin-nat.c (darwin_nat_target::wait): Update. * bsd-uthread.c (struct bsd_uthread_target) <wait>: Update. (bsd_uthread_target::wait): Update. * aix-thread.c (class aix_thread_target) <wait>: Update. (aix_thread_target::wait): Update. gdbserver/ChangeLog 2020-09-18 Tom Tromey <tromey@adacore.com> * netbsd-low.h (class netbsd_process_target) <wait>: Update. * netbsd-low.cc (netbsd_waitpid, netbsd_wait) (netbsd_process_target::wait): Change type of target_options. * win32-low.h (class win32_process_target) <wait>: Update. * win32-low.cc (win32_process_target::wait): Update. * target.h (class process_stratum_target) <wait>: Update. (mywait): Update. * target.cc (mywait, target_wait): Change type of "options". * linux-low.h (class linux_process_target) <wait, wait_1>: Update. * linux-low.cc (linux_process_target::wait) (linux_process_target::wait_1): Update.
This commit is contained in:
parent
0295dde655
commit
b60cea74de
45 changed files with 189 additions and 88 deletions
|
@ -1,3 +1,69 @@
|
|||
2020-09-18 Tom Tromey <tromey@adacore.com>
|
||||
|
||||
* windows-nat.c (struct windows_nat_target) <wait>: Update.
|
||||
(windows_nat_target::wait): Update.
|
||||
* target/wait.h (enum target_wait_flag): New. Use
|
||||
DEF_ENUM_FLAGS_TYPE.
|
||||
* target/target.h (target_wait): Change type of options.
|
||||
* target.h (target_options_to_string, default_target_wait):
|
||||
Update.
|
||||
(struct target_ops) <wait>: Change type of options.
|
||||
* target.c (target_wait, default_target_wait, do_option): Change
|
||||
type of "options".
|
||||
(target_options_to_string): Likewise.
|
||||
* target-delegates.c: Rebuild.
|
||||
* target-debug.h (target_debug_print_target_wait_flags): Rename
|
||||
from target_debug_print_options.
|
||||
* sol-thread.c (class sol_thread_target) <wait>: Update.
|
||||
(sol_thread_target::wait): Update.
|
||||
* rs6000-nat.c (class rs6000_nat_target) <wait>: Update.
|
||||
(rs6000_nat_target::wait): Update.
|
||||
* remote.c (class remote_target) <wait, wait_ns, wait_as>:
|
||||
Update.
|
||||
(remote_target::wait_ns, remote_target::wait_as): Change type of
|
||||
"options".
|
||||
(remote_target::wait): Update.
|
||||
* remote-sim.c (struct gdbsim_target) <wait>: Update.
|
||||
(gdbsim_target::wait): Update.
|
||||
* record-full.c (class record_full_base_target) <wait>: Update.
|
||||
(record_full_wait_1): Change type of "options".
|
||||
(record_full_base_target::wait): Update.
|
||||
* record-btrace.c (class record_btrace_target) <wait>: Update.
|
||||
(record_btrace_target::wait): Update.
|
||||
* ravenscar-thread.c (struct ravenscar_thread_target) <wait>:
|
||||
Update.
|
||||
(ravenscar_thread_target::wait): Update.
|
||||
* procfs.c (class procfs_target) <wait>: Update.
|
||||
(procfs_target::wait): Update.
|
||||
* obsd-nat.h (class obsd_nat_target) <wait>: Update.
|
||||
* obsd-nat.c (obsd_nat_target::wait): Update.
|
||||
* nto-procfs.c (struct nto_procfs_target) <wait>: Update.
|
||||
(nto_procfs_target::wait): Update.
|
||||
* nbsd-nat.h (struct nbsd_nat_target) <wait>: Update.
|
||||
* nbsd-nat.c (nbsd_wait): Change type of "options".
|
||||
(nbsd_nat_target::wait): Update.
|
||||
* linux-thread-db.c (class thread_db_target) <wait>: Update.
|
||||
(thread_db_target::wait): Update.
|
||||
* linux-nat.h (class linux_nat_target) <wait>: Update.
|
||||
* linux-nat.c (linux_nat_target::wait): Update.
|
||||
(linux_nat_wait_1): Update.
|
||||
* infrun.c (do_target_wait_1, do_target_wait): Change type of
|
||||
"options".
|
||||
* inf-ptrace.h (struct inf_ptrace_target) <wait>: Update.
|
||||
* inf-ptrace.c (inf_ptrace_target::wait): Update.
|
||||
* go32-nat.c (struct go32_nat_target) <wait>: Update.
|
||||
(go32_nat_target::wait): Update.
|
||||
* gnu-nat.h (struct gnu_nat_target) <wait>: Update.
|
||||
* gnu-nat.c (gnu_nat_target::wait): Update.
|
||||
* fbsd-nat.h (class fbsd_nat_target) <wait>: Update.
|
||||
* fbsd-nat.c (fbsd_nat_target::wait): Update.
|
||||
* darwin-nat.h (class darwin_nat_target) <wait>: Update.
|
||||
* darwin-nat.c (darwin_nat_target::wait): Update.
|
||||
* bsd-uthread.c (struct bsd_uthread_target) <wait>: Update.
|
||||
(bsd_uthread_target::wait): Update.
|
||||
* aix-thread.c (class aix_thread_target) <wait>: Update.
|
||||
(aix_thread_target::wait): Update.
|
||||
|
||||
2020-09-18 Andrew Burgess <andrew.burgess@embecosm.com>
|
||||
|
||||
* compile/compile-object-run.c (create_copied_type_recursive): New
|
||||
|
|
|
@ -124,7 +124,7 @@ public:
|
|||
|
||||
void detach (inferior *, int) override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -1077,7 +1077,7 @@ aix_thread_target::resume (ptid_t ptid, int step, enum gdb_signal sig)
|
|||
|
||||
ptid_t
|
||||
aix_thread_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
{
|
||||
scoped_restore save_inferior_ptid = make_scoped_restore (&inferior_ptid);
|
||||
|
|
|
@ -53,7 +53,7 @@ struct bsd_uthread_target final : public target_ops
|
|||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
bool thread_alive (ptid_t ptid) override;
|
||||
|
@ -377,7 +377,7 @@ bsd_uthread_target::store_registers (struct regcache *regcache, int regnum)
|
|||
|
||||
ptid_t
|
||||
bsd_uthread_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch ());
|
||||
CORE_ADDR addr;
|
||||
|
|
|
@ -1285,7 +1285,7 @@ darwin_nat_target::wait_1 (ptid_t ptid, struct target_waitstatus *status)
|
|||
|
||||
ptid_t
|
||||
darwin_nat_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
return wait_1 (ptid, status);
|
||||
}
|
||||
|
|
|
@ -86,7 +86,7 @@ class darwin_nat_target : public inf_child_target
|
|||
|
||||
void detach (inferior *, int) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void mourn_inferior () override;
|
||||
|
||||
|
|
|
@ -1166,7 +1166,7 @@ fbsd_handle_debug_trap (fbsd_nat_target *target, ptid_t ptid,
|
|||
|
||||
ptid_t
|
||||
fbsd_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
ptid_t wptid;
|
||||
|
||||
|
|
|
@ -69,7 +69,7 @@ public:
|
|||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void post_startup_inferior (ptid_t) override;
|
||||
void post_attach (int) override;
|
||||
|
|
|
@ -1436,7 +1436,7 @@ static struct inf *waiting_inf;
|
|||
|
||||
ptid_t
|
||||
gnu_nat_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
struct msg
|
||||
{
|
||||
|
|
|
@ -124,7 +124,7 @@ struct gnu_nat_target : public inf_child_target
|
|||
void detach (inferior *, int) override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
enum target_xfer_status xfer_partial (enum target_object object,
|
||||
const char *annex,
|
||||
gdb_byte *readbuf,
|
||||
|
|
|
@ -342,7 +342,7 @@ struct go32_nat_target final : public x86_nat_target<inf_child_target>
|
|||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -419,7 +419,7 @@ static char child_cwd[FILENAME_MAX];
|
|||
|
||||
ptid_t
|
||||
go32_nat_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
int i;
|
||||
unsigned char saved_opcode;
|
||||
|
|
|
@ -316,7 +316,7 @@ inf_ptrace_target::resume (ptid_t ptid, int step, enum gdb_signal signal)
|
|||
|
||||
ptid_t
|
||||
inf_ptrace_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
pid_t pid;
|
||||
int status, save_errno;
|
||||
|
|
|
@ -35,7 +35,7 @@ struct inf_ptrace_target : public inf_child_target
|
|||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void files_info () override;
|
||||
|
||||
|
|
|
@ -3423,7 +3423,7 @@ random_pending_event_thread (inferior *inf, ptid_t waiton_ptid)
|
|||
|
||||
static ptid_t
|
||||
do_target_wait_1 (inferior *inf, ptid_t ptid,
|
||||
target_waitstatus *status, int options)
|
||||
target_waitstatus *status, target_wait_flags options)
|
||||
{
|
||||
ptid_t event_ptid;
|
||||
struct thread_info *tp;
|
||||
|
@ -3546,7 +3546,8 @@ do_target_wait_1 (inferior *inf, ptid_t ptid,
|
|||
more events. Polls for events from all inferiors/targets. */
|
||||
|
||||
static bool
|
||||
do_target_wait (ptid_t wait_ptid, execution_control_state *ecs, int options)
|
||||
do_target_wait (ptid_t wait_ptid, execution_control_state *ecs,
|
||||
target_wait_flags options)
|
||||
{
|
||||
int num_inferiors = 0;
|
||||
int random_selector;
|
||||
|
|
|
@ -3131,7 +3131,7 @@ filter_exit_event (struct lwp_info *event_child,
|
|||
|
||||
static ptid_t
|
||||
linux_nat_wait_1 (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
sigset_t prev_mask;
|
||||
enum resume_kind last_resume_kind;
|
||||
|
@ -3411,7 +3411,7 @@ resume_stopped_resumed_lwps (struct lwp_info *lp, const ptid_t wait_ptid)
|
|||
|
||||
ptid_t
|
||||
linux_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
ptid_t event_ptid;
|
||||
|
||||
|
|
|
@ -46,7 +46,7 @@ public:
|
|||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void pass_signals (gdb::array_view<const unsigned char>) override;
|
||||
|
||||
|
|
|
@ -92,7 +92,7 @@ public:
|
|||
strata stratum () const override { return thread_stratum; }
|
||||
|
||||
void detach (inferior *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
void mourn_inferior () override;
|
||||
void update_thread_list () override;
|
||||
|
@ -1379,7 +1379,7 @@ thread_db_target::detach (inferior *inf, int from_tty)
|
|||
|
||||
ptid_t
|
||||
thread_db_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
struct thread_db_info *info;
|
||||
|
||||
|
|
|
@ -540,7 +540,8 @@ nbsd_nat_target::resume (ptid_t ptid, int step, enum gdb_signal signal)
|
|||
/* Implement a safe wrapper around waitpid(). */
|
||||
|
||||
static pid_t
|
||||
nbsd_wait (ptid_t ptid, struct target_waitstatus *ourstatus, int options)
|
||||
nbsd_wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
target_wait_flags options)
|
||||
{
|
||||
pid_t pid;
|
||||
int status;
|
||||
|
@ -569,7 +570,7 @@ nbsd_wait (ptid_t ptid, struct target_waitstatus *ourstatus, int options)
|
|||
|
||||
ptid_t
|
||||
nbsd_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
pid_t pid = nbsd_wait (ptid, ourstatus, target_options);
|
||||
ptid_t wptid = ptid_t (pid);
|
||||
|
|
|
@ -41,7 +41,7 @@ struct nbsd_nat_target : public inf_ptrace_target
|
|||
bool info_proc (const char *, enum info_proc_what) override;
|
||||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
int insert_exec_catchpoint (int pid) override;
|
||||
int remove_exec_catchpoint (int pid) override;
|
||||
int set_syscall_catchpoint (int pid, bool needed, int any_count,
|
||||
|
|
|
@ -69,7 +69,7 @@ struct nto_procfs_target : public inf_child_target
|
|||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -795,7 +795,7 @@ nto_handle_sigint (int signo)
|
|||
|
||||
sptid_t
|
||||
nto_procfs_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
sigset_t set;
|
||||
siginfo_t info;
|
||||
|
|
|
@ -74,7 +74,7 @@ obsd_nat_target::update_thread_list ()
|
|||
|
||||
ptid_t
|
||||
obsd_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
pid_t pid;
|
||||
int status, save_errno;
|
||||
|
|
|
@ -27,7 +27,7 @@ class obsd_nat_target : public inf_ptrace_target
|
|||
/* Override some methods to support threads. */
|
||||
std::string pid_to_str (ptid_t) override;
|
||||
void update_thread_list () override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
#ifdef PT_GET_PROCESS_STATE
|
||||
bool follow_fork (bool, bool) override;
|
||||
|
|
|
@ -105,7 +105,7 @@ public:
|
|||
void detach (inferior *inf, int) override;
|
||||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -2033,7 +2033,7 @@ procfs_target::store_registers (struct regcache *regcache, int regnum)
|
|||
|
||||
ptid_t
|
||||
procfs_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
/* First cut: loosely based on original version 2.1. */
|
||||
procinfo *pi;
|
||||
|
|
|
@ -87,7 +87,7 @@ struct ravenscar_thread_target final : public target_ops
|
|||
|
||||
strata stratum () const override { return thread_stratum; }
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
|
@ -385,7 +385,7 @@ ravenscar_thread_target::resume (ptid_t ptid, int step,
|
|||
ptid_t
|
||||
ravenscar_thread_target::wait (ptid_t ptid,
|
||||
struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
process_stratum_target *beneath
|
||||
= as_process_stratum_target (this->beneath ());
|
||||
|
|
|
@ -118,7 +118,7 @@ public:
|
|||
|
||||
void commit_resume () override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void stop (ptid_t) override;
|
||||
void update_thread_list () override;
|
||||
|
@ -2537,12 +2537,13 @@ record_btrace_maybe_mark_async_event
|
|||
|
||||
ptid_t
|
||||
record_btrace_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
std::vector<thread_info *> moving;
|
||||
std::vector<thread_info *> no_history;
|
||||
|
||||
DEBUG ("wait %s (0x%x)", target_pid_to_str (ptid).c_str (), options);
|
||||
DEBUG ("wait %s (0x%x)", target_pid_to_str (ptid).c_str (),
|
||||
(unsigned) options);
|
||||
|
||||
/* As long as we're not replaying, just forward the request. */
|
||||
if ((::execution_direction != EXEC_REVERSE)
|
||||
|
|
|
@ -225,7 +225,7 @@ public:
|
|||
|
||||
void close () override;
|
||||
void async (int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
bool stopped_by_watchpoint () override;
|
||||
bool stopped_data_address (CORE_ADDR *) override;
|
||||
|
||||
|
@ -1155,7 +1155,7 @@ record_full_sig_handler (int signo)
|
|||
static ptid_t
|
||||
record_full_wait_1 (struct target_ops *ops,
|
||||
ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
scoped_restore restore_operation_disable
|
||||
= record_full_gdb_operation_disable_set ();
|
||||
|
@ -1468,7 +1468,7 @@ record_full_wait_1 (struct target_ops *ops,
|
|||
|
||||
ptid_t
|
||||
record_full_base_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
ptid_t return_ptid;
|
||||
|
||||
|
|
|
@ -131,7 +131,7 @@ struct gdbsim_target final
|
|||
void detach (inferior *inf, int) override;
|
||||
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -927,7 +927,8 @@ gdbsim_cntrl_c (int signo)
|
|||
}
|
||||
|
||||
ptid_t
|
||||
gdbsim_target::wait (ptid_t ptid, struct target_waitstatus *status, int options)
|
||||
gdbsim_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
target_wait_flags options)
|
||||
{
|
||||
struct sim_inferior_data *sim_data;
|
||||
static sighandler_t prev_sigint;
|
||||
|
|
15
gdb/remote.c
15
gdb/remote.c
|
@ -422,7 +422,7 @@ public:
|
|||
|
||||
void commit_resume () override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -739,9 +739,9 @@ public: /* Remote specific methods. */
|
|||
void add_current_inferior_and_thread (char *wait_status);
|
||||
|
||||
ptid_t wait_ns (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options);
|
||||
target_wait_flags options);
|
||||
ptid_t wait_as (ptid_t ptid, target_waitstatus *status,
|
||||
int options);
|
||||
target_wait_flags options);
|
||||
|
||||
ptid_t process_stop_reply (struct stop_reply *stop_reply,
|
||||
target_waitstatus *status);
|
||||
|
@ -7771,7 +7771,8 @@ remote_target::process_stop_reply (struct stop_reply *stop_reply,
|
|||
/* The non-stop mode version of target_wait. */
|
||||
|
||||
ptid_t
|
||||
remote_target::wait_ns (ptid_t ptid, struct target_waitstatus *status, int options)
|
||||
remote_target::wait_ns (ptid_t ptid, struct target_waitstatus *status,
|
||||
target_wait_flags options)
|
||||
{
|
||||
struct remote_state *rs = get_remote_state ();
|
||||
struct stop_reply *stop_reply;
|
||||
|
@ -7839,7 +7840,8 @@ first_remote_resumed_thread (remote_target *target)
|
|||
STATUS just as `wait' would. */
|
||||
|
||||
ptid_t
|
||||
remote_target::wait_as (ptid_t ptid, target_waitstatus *status, int options)
|
||||
remote_target::wait_as (ptid_t ptid, target_waitstatus *status,
|
||||
target_wait_flags options)
|
||||
{
|
||||
struct remote_state *rs = get_remote_state ();
|
||||
ptid_t event_ptid = null_ptid;
|
||||
|
@ -7992,7 +7994,8 @@ remote_target::wait_as (ptid_t ptid, target_waitstatus *status, int options)
|
|||
STATUS just as `wait' would. */
|
||||
|
||||
ptid_t
|
||||
remote_target::wait (ptid_t ptid, struct target_waitstatus *status, int options)
|
||||
remote_target::wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
target_wait_flags options)
|
||||
{
|
||||
ptid_t event_ptid;
|
||||
|
||||
|
|
|
@ -90,7 +90,7 @@ public:
|
|||
void create_inferior (const char *, const std::string &,
|
||||
char **, int) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
private:
|
||||
enum target_xfer_status
|
||||
|
@ -495,7 +495,7 @@ rs6000_nat_target::xfer_partial (enum target_object object,
|
|||
|
||||
ptid_t
|
||||
rs6000_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
pid_t pid;
|
||||
int status, save_errno;
|
||||
|
|
|
@ -84,7 +84,7 @@ public:
|
|||
strata stratum () const override { return thread_stratum; }
|
||||
|
||||
void detach (inferior *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
void resume (ptid_t, int, enum gdb_signal) override;
|
||||
void mourn_inferior () override;
|
||||
std::string pid_to_str (ptid_t) override;
|
||||
|
@ -425,7 +425,7 @@ sol_thread_target::resume (ptid_t ptid, int step, enum gdb_signal signo)
|
|||
|
||||
ptid_t
|
||||
sol_thread_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
if (ptid.pid () != -1)
|
||||
{
|
||||
|
|
|
@ -205,7 +205,7 @@ target_debug_print_struct_target_waitstatus_p (struct target_waitstatus *status)
|
|||
target_debug_do_print ((X) ? "step" : "continue")
|
||||
|
||||
static void
|
||||
target_debug_print_options (int options)
|
||||
target_debug_print_target_wait_flags (target_wait_flags options)
|
||||
{
|
||||
std::string str = target_options_to_string (options);
|
||||
|
||||
|
|
|
@ -15,7 +15,7 @@ struct dummy_target : public target_ops
|
|||
void disconnect (const char *arg0, int arg1) override;
|
||||
void resume (ptid_t arg0, int arg1, enum gdb_signal arg2) override;
|
||||
void commit_resume () override;
|
||||
ptid_t wait (ptid_t arg0, struct target_waitstatus *arg1, int arg2) override;
|
||||
ptid_t wait (ptid_t arg0, struct target_waitstatus *arg1, target_wait_flags arg2) override;
|
||||
void fetch_registers (struct regcache *arg0, int arg1) override;
|
||||
void store_registers (struct regcache *arg0, int arg1) override;
|
||||
void prepare_to_store (struct regcache *arg0) override;
|
||||
|
@ -186,7 +186,7 @@ struct debug_target : public target_ops
|
|||
void disconnect (const char *arg0, int arg1) override;
|
||||
void resume (ptid_t arg0, int arg1, enum gdb_signal arg2) override;
|
||||
void commit_resume () override;
|
||||
ptid_t wait (ptid_t arg0, struct target_waitstatus *arg1, int arg2) override;
|
||||
ptid_t wait (ptid_t arg0, struct target_waitstatus *arg1, target_wait_flags arg2) override;
|
||||
void fetch_registers (struct regcache *arg0, int arg1) override;
|
||||
void store_registers (struct regcache *arg0, int arg1) override;
|
||||
void prepare_to_store (struct regcache *arg0) override;
|
||||
|
@ -461,19 +461,19 @@ debug_target::commit_resume ()
|
|||
}
|
||||
|
||||
ptid_t
|
||||
target_ops::wait (ptid_t arg0, struct target_waitstatus *arg1, int arg2)
|
||||
target_ops::wait (ptid_t arg0, struct target_waitstatus *arg1, target_wait_flags arg2)
|
||||
{
|
||||
return this->beneath ()->wait (arg0, arg1, arg2);
|
||||
}
|
||||
|
||||
ptid_t
|
||||
dummy_target::wait (ptid_t arg0, struct target_waitstatus *arg1, int arg2)
|
||||
dummy_target::wait (ptid_t arg0, struct target_waitstatus *arg1, target_wait_flags arg2)
|
||||
{
|
||||
return default_target_wait (this, arg0, arg1, arg2);
|
||||
}
|
||||
|
||||
ptid_t
|
||||
debug_target::wait (ptid_t arg0, struct target_waitstatus *arg1, int arg2)
|
||||
debug_target::wait (ptid_t arg0, struct target_waitstatus *arg1, target_wait_flags arg2)
|
||||
{
|
||||
ptid_t result;
|
||||
fprintf_unfiltered (gdb_stdlog, "-> %s->wait (...)\n", this->beneath ()->shortname ());
|
||||
|
@ -483,7 +483,7 @@ debug_target::wait (ptid_t arg0, struct target_waitstatus *arg1, int arg2)
|
|||
fputs_unfiltered (", ", gdb_stdlog);
|
||||
target_debug_print_struct_target_waitstatus_p (arg1);
|
||||
fputs_unfiltered (", ", gdb_stdlog);
|
||||
target_debug_print_options (arg2);
|
||||
target_debug_print_target_wait_flags (arg2);
|
||||
fputs_unfiltered (") = ", gdb_stdlog);
|
||||
target_debug_print_ptid_t (result);
|
||||
fputs_unfiltered ("\n", gdb_stdlog);
|
||||
|
|
11
gdb/target.c
11
gdb/target.c
|
@ -2007,7 +2007,8 @@ target_disconnect (const char *args, int from_tty)
|
|||
/* See target/target.h. */
|
||||
|
||||
ptid_t
|
||||
target_wait (ptid_t ptid, struct target_waitstatus *status, int options)
|
||||
target_wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
target_wait_flags options)
|
||||
{
|
||||
return current_top_target ()->wait (ptid, status, options);
|
||||
}
|
||||
|
@ -2017,7 +2018,7 @@ target_wait (ptid_t ptid, struct target_waitstatus *status, int options)
|
|||
ptid_t
|
||||
default_target_wait (struct target_ops *ops,
|
||||
ptid_t ptid, struct target_waitstatus *status,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
status->kind = TARGET_WAITKIND_IGNORE;
|
||||
return minus_one_ptid;
|
||||
|
@ -3360,8 +3361,8 @@ str_comma_list_concat_elem (std::string *list, const char *elem)
|
|||
OPT is removed from TARGET_OPTIONS. */
|
||||
|
||||
static void
|
||||
do_option (int *target_options, std::string *ret,
|
||||
int opt, const char *opt_str)
|
||||
do_option (target_wait_flags *target_options, std::string *ret,
|
||||
target_wait_flag opt, const char *opt_str)
|
||||
{
|
||||
if ((*target_options & opt) != 0)
|
||||
{
|
||||
|
@ -3373,7 +3374,7 @@ do_option (int *target_options, std::string *ret,
|
|||
/* See target.h. */
|
||||
|
||||
std::string
|
||||
target_options_to_string (int target_options)
|
||||
target_options_to_string (target_wait_flags target_options)
|
||||
{
|
||||
std::string ret;
|
||||
|
||||
|
|
|
@ -117,7 +117,7 @@ struct syscall
|
|||
};
|
||||
|
||||
/* Return a pretty printed form of TARGET_OPTIONS. */
|
||||
extern std::string target_options_to_string (int target_options);
|
||||
extern std::string target_options_to_string (target_wait_flags target_options);
|
||||
|
||||
/* Possible types of events that the inferior handler will have to
|
||||
deal with. */
|
||||
|
@ -488,7 +488,7 @@ struct target_ops
|
|||
current target. inferior_ptid may also be null_ptid on
|
||||
entry. */
|
||||
virtual ptid_t wait (ptid_t, struct target_waitstatus *,
|
||||
int TARGET_DEBUG_PRINTER (target_debug_print_options))
|
||||
target_wait_flags options)
|
||||
TARGET_DEFAULT_FUNC (default_target_wait);
|
||||
virtual void fetch_registers (struct regcache *, int)
|
||||
TARGET_DEFAULT_IGNORE ();
|
||||
|
@ -1456,7 +1456,7 @@ extern scoped_restore_tmpl<int> make_scoped_defer_target_commit_resume ();
|
|||
extern ptid_t default_target_wait (struct target_ops *ops,
|
||||
ptid_t ptid,
|
||||
struct target_waitstatus *status,
|
||||
int options);
|
||||
target_wait_flags options);
|
||||
|
||||
/* Fetch at least register REGNO, or all regs if regno == -1. No result. */
|
||||
|
||||
|
|
|
@ -21,6 +21,8 @@
|
|||
#define TARGET_TARGET_H
|
||||
|
||||
#include "target/waitstatus.h"
|
||||
#include "target/wait.h"
|
||||
|
||||
/* This header is a stopgap until more code is shared. */
|
||||
|
||||
/* Read LEN bytes of target memory at address MEMADDR, placing the
|
||||
|
@ -84,7 +86,7 @@ extern void target_continue (ptid_t ptid, enum gdb_signal signal);
|
|||
options. */
|
||||
|
||||
extern ptid_t target_wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
int options);
|
||||
target_wait_flags options);
|
||||
|
||||
/* The inferior process has died. Do what is right. */
|
||||
|
||||
|
|
|
@ -20,11 +20,18 @@
|
|||
#ifndef TARGET_WAIT_H
|
||||
#define TARGET_WAIT_H
|
||||
|
||||
#include "gdbsupport/enum-flags.h"
|
||||
|
||||
/* Options that can be passed to target_wait. */
|
||||
|
||||
/* Return immediately if there's no event already queued. If this
|
||||
options is not requested, target_wait blocks waiting for an
|
||||
event. */
|
||||
#define TARGET_WNOHANG 1
|
||||
enum target_wait_flag : unsigned
|
||||
{
|
||||
/* Return immediately if there's no event already queued. If this
|
||||
options is not requested, target_wait blocks waiting for an
|
||||
event. */
|
||||
TARGET_WNOHANG = 1,
|
||||
};
|
||||
|
||||
DEF_ENUM_FLAGS_TYPE (enum target_wait_flag, target_wait_flags);
|
||||
|
||||
#endif /* TARGET_WAIT_H */
|
||||
|
|
|
@ -312,7 +312,7 @@ struct windows_nat_target final : public x86_nat_target<inf_child_target>
|
|||
|
||||
void resume (ptid_t, int , enum gdb_signal) override;
|
||||
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, int) override;
|
||||
ptid_t wait (ptid_t, struct target_waitstatus *, target_wait_flags) override;
|
||||
|
||||
void fetch_registers (struct regcache *, int) override;
|
||||
void store_registers (struct regcache *, int) override;
|
||||
|
@ -1803,7 +1803,7 @@ out:
|
|||
/* Wait for interesting events to occur in the target process. */
|
||||
ptid_t
|
||||
windows_nat_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
int pid = -1;
|
||||
|
||||
|
|
|
@ -1,3 +1,18 @@
|
|||
2020-09-18 Tom Tromey <tromey@adacore.com>
|
||||
|
||||
* netbsd-low.h (class netbsd_process_target) <wait>: Update.
|
||||
* netbsd-low.cc (netbsd_waitpid, netbsd_wait)
|
||||
(netbsd_process_target::wait): Change type of target_options.
|
||||
* win32-low.h (class win32_process_target) <wait>: Update.
|
||||
* win32-low.cc (win32_process_target::wait): Update.
|
||||
* target.h (class process_stratum_target) <wait>: Update.
|
||||
(mywait): Update.
|
||||
* target.cc (mywait, target_wait): Change type of "options".
|
||||
* linux-low.h (class linux_process_target) <wait, wait_1>:
|
||||
Update.
|
||||
* linux-low.cc (linux_process_target::wait)
|
||||
(linux_process_target::wait_1): Update.
|
||||
|
||||
2020-09-15 Tom Tromey <tromey@adacore.com>
|
||||
|
||||
* linux-x86-low.cc (xmltarget_i386_linux_no_xml)
|
||||
|
|
|
@ -2948,7 +2948,7 @@ linux_process_target::gdb_catch_this_syscall (lwp_info *event_child)
|
|||
|
||||
ptid_t
|
||||
linux_process_target::wait_1 (ptid_t ptid, target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
client_state &cs = get_client_state ();
|
||||
int w;
|
||||
|
@ -3710,7 +3710,7 @@ async_file_mark (void)
|
|||
ptid_t
|
||||
linux_process_target::wait (ptid_t ptid,
|
||||
target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
ptid_t event_ptid;
|
||||
|
||||
|
|
|
@ -157,7 +157,7 @@ public:
|
|||
void resume (thread_resume *resume_info, size_t n) override;
|
||||
|
||||
ptid_t wait (ptid_t ptid, target_waitstatus *status,
|
||||
int options) override;
|
||||
target_wait_flags options) override;
|
||||
|
||||
void fetch_registers (regcache *regcache, int regno) override;
|
||||
|
||||
|
@ -356,7 +356,7 @@ private:
|
|||
|
||||
/* Wait for process, returns status. */
|
||||
ptid_t wait_1 (ptid_t ptid, target_waitstatus *ourstatus,
|
||||
int target_options);
|
||||
target_wait_flags target_options);
|
||||
|
||||
/* Stop all lwps that aren't stopped yet, except EXCEPT, if not NULL.
|
||||
If SUSPEND, then also increase the suspend count of every LWP,
|
||||
|
|
|
@ -236,9 +236,11 @@ netbsd_store_waitstatus (struct target_waitstatus *ourstatus, int hoststatus)
|
|||
/* Implement a safe wrapper around waitpid(). */
|
||||
|
||||
static pid_t
|
||||
netbsd_waitpid (ptid_t ptid, struct target_waitstatus *ourstatus, int options)
|
||||
netbsd_waitpid (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
int status;
|
||||
int options = (target_options & TARGET_WNOHANG) ? WNOHANG : 0;
|
||||
|
||||
pid_t pid
|
||||
= gdb::handle_eintr<int> (-1, ::waitpid, ptid.pid (), &status, options);
|
||||
|
@ -259,7 +261,7 @@ netbsd_waitpid (ptid_t ptid, struct target_waitstatus *ourstatus, int options)
|
|||
|
||||
static ptid_t
|
||||
netbsd_wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
pid_t pid = netbsd_waitpid (ptid, ourstatus, target_options);
|
||||
ptid_t wptid = ptid_t (pid);
|
||||
|
@ -398,7 +400,7 @@ netbsd_wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
|||
|
||||
ptid_t
|
||||
netbsd_process_target::wait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
int target_options)
|
||||
target_wait_flags target_options)
|
||||
{
|
||||
while (true)
|
||||
{
|
||||
|
|
|
@ -76,7 +76,7 @@ public:
|
|||
void resume (thread_resume *resume_info, size_t n) override;
|
||||
|
||||
ptid_t wait (ptid_t ptid, target_waitstatus *status,
|
||||
int options) override;
|
||||
target_wait_flags options) override;
|
||||
|
||||
void fetch_registers (regcache *regcache, int regno) override;
|
||||
|
||||
|
|
|
@ -160,8 +160,8 @@ target_write_memory (CORE_ADDR memaddr, const unsigned char *myaddr,
|
|||
}
|
||||
|
||||
ptid_t
|
||||
mywait (ptid_t ptid, struct target_waitstatus *ourstatus, int options,
|
||||
int connected_wait)
|
||||
mywait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
target_wait_flags options, int connected_wait)
|
||||
{
|
||||
ptid_t ret;
|
||||
|
||||
|
@ -220,7 +220,8 @@ target_stop_and_wait (ptid_t ptid)
|
|||
/* See target/target.h. */
|
||||
|
||||
ptid_t
|
||||
target_wait (ptid_t ptid, struct target_waitstatus *status, int options)
|
||||
target_wait (ptid_t ptid, struct target_waitstatus *status,
|
||||
target_wait_flags options)
|
||||
{
|
||||
return the_target->wait (ptid, status, options);
|
||||
}
|
||||
|
|
|
@ -128,7 +128,7 @@ public:
|
|||
no child stop to report, return is
|
||||
null_ptid/TARGET_WAITKIND_IGNORE. */
|
||||
virtual ptid_t wait (ptid_t ptid, target_waitstatus *status,
|
||||
int options) = 0;
|
||||
target_wait_flags options) = 0;
|
||||
|
||||
/* Fetch registers from the inferior process.
|
||||
|
||||
|
@ -663,8 +663,8 @@ target_read_btrace_conf (struct btrace_target_info *tinfo,
|
|||
#define target_supports_software_single_step() \
|
||||
the_target->supports_software_single_step ()
|
||||
|
||||
ptid_t mywait (ptid_t ptid, struct target_waitstatus *ourstatus, int options,
|
||||
int connected_wait);
|
||||
ptid_t mywait (ptid_t ptid, struct target_waitstatus *ourstatus,
|
||||
target_wait_flags options, int connected_wait);
|
||||
|
||||
/* Prepare to read or write memory from the inferior process. See the
|
||||
corresponding process_stratum_target methods for more details. */
|
||||
|
|
|
@ -1610,7 +1610,7 @@ get_child_debug_event (DWORD *continue_status,
|
|||
Returns the signal which caused the process to stop. */
|
||||
ptid_t
|
||||
win32_process_target::wait (ptid_t ptid, target_waitstatus *ourstatus,
|
||||
int options)
|
||||
target_wait_flags options)
|
||||
{
|
||||
if (cached_status.kind != TARGET_WAITKIND_IGNORE)
|
||||
{
|
||||
|
|
|
@ -116,7 +116,7 @@ public:
|
|||
void resume (thread_resume *resume_info, size_t n) override;
|
||||
|
||||
ptid_t wait (ptid_t ptid, target_waitstatus *status,
|
||||
int options) override;
|
||||
target_wait_flags options) override;
|
||||
|
||||
void fetch_registers (regcache *regcache, int regno) override;
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue