From f6ac5f3d63e03a81c4ff3749aba234961cc9090e Mon Sep 17 00:00:00 2001 From: Pedro Alves Date: Thu, 3 May 2018 00:37:22 +0100 Subject: Convert struct target_ops to C++ I.e., use C++ virtual methods and inheritance instead of tables of function pointers. Unfortunately, there's no way to do a smooth transition. ALL native targets in the tree must be converted at the same time. I've tested all I could with cross compilers and with help from GCC compile farm, but naturally I haven't been able to test many of the ports. Still, I made a best effort to port everything over, and while I expect some build problems due to typos and such, which should be trivial to fix, I don't expect any design problems. * Implementation notes: - The flattened current_target is gone. References to current_target or current_target.beneath are replaced with references to target_stack (the top of the stack) directly. - To keep "set debug target" working, this adds a new debug_stratum layer that sits on top of the stack, prints the debug, and delegates to the target beneath. In addition, this makes the shortname and longname properties of target_ops be virtual methods instead of data fields, and makes the debug target defer those to the target beneath. This is so that debug code sprinkled around that does "if (debugtarget) ..." can transparently print the name of the target beneath. A patch later in the series actually splits out the shortname/longname methods to a separate structure, but I preferred to keep that chance separate as it is associated with changing a bit the design of how targets are registered and open. - Since you can't check whether a C++ virtual method is overridden, the old method of checking whether a target_ops implements a method by comparing the function pointer must be replaced with something else. Some cases are fixed by adding a parallel "can_do_foo" target_ops methods. E.g.,: + for (t = target_stack; t != NULL; t = t->beneath) { - if (t->to_create_inferior != NULL) + if (t->can_create_inferior ()) break; } Others are fixed by changing void return type to bool or int return type, and have the default implementation return false or -1, to indicate lack of support. - make-target-delegates was adjusted to generate C++ classes and methods. It needed tweaks to grok "virtual" in front of the target method name, and for the fact that methods are no longer function pointers. (In particular, the current code parsing the return type was simple because it could simply parse up until the '(' in '(*to_foo)'. It now generates a couple C++ classes that inherit target_ops: dummy_target and debug_target. Since we need to generate the class declarations as well, i.e., we need to emit methods twice, we now generate the code in two passes. - The core_target global is renamed to avoid conflict with the "core_target" class. - ctf/tfile targets init_tracefile_ops is replaced by a base class that is inherited by both ctf and tfile. - bsd-uthread The bsd_uthread_ops_hack hack is gone. It's not needed because nothing was extending a target created by bsd_uthread_target. - remote/extended-remote targets This is a first pass, just enough to C++ify target_ops. A later pass will convert more free functions to methods, and make remote_state be truly per remote instance, allowing multiple simultaneous instances of remote targets. - inf-child/"native" is converted to an actual base class (inf_child_target), that is inherited by all native targets. - GNU/Linux The old weird double-target linux_ops mechanism in linux-nat.c, is gone, replaced by adding a few virtual methods to linux-nat.h's target_ops, called low_XXX, that the concrete linux-nat implementations override. Sort of like gdbserver's linux_target_ops, but simpler, for requiring only one target_ops-like hierarchy, which spares implementing the same method twice when we need to forward the method to a low implementation. The low target simply reimplements the target_ops method directly in that case. There are a few remaining linux-nat.c hooks that would be better converted to low_ methods like above too. E.g.: linux_nat_set_new_thread (t, x86_linux_new_thread); linux_nat_set_new_fork (t, x86_linux_new_fork); linux_nat_set_forget_process That'll be done in a follow up patch. - We can no longer use functions like x86_use_watchpoints to install custom methods on an arbitrary base target. The patch replaces instances of such a pattern with template mixins. For example memory_breakpoint_target defined in target.h, or x86_nat_target in x86-nat.h. - linux_trad_target, MIPS and Alpha GNU/Linux The code in the new linux-nat-trad.h/c files which was split off of inf-ptrace.h/c recently, is converted to a C++ base class, and used by the MIPS and Alpha GNU/Linux ports. - BSD targets The $architecture x NetBSD/OpenBSD/FreeBSD support matrix complicates things a bit. There's common BSD target code, and there's common architecture-specific code shared between the different BSDs. Currently, all that is stiched together to form a final target, via the i386bsd_target, x86bsd_target, fbsd_nat_add_target functions etc. This introduces new fbsd_nat_target, obsd_nat_target and nbsd_nat_target classes that serve as base/prototype target for the corresponding BSD variant. And introduces generic i386/AMD64 BSD targets, to be used as template mixin to build a final target. Similarly, a generic SPARC target is added, used by both BSD and Linux ports. - bsd_kvm_add_target, BSD libkvm target I considered making bsd_kvm_supply_pcb a virtual method, and then have each port inherit bsd_kvm_target and override that method, but that was resulting in lots of unjustified churn, so I left the function pointer mechanism alone. gdb/ChangeLog: 2018-05-02 Pedro Alves John Baldwin * target.h (enum strata) : New. (struct target_ops) : Replace by C++ virtual methods, and drop "to_" prefix. All references updated throughout. : Delete, replaced by virtual methods. All references updated throughout. : New virtual methods. : Now TARGET_DEFAULT_NORETURN methods. : Now returns bool. : Delete. (OPS_MAGIC): Delete. (current_target): Delete. All references replaced by references to ... (target_stack): ... this. New. (target_shortname, target_longname): Adjust. (target_can_run): Now a function declaration. (default_child_has_all_memory, default_child_has_memory) (default_child_has_stack, default_child_has_registers) (default_child_has_execution): Remove target_ops parameter. (complete_target_initialization): Delete. (memory_breakpoint_target): New template class. (test_target_ops): Refactor as a C++ class with virtual methods. * make-target-delegates (NAME_PART): Tighten. (POINTER_PART, CP_SYMBOL): New. (SIMPLE_RETURN_PART): Reimplement. (VEC_RETURN_PART): Expect less. (RETURN_PART, VIRTUAL_PART): New. (METHOD): Adjust to C++ virtual methods. (scan_target_h): Remove reference to C99. (dname): Output "target_ops::" prefix. (write_function_header): Adjust to output a C++ class method. (write_declaration): New. (write_delegator): Adjust to output a C++ class method. (tdname): Output "dummy_target::" prefix. (write_tdefault, write_debugmethod): Adjust to output a C++ class method. (tdefault_names, debug_names): Delete. (return_types, tdefaults, styles, argtypes_array): New. (top level): All methods are delegators. (print_class): New. (top level): Print dummy_target and debug_target classes. * target-delegates.c: Regenerate. * target-debug.h (target_debug_print_enum_info_proc_what) (target_debug_print_thread_control_capabilities) (target_debug_print_thread_info_p): New. * target.c (dummy_target): Delete. (the_dummy_target, the_debug_target): New. (target_stack): Now extern. (set_targetdebug): Push/unpush debug target. (default_child_has_all_memory, default_child_has_memory) (default_child_has_stack, default_child_has_registers) (default_child_has_execution): Remove target_ops parameter. (complete_target_initialization): Delete. (add_target_with_completer): No longer call complete_target_initialization. (target_supports_terminal_ours): Use regular delegation. (update_current_target): Delete. (push_target): No longer check magic number. Don't call update_current_target. (unpush_target): Don't call update_current_target. (target_is_pushed): No longer check magic number. (target_require_runnable): Skip for all stratums over process_stratum. (target_ops::info_proc): New. (target_info_proc): Use find_target_at and find_default_run_target. (target_supports_disable_randomization): Use regular delegation. (target_get_osdata): Use find_target_at. (target_ops::open, target_ops::close, target_ops::can_attach) (target_ops::attach, target_ops::can_create_inferior) (target_ops::create_inferior, target_ops::can_run) (target_can_run): New. (default_fileio_target): Use regular delegation. (target_ops::fileio_open, target_ops::fileio_pwrite) (target_ops::fileio_pread, target_ops::fileio_fstat) (target_ops::fileio_close, target_ops::fileio_unlink) (target_ops::fileio_readlink): New. (target_fileio_open_1, target_fileio_unlink) (target_fileio_readlink): Always call the target method. Handle FILEIO_ENOSYS. (return_zero, return_zero_has_execution): Delete. (init_dummy_target): Delete. (dummy_target::dummy_target, dummy_target::shortname) (dummy_target::longname, dummy_target::doc) (debug_target::debug_target, debug_target::shortname) (debug_target::longname, debug_target::doc): New. (target_supports_delete_record): Use regular delegation. (setup_target_debug): Delete. (maintenance_print_target_stack): Skip debug_stratum. (initialize_targets): Instantiate the_dummy_target and the_debug_target. * auxv.c (target_auxv_parse): Remove 'ops' parameter. Adjust to use target_stack. (target_auxv_search, fprint_target_auxv): Adjust. (info_auxv_command): Adjust to use target_stack. * auxv.h (target_auxv_parse): Remove 'ops' parameter. * exceptions.c (print_flush): Handle a NULL target_stack. * regcache.c (target_ops_no_register): Refactor as class with virtual methods. * exec.c (exec_target): New class. (exec_ops): Now an exec_target. (exec_open, exec_close_1, exec_get_section_table) (exec_xfer_partial, exec_files_info, exec_has_memory) (exec_make_note_section): Refactor as exec_target methods. (exec_file_clear, ignore, exec_remove_breakpoint, init_exec_ops): Delete. (exec_target::find_memory_regions): New. (_initialize_exec): Don't call init_exec_ops. * gdbcore.h (exec_file_clear): Delete. * corefile.c (core_target): Delete. (core_file_command): Adjust. * corelow.c (core_target): New class. (the_core_target): New. (core_close): Remove target_ops parameter. (core_close_cleanup): Adjust. (core_target::close): New. (core_open, core_detach, get_core_registers, core_files_info) (core_xfer_partial, core_thread_alive, core_read_description) (core_pid_to_str, core_thread_name, core_has_memory) (core_has_stack, core_has_registers, core_info_proc): Rework as core_target methods. (ignore, core_remove_breakpoint, init_core_ops): Delete. (_initialize_corelow): Initialize the_core_target. * gdbcore.h (core_target): Delete. (the_core_target): New. * ctf.c: (ctf_target): New class. (ctf_ops): Now a ctf_target. (ctf_open, ctf_close, ctf_files_info, ctf_fetch_registers) (ctf_xfer_partial, ctf_get_trace_state_variable_value) (ctf_trace_find, ctf_traceframe_info): Refactor as ctf_target methods. (init_ctf_ops): Delete. (_initialize_ctf): Don't call it. * tracefile-tfile.c (tfile_target): New class. (tfile_ops): Now a tfile_target. (tfile_open, tfile_close, tfile_files_info) (tfile_get_tracepoint_status, tfile_trace_find) (tfile_fetch_registers, tfile_xfer_partial) (tfile_get_trace_state_variable_value, tfile_traceframe_info): Refactor as tfile_target methods. (tfile_xfer_partial_features): Remove target_ops parameter. (init_tfile_ops): Delete. (_initialize_tracefile_tfile): Don't call it. * tracefile.c (tracefile_has_all_memory, tracefile_has_memory) (tracefile_has_stack, tracefile_has_registers) (tracefile_thread_alive, tracefile_get_trace_status): Refactor as tracefile_target methods. (init_tracefile_ops): Delete. (tracefile_target::tracefile_target): New. * tracefile.h: Include "target.h". (tracefile_target): New class. (init_tracefile_ops): Delete. * spu-multiarch.c (spu_multiarch_target): New class. (spu_ops): Now a spu_multiarch_target. (spu_thread_architecture, spu_region_ok_for_hw_watchpoint) (spu_fetch_registers, spu_store_registers, spu_xfer_partial) (spu_search_memory, spu_mourn_inferior): Refactor as spu_multiarch_target methods. (init_spu_ops): Delete. (_initialize_spu_multiarch): Remove references to init_spu_ops, complete_target_initialization. * ravenscar-thread.c (ravenscar_thread_target): New class. (ravenscar_ops): Now a ravenscar_thread_target. (ravenscar_resume, ravenscar_wait, ravenscar_update_thread_list) (ravenscar_thread_alive, ravenscar_pid_to_str) (ravenscar_fetch_registers, ravenscar_store_registers) (ravenscar_prepare_to_store, ravenscar_stopped_by_sw_breakpoint) (ravenscar_stopped_by_hw_breakpoint) (ravenscar_stopped_by_watchpoint, ravenscar_stopped_data_address) (ravenscar_mourn_inferior, ravenscar_core_of_thread) (ravenscar_get_ada_task_ptid): Refactor as ravenscar_thread_target methods. (init_ravenscar_thread_ops): Delete. (_initialize_ravenscar): Remove references to init_ravenscar_thread_ops and complete_target_initialization. * bsd-uthread.c (bsd_uthread_ops_hack): Delete. (bsd_uthread_target): New class. (bsd_uthread_ops): Now a bsd_uthread_target. (bsd_uthread_activate): Adjust to refer to bsd_uthread_ops. (bsd_uthread_close, bsd_uthread_mourn_inferior) (bsd_uthread_fetch_registers, bsd_uthread_store_registers) (bsd_uthread_wait, bsd_uthread_resume, bsd_uthread_thread_alive) (bsd_uthread_update_thread_list, bsd_uthread_extra_thread_info) (bsd_uthread_pid_to_str): Refactor as bsd_uthread_target methods. (bsd_uthread_target): Delete function. (_initialize_bsd_uthread): Remove reference to complete_target_initialization. * bfd-target.c (target_bfd_data): Delete. Fields folded into ... (target_bfd): ... this new class. (target_bfd_xfer_partial, target_bfd_get_section_table) (target_bfd_close): Refactor as target_bfd methods. (target_bfd::~target_bfd): New. (target_bfd_reopen): Adjust. (target_bfd::close): New. * record-btrace.c (record_btrace_target): New class. (record_btrace_ops): Now a record_btrace_target. (record_btrace_open, record_btrace_stop_recording) (record_btrace_disconnect, record_btrace_close) (record_btrace_async, record_btrace_info) (record_btrace_insn_history, record_btrace_insn_history_range) (record_btrace_insn_history_from, record_btrace_call_history) (record_btrace_call_history_range) (record_btrace_call_history_from, record_btrace_record_method) (record_btrace_is_replaying, record_btrace_will_replay) (record_btrace_xfer_partial, record_btrace_insert_breakpoint) (record_btrace_remove_breakpoint, record_btrace_fetch_registers) (record_btrace_store_registers, record_btrace_prepare_to_store) (record_btrace_to_get_unwinder) (record_btrace_to_get_tailcall_unwinder, record_btrace_resume) (record_btrace_commit_resume, record_btrace_wait) (record_btrace_stop, record_btrace_can_execute_reverse) (record_btrace_stopped_by_sw_breakpoint) (record_btrace_supports_stopped_by_sw_breakpoint) (record_btrace_stopped_by_hw_breakpoint) (record_btrace_supports_stopped_by_hw_breakpoint) (record_btrace_update_thread_list, record_btrace_thread_alive) (record_btrace_goto_begin, record_btrace_goto_end) (record_btrace_goto, record_btrace_stop_replaying_all) (record_btrace_execution_direction) (record_btrace_prepare_to_generate_core) (record_btrace_done_generating_core): Refactor as record_btrace_target methods. (init_record_btrace_ops): Delete. (_initialize_record_btrace): Remove reference to init_record_btrace_ops. * record-full.c (RECORD_FULL_IS_REPLAY): Adjust to always refer to the execution_direction global. (record_full_base_target, record_full_target) (record_full_core_target): New classes. (record_full_ops): Now a record_full_target. (record_full_core_ops): Now a record_full_core_target. (record_full_target::detach, record_full_target::disconnect) (record_full_core_target::disconnect) (record_full_target::mourn_inferior, record_full_target::kill): New. (record_full_open, record_full_close, record_full_async): Refactor as methods of the record_full_base_target class. (record_full_resume, record_full_commit_resume): Refactor as methods of the record_full_target class. (record_full_wait, record_full_stopped_by_watchpoint) (record_full_stopped_data_address) (record_full_stopped_by_sw_breakpoint) (record_full_supports_stopped_by_sw_breakpoint) (record_full_stopped_by_hw_breakpoint) (record_full_supports_stopped_by_hw_breakpoint): Refactor as methods of the record_full_base_target class. (record_full_store_registers, record_full_xfer_partial) (record_full_insert_breakpoint, record_full_remove_breakpoint): Refactor as methods of the record_full_target class. (record_full_can_execute_reverse, record_full_get_bookmark) (record_full_goto_bookmark, record_full_execution_direction) (record_full_record_method, record_full_info, record_full_delete) (record_full_is_replaying, record_full_will_replay) (record_full_goto_begin, record_full_goto_end, record_full_goto) (record_full_stop_replaying): Refactor as methods of the record_full_base_target class. (record_full_core_resume, record_full_core_kill) (record_full_core_fetch_registers) (record_full_core_prepare_to_store) (record_full_core_store_registers, record_full_core_xfer_partial) (record_full_core_insert_breakpoint) (record_full_core_remove_breakpoint) (record_full_core_has_execution): Refactor as methods of the record_full_core_target class. (record_full_base_target::supports_delete_record): New. (init_record_full_ops): Delete. (init_record_full_core_ops): Delete. (record_full_save): Refactor as method of the record_full_base_target class. (_initialize_record_full): Remove references to init_record_full_ops and init_record_full_core_ops. * remote.c (remote_target, extended_remote_target): New classes. (remote_ops): Now a remote_target. (extended_remote_ops): Now an extended_remote_target. (remote_insert_fork_catchpoint, remote_remove_fork_catchpoint) (remote_insert_vfork_catchpoint, remote_remove_vfork_catchpoint) (remote_insert_exec_catchpoint, remote_remove_exec_catchpoint) (remote_pass_signals, remote_set_syscall_catchpoint) (remote_program_signals, ) (remote_thread_always_alive): Remove target_ops parameter. (remote_thread_alive, remote_thread_name) (remote_update_thread_list, remote_threads_extra_info) (remote_static_tracepoint_marker_at) (remote_static_tracepoint_markers_by_strid) (remote_get_ada_task_ptid, remote_close, remote_start_remote) (remote_open): Refactor as methods of remote_target. (extended_remote_open, extended_remote_detach) (extended_remote_attach, extended_remote_post_attach): (extended_remote_supports_disable_randomization) (extended_remote_create_inferior): : Refactor as method of extended_remote_target. (remote_set_permissions, remote_open_1, remote_detach) (remote_follow_fork, remote_follow_exec, remote_disconnect) (remote_resume, remote_commit_resume, remote_stop) (remote_interrupt, remote_pass_ctrlc, remote_terminal_inferior) (remote_terminal_ours, remote_wait, remote_fetch_registers) (remote_prepare_to_store, remote_store_registers) (remote_flash_erase, remote_flash_done, remote_files_info) (remote_kill, remote_mourn, remote_insert_breakpoint) (remote_remove_breakpoint, remote_insert_watchpoint) (remote_watchpoint_addr_within_range) (remote_remove_watchpoint, remote_region_ok_for_hw_watchpoint) (remote_check_watch_resources, remote_stopped_by_sw_breakpoint) (remote_supports_stopped_by_sw_breakpoint) (remote_stopped_by_hw_breakpoint) (remote_supports_stopped_by_hw_breakpoint) (remote_stopped_by_watchpoint, remote_stopped_data_address) (remote_insert_hw_breakpoint, remote_remove_hw_breakpoint) (remote_verify_memory): Refactor as methods of remote_target. (remote_write_qxfer, remote_read_qxfer): Remove target_ops parameter. (remote_xfer_partial, remote_get_memory_xfer_limit) (remote_search_memory, remote_rcmd, remote_memory_map) (remote_pid_to_str, remote_get_thread_local_address) (remote_get_tib_address, remote_read_description): Refactor as methods of remote_target. (remote_target::fileio_open, remote_target::fileio_pwrite) (remote_target::fileio_pread, remote_target::fileio_close): New. (remote_hostio_readlink, remote_hostio_fstat) (remote_filesystem_is_local, remote_can_execute_reverse) (remote_supports_non_stop, remote_supports_disable_randomization) (remote_supports_multi_process, remote_supports_cond_breakpoints) (remote_supports_enable_disable_tracepoint) (remote_supports_string_tracing) (remote_can_run_breakpoint_commands, remote_trace_init) (remote_download_tracepoint, remote_can_download_tracepoint) (remote_download_trace_state_variable, remote_enable_tracepoint) (remote_disable_tracepoint, remote_trace_set_readonly_regions) (remote_trace_start, remote_get_trace_status) (remote_get_tracepoint_status, remote_trace_stop) (remote_trace_find, remote_get_trace_state_variable_value) (remote_save_trace_data, remote_get_raw_trace_data) (remote_set_disconnected_tracing, remote_core_of_thread) (remote_set_circular_trace_buffer, remote_traceframe_info) (remote_get_min_fast_tracepoint_insn_len) (remote_set_trace_buffer_size, remote_set_trace_notes) (remote_use_agent, remote_can_use_agent, remote_enable_btrace) (remote_disable_btrace, remote_teardown_btrace) (remote_read_btrace, remote_btrace_conf) (remote_augmented_libraries_svr4_read, remote_load) (remote_pid_to_exec_file, remote_can_do_single_step) (remote_execution_direction, remote_thread_handle_to_thread_info): Refactor as methods of remote_target. (init_remote_ops, init_extended_remote_ops): Delete. (remote_can_async_p, remote_is_async_p, remote_async) (remote_thread_events, remote_upload_tracepoints) (remote_upload_trace_state_variables): Refactor as methods of remote_target. (_initialize_remote): Remove references to init_remote_ops and init_extended_remote_ops. * remote-sim.c (gdbsim_target): New class. (gdbsim_fetch_register, gdbsim_store_register, gdbsim_kill) (gdbsim_load, gdbsim_create_inferior, gdbsim_open, gdbsim_close) (gdbsim_detach, gdbsim_resume, gdbsim_interrupt) (gdbsim_wait, gdbsim_prepare_to_store, gdbsim_xfer_partial) (gdbsim_files_info, gdbsim_mourn_inferior, gdbsim_thread_alive) (gdbsim_pid_to_str, gdbsim_has_all_memory, gdbsim_has_memory): Refactor as methods of gdbsim_target. (gdbsim_ops): Now a gdbsim_target. (init_gdbsim_ops): Delete. (gdbsim_cntrl_c): Adjust. (_initialize_remote_sim): Remove reference to init_gdbsim_ops. * amd64-linux-nat.c (amd64_linux_nat_target): New class. (the_amd64_linux_nat_target): New. (amd64_linux_fetch_inferior_registers) (amd64_linux_store_inferior_registers): Refactor as methods of amd64_linux_nat_target. (_initialize_amd64_linux_nat): Adjust. Set linux_target. * i386-linux-nat.c: Don't include "linux-nat.h". (i386_linux_nat_target): New class. (the_i386_linux_nat_target): New. (i386_linux_fetch_inferior_registers) (i386_linux_store_inferior_registers, i386_linux_resume): Refactor as methods of i386_linux_nat_target. (_initialize_i386_linux_nat): Adjust. Set linux_target. * inf-child.c (inf_child_ops): Delete. (inf_child_fetch_inferior_registers) (inf_child_store_inferior_registers): Delete. (inf_child_post_attach, inf_child_prepare_to_store): Refactor as methods of inf_child_target. (inf_child_target::supports_terminal_ours) (inf_child_target::terminal_init) (inf_child_target::terminal_inferior) (inf_child_target::terminal_ours_for_output) (inf_child_target::terminal_ours, inf_child_target::interrupt) (inf_child_target::pass_ctrlc, inf_child_target::terminal_info): New. (inf_child_open, inf_child_disconnect, inf_child_close) (inf_child_mourn_inferior, inf_child_maybe_unpush_target) (inf_child_post_startup_inferior, inf_child_can_run) (inf_child_pid_to_exec_file): Refactor as methods of inf_child_target. (inf_child_follow_fork): Delete. (inf_child_target::can_create_inferior) (inf_child_target::can_attach): New. (inf_child_target::has_all_memory, inf_child_target::has_memory) (inf_child_target::has_stack, inf_child_target::has_registers) (inf_child_target::has_execution): New. (inf_child_fileio_open, inf_child_fileio_pwrite) (inf_child_fileio_pread, inf_child_fileio_fstat) (inf_child_fileio_close, inf_child_fileio_unlink) (inf_child_fileio_readlink, inf_child_use_agent) (inf_child_can_use_agent): Refactor as methods of inf_child_target. (return_zero, inf_child_target): Delete. (inf_child_target::inf_child_target): New. * inf-child.h: Include "target.h". (inf_child_target): Delete function prototype. (inf_child_target): New class. (inf_child_open_target, inf_child_mourn_inferior) (inf_child_maybe_unpush_target): Delete. * inf-ptrace.c (inf_ptrace_target::~inf_ptrace_target): New. (inf_ptrace_follow_fork, inf_ptrace_insert_fork_catchpoint) (inf_ptrace_remove_fork_catchpoint, inf_ptrace_create_inferior) (inf_ptrace_post_startup_inferior, inf_ptrace_mourn_inferior) (inf_ptrace_attach, inf_ptrace_post_attach, inf_ptrace_detach) (inf_ptrace_detach_success, inf_ptrace_kill, inf_ptrace_resume) (inf_ptrace_wait, inf_ptrace_xfer_partial) (inf_ptrace_thread_alive, inf_ptrace_files_info) (inf_ptrace_pid_to_str, inf_ptrace_auxv_parse): Refactor as methods of inf_ptrace_target. (inf_ptrace_target): Delete function. * inf-ptrace.h: Include "inf-child.h". (inf_ptrace_target): Delete function declaration. (inf_ptrace_target): New class. (inf_ptrace_trad_target, inf_ptrace_detach_success): Delete. * linux-nat.c (linux_target): New. (linux_ops, linux_ops_saved, super_xfer_partial): Delete. (linux_nat_target::~linux_nat_target): New. (linux_child_post_attach, linux_child_post_startup_inferior) (linux_child_follow_fork, linux_child_insert_fork_catchpoint) (linux_child_remove_fork_catchpoint) (linux_child_insert_vfork_catchpoint) (linux_child_remove_vfork_catchpoint) (linux_child_insert_exec_catchpoint) (linux_child_remove_exec_catchpoint) (linux_child_set_syscall_catchpoint, linux_nat_pass_signals) (linux_nat_create_inferior, linux_nat_attach, linux_nat_detach) (linux_nat_resume, linux_nat_stopped_by_watchpoint) (linux_nat_stopped_data_address) (linux_nat_stopped_by_sw_breakpoint) (linux_nat_supports_stopped_by_sw_breakpoint) (linux_nat_stopped_by_hw_breakpoint) (linux_nat_supports_stopped_by_hw_breakpoint, linux_nat_wait) (linux_nat_kill, linux_nat_mourn_inferior) (linux_nat_xfer_partial, linux_nat_thread_alive) (linux_nat_update_thread_list, linux_nat_pid_to_str) (linux_nat_thread_name, linux_child_pid_to_exec_file) (linux_child_static_tracepoint_markers_by_strid) (linux_nat_is_async_p, linux_nat_can_async_p) (linux_nat_supports_non_stop, linux_nat_always_non_stop_p) (linux_nat_supports_multi_process) (linux_nat_supports_disable_randomization, linux_nat_async) (linux_nat_stop, linux_nat_close, linux_nat_thread_address_space) (linux_nat_core_of_thread, linux_nat_filesystem_is_local) (linux_nat_fileio_open, linux_nat_fileio_readlink) (linux_nat_fileio_unlink, linux_nat_thread_events): Refactor as methods of linux_nat_target. (linux_nat_wait_1, linux_xfer_siginfo, linux_proc_xfer_partial) (linux_proc_xfer_spu, linux_nat_xfer_osdata): Remove target_ops parameter. (check_stopped_by_watchpoint): Adjust. (linux_xfer_partial): Delete. (linux_target_install_ops, linux_target, linux_nat_add_target): Delete. (linux_nat_target::linux_nat_target): New. * linux-nat.h: Include "inf-ptrace.h". (linux_nat_target): New. (linux_target, linux_target_install_ops, linux_nat_add_target): Delete function declarations. (linux_target): Declare global. * linux-thread-db.c (thread_db_target): New. (thread_db_target::thread_db_target): New. (thread_db_ops): Delete. (the_thread_db_target): New. (thread_db_detach, thread_db_wait, thread_db_mourn_inferior) (thread_db_update_thread_list, thread_db_pid_to_str) (thread_db_extra_thread_info) (thread_db_thread_handle_to_thread_info) (thread_db_get_thread_local_address, thread_db_get_ada_task_ptid) (thread_db_resume): Refactor as methods of thread_db_target. (init_thread_db_ops): Delete. (_initialize_thread_db): Remove reference to init_thread_db_ops. * x86-linux-nat.c: Don't include "linux-nat.h". (super_post_startup_inferior): Delete. (x86_linux_nat_target::~x86_linux_nat_target): New. (x86_linux_child_post_startup_inferior) (x86_linux_read_description, x86_linux_enable_btrace) (x86_linux_disable_btrace, x86_linux_teardown_btrace) (x86_linux_read_btrace, x86_linux_btrace_conf): Refactor as methods of x86_linux_nat_target. (x86_linux_create_target): Delete. Bits folded ... (x86_linux_add_target): ... here. Now takes a linux_nat_target pointer. * x86-linux-nat.h: Include "linux-nat.h" and "x86-nat.h". (x86_linux_nat_target): New class. (x86_linux_create_target): Delete. (x86_linux_add_target): Now takes a linux_nat_target pointer. * x86-nat.c (x86_insert_watchpoint, x86_remove_watchpoint) (x86_region_ok_for_watchpoint, x86_stopped_data_address) (x86_stopped_by_watchpoint, x86_insert_hw_breakpoint) (x86_remove_hw_breakpoint, x86_can_use_hw_breakpoint) (x86_stopped_by_hw_breakpoint): Remove target_ops parameter and make extern. (x86_use_watchpoints): Delete. * x86-nat.h: Include "breakpoint.h" and "target.h". (x86_use_watchpoints): Delete. (x86_can_use_hw_breakpoint, x86_region_ok_for_hw_watchpoint) (x86_stopped_by_watchpoint, x86_stopped_data_address) (x86_insert_watchpoint, x86_remove_watchpoint) (x86_insert_hw_breakpoint, x86_remove_hw_breakpoint) (x86_stopped_by_hw_breakpoint): New declarations. (x86_nat_target): New template class. * ppc-linux-nat.c (ppc_linux_nat_target): New class. (the_ppc_linux_nat_target): New. (ppc_linux_fetch_inferior_registers) (ppc_linux_can_use_hw_breakpoint) (ppc_linux_region_ok_for_hw_watchpoint) (ppc_linux_ranged_break_num_registers) (ppc_linux_insert_hw_breakpoint, ppc_linux_remove_hw_breakpoint) (ppc_linux_insert_mask_watchpoint) (ppc_linux_remove_mask_watchpoint) (ppc_linux_can_accel_watchpoint_condition) (ppc_linux_insert_watchpoint, ppc_linux_remove_watchpoint) (ppc_linux_stopped_data_address, ppc_linux_stopped_by_watchpoint) (ppc_linux_watchpoint_addr_within_range) (ppc_linux_masked_watch_num_registers) (ppc_linux_store_inferior_registers, ppc_linux_auxv_parse) (ppc_linux_read_description): Refactor as methods of ppc_linux_nat_target. (_initialize_ppc_linux_nat): Adjust. Set linux_target. * procfs.c (procfs_xfer_partial): Delete forward declaration. (procfs_target): New class. (the_procfs_target): New. (procfs_target): Delete function. (procfs_auxv_parse, procfs_attach, procfs_detach) (procfs_fetch_registers, procfs_store_registers, procfs_wait) (procfs_xfer_partial, procfs_resume, procfs_pass_signals) (procfs_files_info, procfs_kill_inferior, procfs_mourn_inferior) (procfs_create_inferior, procfs_update_thread_list) (procfs_thread_alive, procfs_pid_to_str) (procfs_can_use_hw_breakpoint, procfs_stopped_by_watchpoint) (procfs_stopped_data_address, procfs_insert_watchpoint) (procfs_remove_watchpoint, procfs_region_ok_for_hw_watchpoint) (proc_find_memory_regions, procfs_info_proc) (procfs_make_note_section): Refactor as methods of procfs_target. (_initialize_procfs): Adjust. * sol-thread.c (sol_thread_target): New class. (sol_thread_ops): Now a sol_thread_target. (sol_thread_detach, sol_thread_resume, sol_thread_wait) (sol_thread_fetch_registers, sol_thread_store_registers) (sol_thread_xfer_partial, sol_thread_mourn_inferior) (sol_thread_alive, solaris_pid_to_str, sol_update_thread_list) (sol_get_ada_task_ptid): Refactor as methods of sol_thread_target. (init_sol_thread_ops): Delete. (_initialize_sol_thread): Adjust. Remove references to init_sol_thread_ops and complete_target_initialization. * windows-nat.c (windows_nat_target): New class. (windows_fetch_inferior_registers) (windows_store_inferior_registers, windows_resume, windows_wait) (windows_attach, windows_detach, windows_pid_to_exec_file) (windows_files_info, windows_create_inferior) (windows_mourn_inferior, windows_interrupt, windows_kill_inferior) (windows_close, windows_pid_to_str, windows_xfer_partial) (windows_get_tib_address, windows_get_ada_task_ptid) (windows_thread_name, windows_thread_alive): Refactor as windows_nat_target methods. (do_initial_windows_stuff): Adjust. (windows_target): Delete function. (_initialize_windows_nat): Adjust. * darwin-nat.c (darwin_resume, darwin_wait_to, darwin_interrupt) (darwin_mourn_inferior, darwin_kill_inferior) (darwin_create_inferior, darwin_attach, darwin_detach) (darwin_pid_to_str, darwin_thread_alive, darwin_xfer_partial) (darwin_pid_to_exec_file, darwin_get_ada_task_ptid) (darwin_supports_multi_process): Refactor as darwin_nat_target methods. (darwin_resume_to, darwin_files_info): Delete. (_initialize_darwin_inferior): Rename to ... (_initialize_darwin_nat): ... this. Adjust to C++ification. * darwin-nat.h: Include "inf-child.h". (darwin_nat_target): New class. (darwin_complete_target): Delete. * i386-darwin-nat.c (i386_darwin_nat_target): New class. (darwin_target): New. (i386_darwin_fetch_inferior_registers) (i386_darwin_store_inferior_registers): Refactor as methods of darwin_nat_target. (darwin_complete_target): Delete, with ... (_initialize_i386_darwin_nat): ... bits factored out here. * alpha-linux-nat.c (alpha_linux_nat_target): New class. (the_alpha_linux_nat_target): New. (alpha_linux_register_u_offset): Refactor as alpha_linux_nat_target method. (_initialize_alpha_linux_nat): Adjust. * linux-nat-trad.c (inf_ptrace_register_u_offset): Delete. (inf_ptrace_fetch_register, inf_ptrace_fetch_registers) (inf_ptrace_store_register, inf_ptrace_store_registers): Refact as methods of linux_nat_trad_target. (linux_trad_target): Delete. * linux-nat-trad.h (linux_trad_target): Delete function. (linux_nat_trad_target): New class. * mips-linux-nat.c (mips_linux_nat_target): New class. (super_fetch_registers, super_store_registers, super_close): Delete. (the_mips_linux_nat_target): New. (mips64_linux_regsets_fetch_registers) (mips64_linux_regsets_store_registers) (mips64_linux_fetch_registers, mips64_linux_store_registers) (mips_linux_register_u_offset, mips_linux_read_description) (mips_linux_can_use_hw_breakpoint) (mips_linux_stopped_by_watchpoint) (mips_linux_stopped_data_address) (mips_linux_region_ok_for_hw_watchpoint) (mips_linux_insert_watchpoint, mips_linux_remove_watchpoint) (mips_linux_close): Refactor as methods of mips_linux_nat. (_initialize_mips_linux_nat): Adjust to C++ification. * aix-thread.c (aix_thread_target): New class. (aix_thread_ops): Now an aix_thread_target. (aix_thread_detach, aix_thread_resume, aix_thread_wait) (aix_thread_fetch_registers, aix_thread_store_registers) (aix_thread_xfer_partial, aix_thread_mourn_inferior) (aix_thread_thread_alive, aix_thread_pid_to_str) (aix_thread_extra_thread_info, aix_thread_get_ada_task_ptid): Refactor as methods of aix_thread_target. (init_aix_thread_ops): Delete. (_initialize_aix_thread): Remove references to init_aix_thread_ops and complete_target_initialization. * rs6000-nat.c (rs6000_xfer_shared_libraries): Delete. (rs6000_nat_target): New class. (the_rs6000_nat_target): New. (rs6000_fetch_inferior_registers, rs6000_store_inferior_registers) (rs6000_xfer_partial, rs6000_wait, rs6000_create_inferior) (rs6000_xfer_shared_libraries): Refactor as rs6000_nat_target methods. (super_create_inferior): Delete. (_initialize_rs6000_nat): Adjust to C++ification. * arm-linux-nat.c (arm_linux_nat_target): New class. (the_arm_linux_nat_target): New. (arm_linux_fetch_inferior_registers) (arm_linux_store_inferior_registers, arm_linux_read_description) (arm_linux_can_use_hw_breakpoint, arm_linux_insert_hw_breakpoint) (arm_linux_remove_hw_breakpoint) (arm_linux_region_ok_for_hw_watchpoint) (arm_linux_insert_watchpoint, arm_linux_remove_watchpoint) (arm_linux_stopped_data_address, arm_linux_stopped_by_watchpoint) (arm_linux_watchpoint_addr_within_range): Refactor as methods of arm_linux_nat_target. (_initialize_arm_linux_nat): Adjust to C++ification. * aarch64-linux-nat.c (aarch64_linux_nat_target): New class. (the_aarch64_linux_nat_target): New. (aarch64_linux_fetch_inferior_registers) (aarch64_linux_store_inferior_registers) (aarch64_linux_child_post_startup_inferior) (aarch64_linux_read_description) (aarch64_linux_can_use_hw_breakpoint) (aarch64_linux_insert_hw_breakpoint) (aarch64_linux_remove_hw_breakpoint) (aarch64_linux_insert_watchpoint, aarch64_linux_remove_watchpoint) (aarch64_linux_region_ok_for_hw_watchpoint) (aarch64_linux_stopped_data_address) (aarch64_linux_stopped_by_watchpoint) (aarch64_linux_watchpoint_addr_within_range) (aarch64_linux_can_do_single_step): Refactor as methods of aarch64_linux_nat_target. (super_post_startup_inferior): Delete. (_initialize_aarch64_linux_nat): Adjust to C++ification. * hppa-linux-nat.c (hppa_linux_nat_target): New class. (the_hppa_linux_nat_target): New. (hppa_linux_fetch_inferior_registers) (hppa_linux_store_inferior_registers): Refactor as methods of hppa_linux_nat_target. (_initialize_hppa_linux_nat): Adjust to C++ification. * ia64-linux-nat.c (ia64_linux_nat_target): New class. (the_ia64_linux_nat_target): New. (ia64_linux_insert_watchpoint, ia64_linux_remove_watchpoint) (ia64_linux_stopped_data_address) (ia64_linux_stopped_by_watchpoint, ia64_linux_fetch_registers) (ia64_linux_store_registers, ia64_linux_xfer_partial): Refactor as ia64_linux_nat_target methods. (super_xfer_partial): Delete. (_initialize_ia64_linux_nat): Adjust to C++ification. * m32r-linux-nat.c (m32r_linux_nat_target): New class. (the_m32r_linux_nat_target): New. (m32r_linux_fetch_inferior_registers) (m32r_linux_store_inferior_registers): Refactor as m32r_linux_nat_target methods. (_initialize_m32r_linux_nat): Adjust to C++ification. * m68k-linux-nat.c (m68k_linux_nat_target): New class. (the_m68k_linux_nat_target): New. (m68k_linux_fetch_inferior_registers) (m68k_linux_store_inferior_registers): Refactor as m68k_linux_nat_target methods. (_initialize_m68k_linux_nat): Adjust to C++ification. * s390-linux-nat.c (s390_linux_nat_target): New class. (the_s390_linux_nat_target): New. (s390_linux_fetch_inferior_registers) (s390_linux_store_inferior_registers, s390_stopped_by_watchpoint) (s390_insert_watchpoint, s390_remove_watchpoint) (s390_can_use_hw_breakpoint, s390_insert_hw_breakpoint) (s390_remove_hw_breakpoint, s390_region_ok_for_hw_watchpoint) (s390_auxv_parse, s390_read_description): Refactor as methods of s390_linux_nat_target. (_initialize_s390_nat): Adjust to C++ification. * sparc-linux-nat.c (sparc_linux_nat_target): New class. (the_sparc_linux_nat_target): New. (_initialize_sparc_linux_nat): Adjust to C++ification. * sparc-nat.c (sparc_fetch_inferior_registers) (sparc_store_inferior_registers): Remove target_ops parameter. * sparc-nat.h (sparc_fetch_inferior_registers) (sparc_store_inferior_registers): Remove target_ops parameter. * sparc64-linux-nat.c (sparc64_linux_nat_target): New class. (the_sparc64_linux_nat_target): New. (_initialize_sparc64_linux_nat): Adjust to C++ification. * spu-linux-nat.c (spu_linux_nat_target): New class. (the_spu_linux_nat_target): New. (spu_child_post_startup_inferior, spu_child_post_attach) (spu_child_wait, spu_fetch_inferior_registers) (spu_store_inferior_registers, spu_xfer_partial) (spu_can_use_hw_breakpoint): Refactor as spu_linux_nat_target methods. (_initialize_spu_nat): Adjust to C++ification. * tilegx-linux-nat.c (tilegx_linux_nat_target): New class. (the_tilegx_linux_nat_target): New. (fetch_inferior_registers, store_inferior_registers): Refactor as methods. (_initialize_tile_linux_nat): Adjust to C++ification. * xtensa-linux-nat.c (xtensa_linux_nat_target): New class. (the_xtensa_linux_nat_target): New. (xtensa_linux_fetch_inferior_registers) (xtensa_linux_store_inferior_registers): Refactor as xtensa_linux_nat_target methods. (_initialize_xtensa_linux_nat): Adjust to C++ification. * fbsd-nat.c (USE_SIGTRAP_SIGINFO): Delete. (fbsd_pid_to_exec_file, fbsd_find_memory_regions) (fbsd_find_memory_regions, fbsd_info_proc, fbsd_xfer_partial) (fbsd_thread_alive, fbsd_pid_to_str, fbsd_thread_name) (fbsd_update_thread_list, fbsd_resume, fbsd_wait) (fbsd_stopped_by_sw_breakpoint) (fbsd_supports_stopped_by_sw_breakpoint, fbsd_follow_fork) (fbsd_insert_fork_catchpoint, fbsd_remove_fork_catchpoint) (fbsd_insert_vfork_catchpoint, fbsd_remove_vfork_catchpoint) (fbsd_post_startup_inferior, fbsd_post_attach) (fbsd_insert_exec_catchpoint, fbsd_remove_exec_catchpoint) (fbsd_set_syscall_catchpoint) (super_xfer_partial, super_resume, super_wait) (fbsd_supports_stopped_by_hw_breakpoint): Delete. (fbsd_handle_debug_trap): Remove target_ops parameter. (fbsd_nat_add_target): Delete. * fbsd-nat.h: Include "inf-ptrace.h". (fbsd_nat_add_target): Delete. (USE_SIGTRAP_SIGINFO): Define. (fbsd_nat_target): New class. * amd64-bsd-nat.c (amd64bsd_fetch_inferior_registers) (amd64bsd_store_inferior_registers): Remove target_ops parameter. (amd64bsd_target): Delete. * amd64-bsd-nat.h: New file. * amd64-fbsd-nat.c: Include "amd64-bsd-nat.h" instead of "x86-bsd-nat.h". (amd64_fbsd_nat_target): New class. (the_amd64_fbsd_nat_target): New. (amd64fbsd_read_description): Refactor as method of amd64_fbsd_nat_target. (amd64_fbsd_nat_target::supports_stopped_by_hw_breakpoint): New. (_initialize_amd64fbsd_nat): Adjust to C++ification. * amd64-nat.h (amd64bsd_target): Delete function declaration. * i386-bsd-nat.c (i386bsd_fetch_inferior_registers) (i386bsd_store_inferior_registers): Remove target_ops parameter. (i386bsd_target): Delete. * i386-bsd-nat.h (i386bsd_target): Delete function declaration. (i386bsd_fetch_inferior_registers) (i386bsd_store_inferior_registers): Declare. (i386_bsd_nat_target): New class. * i386-fbsd-nat.c (i386_fbsd_nat_target): New class. (the_i386_fbsd_nat_target): New. (i386fbsd_resume, i386fbsd_read_description): Refactor as i386_fbsd_nat_target methods. (i386_fbsd_nat_target::supports_stopped_by_hw_breakpoint): New. (_initialize_i386fbsd_nat): Adjust to C++ification. * x86-bsd-nat.c (super_mourn_inferior): Delete. (x86bsd_mourn_inferior, x86bsd_target): Delete. (_initialize_x86_bsd_nat): Adjust to C++ification. * x86-bsd-nat.h: Include "x86-nat.h". (x86bsd_target): Delete declaration. (x86bsd_nat_target): New class. * aarch64-fbsd-nat.c (aarch64_fbsd_nat_target): New class. (the_aarch64_fbsd_nat_target): New. (aarch64_fbsd_fetch_inferior_registers) (aarch64_fbsd_store_inferior_registers): Refactor as methods of aarch64_fbsd_nat_target. (_initialize_aarch64_fbsd_nat): Adjust to C++ification. * alpha-bsd-nat.c (alpha_bsd_nat_target): New class. (the_alpha_bsd_nat_target): New. (alphabsd_fetch_inferior_registers) (alphabsd_store_inferior_registers): Refactor as alpha_bsd_nat_target methods. (_initialize_alphabsd_nat): Refactor as methods of alpha_bsd_nat_target. * amd64-nbsd-nat.c: Include "amd64-bsd-nat.h". (the_amd64_nbsd_nat_target): New. (_initialize_amd64nbsd_nat): Adjust to C++ification. * amd64-obsd-nat.c: Include "amd64-bsd-nat.h". (the_amd64_obsd_nat_target): New. (_initialize_amd64obsd_nat): Adjust to C++ification. * arm-fbsd-nat.c (arm_fbsd_nat_target): New. (the_arm_fbsd_nat_target): New. (arm_fbsd_fetch_inferior_registers) (arm_fbsd_store_inferior_registers, arm_fbsd_read_description): (_initialize_arm_fbsd_nat): Refactor as methods of arm_fbsd_nat_target. (_initialize_arm_fbsd_nat): Adjust to C++ification. * arm-nbsd-nat.c (arm_netbsd_nat_target): New class. (the_arm_netbsd_nat_target): New. (armnbsd_fetch_registers, armnbsd_store_registers): Refactor as arm_netbsd_nat_target. (_initialize_arm_netbsd_nat): Adjust to C++ification. * hppa-nbsd-nat.c (hppa_nbsd_nat_target): New class. (the_hppa_nbsd_nat_target): New. (hppanbsd_fetch_registers, hppanbsd_store_registers): Refactor as hppa_nbsd_nat_target methods. (_initialize_hppanbsd_nat): Adjust to C++ification. * hppa-obsd-nat.c (hppa_obsd_nat_target): New class. (the_hppa_obsd_nat_target): New. (hppaobsd_fetch_registers, hppaobsd_store_registers): Refactor as methods of hppa_obsd_nat_target. (_initialize_hppaobsd_nat): Adjust to C++ification. Use add_target. * i386-nbsd-nat.c (the_i386_nbsd_nat_target): New. (_initialize_i386nbsd_nat): Adjust to C++ification. Use add_target. * i386-obsd-nat.c (the_i386_obsd_nat_target): New. (_initialize_i386obsd_nat): Use add_target. * m68k-bsd-nat.c (m68k_bsd_nat_target): New class. (the_m68k_bsd_nat_target): New. (m68kbsd_fetch_inferior_registers) (m68kbsd_store_inferior_registers): Refactor as methods of m68k_bsd_nat_target. (_initialize_m68kbsd_nat): Adjust to C++ification. * mips-fbsd-nat.c (mips_fbsd_nat_target): New class. (the_mips_fbsd_nat_target): New. (mips_fbsd_fetch_inferior_registers) (mips_fbsd_store_inferior_registers): Refactor as methods of mips_fbsd_nat_target. (_initialize_mips_fbsd_nat): Adjust to C++ification. Use add_target. * mips-nbsd-nat.c (mips_nbsd_nat_target): New class. (the_mips_nbsd_nat_target): New. (mipsnbsd_fetch_inferior_registers) (mipsnbsd_store_inferior_registers): Refactor as methods of mips_nbsd_nat_target. (_initialize_mipsnbsd_nat): Adjust to C++ification. * mips64-obsd-nat.c (mips64_obsd_nat_target): New class. (the_mips64_obsd_nat_target): New. (mips64obsd_fetch_inferior_registers) (mips64obsd_store_inferior_registers): Refactor as methods of mips64_obsd_nat_target. (_initialize_mips64obsd_nat): Adjust to C++ification. Use add_target. * nbsd-nat.c (nbsd_pid_to_exec_file): Refactor as method of nbsd_nat_target. * nbsd-nat.h: Include "inf-ptrace.h". (nbsd_nat_target): New class. * obsd-nat.c (obsd_pid_to_str, obsd_update_thread_list) (obsd_wait): Refactor as methods of obsd_nat_target. (obsd_add_target): Delete. * obsd-nat.h: Include "inf-ptrace.h". (obsd_nat_target): New class. * ppc-fbsd-nat.c (ppc_fbsd_nat_target): New class. (the_ppc_fbsd_nat_target): New. (ppcfbsd_fetch_inferior_registers) (ppcfbsd_store_inferior_registers): Refactor as methods of ppc_fbsd_nat_target. (_initialize_ppcfbsd_nat): Adjust to C++ification. Use add_target. * ppc-nbsd-nat.c (ppc_nbsd_nat_target): New class. (the_ppc_nbsd_nat_target): New. (ppcnbsd_fetch_inferior_registers) (ppcnbsd_store_inferior_registers): Refactor as methods of ppc_nbsd_nat_target. (_initialize_ppcnbsd_nat): Adjust to C++ification. * ppc-obsd-nat.c (ppc_obsd_nat_target): New class. (the_ppc_obsd_nat_target): New. (ppcobsd_fetch_registers, ppcobsd_store_registers): Refactor as methods of ppc_obsd_nat_target. (_initialize_ppcobsd_nat): Adjust to C++ification. Use add_target. * sh-nbsd-nat.c (sh_nbsd_nat_target): New class. (the_sh_nbsd_nat_target): New. (shnbsd_fetch_inferior_registers) (shnbsd_store_inferior_registers): Refactor as methods of sh_nbsd_nat_target. (_initialize_shnbsd_nat): Adjust to C++ification. * sparc-nat.c (sparc_xfer_wcookie): Make extern. (inf_ptrace_xfer_partial): Delete. (sparc_xfer_partial, sparc_target): Delete. * sparc-nat.h (sparc_fetch_inferior_registers) (sparc_store_inferior_registers, sparc_xfer_wcookie): Declare. (sparc_target): Delete function declaration. (sparc_target): New template class. * sparc-nbsd-nat.c (the_sparc_nbsd_nat_target): New. (_initialize_sparcnbsd_nat): Adjust to C++ification. * sparc64-fbsd-nat.c (the_sparc64_fbsd_nat_target): New. (_initialize_sparc64fbsd_nat): Adjust to C++ification. Use add_target. * sparc64-nbsd-nat.c (the_sparc64_nbsd_nat_target): New. (_initialize_sparc64nbsd_nat): Adjust to C++ification. * sparc64-obsd-nat.c (the_sparc64_obsd_nat_target): New. (_initialize_sparc64obsd_nat): Adjust to C++ification. Use add_target. * vax-bsd-nat.c (vax_bsd_nat_target): New class. (the_vax_bsd_nat_target): New. (vaxbsd_fetch_inferior_registers) (vaxbsd_store_inferior_registers): Refactor as vax_bsd_nat_target methods. (_initialize_vaxbsd_nat): Adjust to C++ification. * bsd-kvm.c (bsd_kvm_target): New class. (bsd_kvm_ops): Now a bsd_kvm_target. (bsd_kvm_open, bsd_kvm_close, bsd_kvm_xfer_partial) (bsd_kvm_files_info, bsd_kvm_fetch_registers) (bsd_kvm_thread_alive, bsd_kvm_pid_to_str): Refactor as methods of bsd_kvm_target. (bsd_kvm_return_one): Delete. (bsd_kvm_add_target): Adjust to C++ification. * nto-procfs.c (nto_procfs_target, nto_procfs_target_native) (nto_procfs_target_procfs): New classes. (procfs_open_1, procfs_thread_alive, procfs_update_thread_list) (procfs_files_info, procfs_pid_to_exec_file, procfs_attach) (procfs_post_attach, procfs_wait, procfs_fetch_registers) (procfs_xfer_partial, procfs_detach, procfs_insert_breakpoint) (procfs_remove_breakpoint, procfs_insert_hw_breakpoint) (procfs_remove_hw_breakpoint, procfs_resume) (procfs_mourn_inferior, procfs_create_inferior, procfs_interrupt) (procfs_kill_inferior, procfs_store_registers) (procfs_pass_signals, procfs_pid_to_str, procfs_can_run): Refactor as methods of nto_procfs_target. (nto_procfs_ops): Now an nto_procfs_target_procfs. (nto_native_ops): Delete. (procfs_open, procfs_native_open): Delete. (nto_native_ops): Now an nto_procfs_target_native. (init_procfs_targets): Adjust to C++ification. (procfs_can_use_hw_breakpoint, procfs_remove_hw_watchpoint) (procfs_insert_hw_watchpoint, procfs_stopped_by_watchpoint): Refactor as methods of nto_procfs_target. * go32-nat.c (go32_nat_target): New class. (the_go32_nat_target): New. (go32_attach, go32_resume, go32_wait, go32_fetch_registers) (go32_store_registers, go32_xfer_partial, go32_files_info) (go32_kill_inferior, go32_create_inferior, go32_mourn_inferior) (go32_terminal_init, go32_terminal_info, go32_terminal_inferior) (go32_terminal_ours, go32_pass_ctrlc, go32_thread_alive) (go32_pid_to_str): Refactor as methods of go32_nat_target. (go32_target): Delete. (_initialize_go32_nat): Adjust to C++ification. * gnu-nat.c (gnu_wait, gnu_resume, gnu_kill_inferior) (gnu_mourn_inferior, gnu_create_inferior, gnu_attach, gnu_detach) (gnu_stop, gnu_thread_alive, gnu_xfer_partial) (gnu_find_memory_regions, gnu_pid_to_str): Refactor as methods of gnu_nat_target. (gnu_target): Delete. * gnu-nat.h (gnu_target): Delete. (gnu_nat_target): New class. * i386-gnu-nat.c (gnu_base_target): New. (i386_gnu_nat_target): New class. (the_i386_gnu_nat_target): New. (_initialize_i386gnu_nat): Adjust to C++ification. gdb/testsuite/ChangeLog: 2018-05-02 Pedro Alves * gdb.base/breakpoint-in-ro-region.exp: Adjust to to_resume and to_log_command renames. * gdb.base/sss-bp-on-user-bp-2.exp: Likewise. --- gdb/target.c | 846 ++++++++++++++++++++++++++--------------------------------- 1 file changed, 377 insertions(+), 469 deletions(-) (limited to 'gdb/target.c') diff --git a/gdb/target.c b/gdb/target.c index 2ff028c..022da59 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -86,10 +86,6 @@ static struct address_space *default_thread_address_space static void tcomplain (void) ATTRIBUTE_NORETURN; -static int return_zero (struct target_ops *); - -static int return_zero_has_execution (struct target_ops *, ptid_t); - static struct target_ops *find_default_run_target (const char *); static struct gdbarch *default_thread_architecture (struct target_ops *ops, @@ -107,14 +103,6 @@ static const char *default_pid_to_str (struct target_ops *ops, ptid_t ptid); static enum exec_direction_kind default_execution_direction (struct target_ops *self); -static struct target_ops debug_target; - -#include "target-delegates.c" - -static void init_dummy_target (void); - -static void update_current_target (void); - /* Vector of existing target structures. */ typedef struct target_ops *target_ops_p; DEF_VEC_P (target_ops_p); @@ -123,16 +111,14 @@ static VEC (target_ops_p) *target_structs; /* The initial current target, so that there is always a semi-valid current target. */ -static struct target_ops dummy_target; +static struct target_ops *the_dummy_target; +static struct target_ops *the_debug_target; /* Top of target stack. */ - -static struct target_ops *target_stack; - /* The target structure we are currently using to talk to a process or file or whatever "inferior" we have. */ -struct target_ops current_target; +struct target_ops *target_stack; /* Command list for target. */ @@ -171,7 +157,10 @@ static unsigned int targetdebug = 0; static void set_targetdebug (const char *args, int from_tty, struct cmd_list_element *c) { - update_current_target (); + if (targetdebug) + push_target (the_debug_target); + else + unpush_target (the_debug_target); } static void @@ -181,8 +170,6 @@ show_targetdebug (struct ui_file *file, int from_tty, fprintf_filtered (file, _("Target debugging is %s.\n"), value); } -static void setup_target_debug (void); - /* The user just typed 'target' without the name of a target. */ static void @@ -195,7 +182,7 @@ target_command (const char *arg, int from_tty) /* Default target_has_* methods for process_stratum targets. */ int -default_child_has_all_memory (struct target_ops *ops) +default_child_has_all_memory () { /* If no inferior selected, then we can't read memory here. */ if (ptid_equal (inferior_ptid, null_ptid)) @@ -205,7 +192,7 @@ default_child_has_all_memory (struct target_ops *ops) } int -default_child_has_memory (struct target_ops *ops) +default_child_has_memory () { /* If no inferior selected, then we can't read memory here. */ if (ptid_equal (inferior_ptid, null_ptid)) @@ -215,7 +202,7 @@ default_child_has_memory (struct target_ops *ops) } int -default_child_has_stack (struct target_ops *ops) +default_child_has_stack () { /* If no inferior selected, there's no stack. */ if (ptid_equal (inferior_ptid, null_ptid)) @@ -225,7 +212,7 @@ default_child_has_stack (struct target_ops *ops) } int -default_child_has_registers (struct target_ops *ops) +default_child_has_registers () { /* Can't read registers from no inferior. */ if (ptid_equal (inferior_ptid, null_ptid)) @@ -235,7 +222,7 @@ default_child_has_registers (struct target_ops *ops) } int -default_child_has_execution (struct target_ops *ops, ptid_t the_ptid) +default_child_has_execution (ptid_t the_ptid) { /* If there's no thread selected, then we can't make it run through hoops. */ @@ -251,8 +238,8 @@ target_has_all_memory_1 (void) { struct target_ops *t; - for (t = current_target.beneath; t != NULL; t = t->beneath) - if (t->to_has_all_memory (t)) + for (t = target_stack; t != NULL; t = t->beneath) + if (t->has_all_memory ()) return 1; return 0; @@ -263,8 +250,8 @@ target_has_memory_1 (void) { struct target_ops *t; - for (t = current_target.beneath; t != NULL; t = t->beneath) - if (t->to_has_memory (t)) + for (t = target_stack; t != NULL; t = t->beneath) + if (t->has_memory ()) return 1; return 0; @@ -275,8 +262,8 @@ target_has_stack_1 (void) { struct target_ops *t; - for (t = current_target.beneath; t != NULL; t = t->beneath) - if (t->to_has_stack (t)) + for (t = target_stack; t != NULL; t = t->beneath) + if (t->has_stack ()) return 1; return 0; @@ -287,8 +274,8 @@ target_has_registers_1 (void) { struct target_ops *t; - for (t = current_target.beneath; t != NULL; t = t->beneath) - if (t->to_has_registers (t)) + for (t = target_stack; t != NULL; t = t->beneath) + if (t->has_registers ()) return 1; return 0; @@ -299,8 +286,8 @@ target_has_execution_1 (ptid_t the_ptid) { struct target_ops *t; - for (t = current_target.beneath; t != NULL; t = t->beneath) - if (t->to_has_execution (t, the_ptid)) + for (t = target_stack; t != NULL; t = t->beneath) + if (t->has_execution (the_ptid)) return 1; return 0; @@ -312,38 +299,6 @@ target_has_execution_current (void) return target_has_execution_1 (inferior_ptid); } -/* Complete initialization of T. This ensures that various fields in - T are set, if needed by the target implementation. */ - -void -complete_target_initialization (struct target_ops *t) -{ - /* Provide default values for all "must have" methods. */ - - if (t->to_has_all_memory == NULL) - t->to_has_all_memory = return_zero; - - if (t->to_has_memory == NULL) - t->to_has_memory = return_zero; - - if (t->to_has_stack == NULL) - t->to_has_stack = return_zero; - - if (t->to_has_registers == NULL) - t->to_has_registers = return_zero; - - if (t->to_has_execution == NULL) - t->to_has_execution = return_zero_has_execution; - - /* These methods can be called on an unpushed target and so require - a default implementation if the target might plausibly be the - default run target. */ - gdb_assert (t->to_can_run == NULL || (t->to_can_async_p != NULL - && t->to_supports_non_stop != NULL)); - - install_delegators (t); -} - /* This is used to implement the various target commands. */ static void @@ -353,17 +308,17 @@ open_target (const char *args, int from_tty, struct cmd_list_element *command) if (targetdebug) fprintf_unfiltered (gdb_stdlog, "-> %s->to_open (...)\n", - ops->to_shortname); + ops->shortname ()); - ops->to_open (args, from_tty); + ops->open (args, from_tty); if (targetdebug) fprintf_unfiltered (gdb_stdlog, "<- %s->to_open (%s, %d)\n", - ops->to_shortname, args, from_tty); + ops->shortname (), args, from_tty); } /* Add possible target architecture T to the list and add a new - command 'target T->to_shortname'. Set COMPLETER as the command's + command 'target T->shortname ()'. Set COMPLETER as the command's completer if not NULL. */ void @@ -372,8 +327,6 @@ add_target_with_completer (struct target_ops *t, { struct cmd_list_element *c; - complete_target_initialization (t); - VEC_safe_push (target_ops_p, target_structs, t); if (targetlist == NULL) @@ -384,7 +337,7 @@ Remaining arguments are interpreted by the target protocol. For more\n\ information on the arguments for a particular protocol, type\n\ `help target ' followed by the protocol name."), &targetlist, "target ", 0, &cmdlist); - c = add_cmd (t->to_shortname, no_class, t->to_doc, &targetlist); + c = add_cmd (t->shortname (), no_class, t->doc (), &targetlist); set_cmd_sfunc (c, open_target); set_cmd_context (c, t); if (completer != NULL) @@ -409,10 +362,10 @@ add_deprecated_target_alias (struct target_ops *t, const char *alias) /* If we use add_alias_cmd, here, we do not get the deprecated warning, see PR cli/15104. */ - c = add_cmd (alias, no_class, t->to_doc, &targetlist); + c = add_cmd (alias, no_class, t->doc (), &targetlist); set_cmd_sfunc (c, open_target); set_cmd_context (c, t); - alt = xstrprintf ("target %s", t->to_shortname); + alt = xstrprintf ("target %s", t->shortname ()); deprecate_cmd (c, alt); } @@ -421,14 +374,14 @@ add_deprecated_target_alias (struct target_ops *t, const char *alias) void target_kill (void) { - current_target.to_kill (¤t_target); + target_stack->kill (); } void target_load (const char *arg, int from_tty) { target_dcache_invalidate (); - (*current_target.to_load) (¤t_target, arg, from_tty); + target_stack->load (arg, from_tty); } /* Define it. */ @@ -441,7 +394,7 @@ target_terminal_state target_terminal::m_terminal_state void target_terminal::init (void) { - (*current_target.to_terminal_init) (¤t_target); + target_stack->terminal_init (); m_terminal_state = target_terminal_state::is_ours; } @@ -472,7 +425,7 @@ target_terminal::inferior (void) if (inf->terminal_state != target_terminal_state::is_inferior) { - (*current_target.to_terminal_inferior) (¤t_target); + target_stack->terminal_inferior (); inf->terminal_state = target_terminal_state::is_inferior; } @@ -508,7 +461,7 @@ target_terminal::restore_inferior (void) if (inf->terminal_state == target_terminal_state::is_ours_for_output) { set_current_inferior (inf); - (*current_target.to_terminal_inferior) (¤t_target); + target_stack->terminal_inferior (); inf->terminal_state = target_terminal_state::is_inferior; } } @@ -541,7 +494,7 @@ target_terminal_is_ours_kind (target_terminal_state desired_state) if (inf->terminal_state == target_terminal_state::is_inferior) { set_current_inferior (inf); - (*current_target.to_terminal_save_inferior) (¤t_target); + target_stack->terminal_save_inferior (); } } @@ -556,9 +509,9 @@ target_terminal_is_ours_kind (target_terminal_state desired_state) { set_current_inferior (inf); if (desired_state == target_terminal_state::is_ours) - (*current_target.to_terminal_ours) (¤t_target); + target_stack->terminal_ours (); else if (desired_state == target_terminal_state::is_ours_for_output) - (*current_target.to_terminal_ours_for_output) (¤t_target); + target_stack->terminal_ours_for_output (); else gdb_assert_not_reached ("unhandled desired state"); inf->terminal_state = desired_state; @@ -607,7 +560,7 @@ target_terminal::ours_for_output () void target_terminal::info (const char *arg, int from_tty) { - (*current_target.to_terminal_info) (¤t_target, arg, from_tty); + target_stack->terminal_info (arg, from_tty); } /* See target.h. */ @@ -615,23 +568,14 @@ target_terminal::info (const char *arg, int from_tty) int target_supports_terminal_ours (void) { - struct target_ops *t; - - for (t = current_target.beneath; t != NULL; t = t->beneath) - { - if (t->to_terminal_ours != delegate_terminal_ours - && t->to_terminal_ours != tdefault_terminal_ours) - return 1; - } - - return 0; + return target_stack->supports_terminal_ours (); } static void tcomplain (void) { error (_("You can't do that when your target is `%s'"), - current_target.to_shortname); + target_stack->shortname ()); } void @@ -670,57 +614,6 @@ default_execution_direction (struct target_ops *self) to_execution_direction must be implemented for reverse async"); } -/* Go through the target stack from top to bottom, copying over zero - entries in current_target, then filling in still empty entries. In - effect, we are doing class inheritance through the pushed target - vectors. - - NOTE: cagney/2003-10-17: The problem with this inheritance, as it - is currently implemented, is that it discards any knowledge of - which target an inherited method originally belonged to. - Consequently, new new target methods should instead explicitly and - locally search the target stack for the target that can handle the - request. */ - -static void -update_current_target (void) -{ - struct target_ops *t; - - /* First, reset current's contents. */ - memset (¤t_target, 0, sizeof (current_target)); - - /* Install the delegators. */ - install_delegators (¤t_target); - - current_target.to_stratum = target_stack->to_stratum; - -#define INHERIT(FIELD, TARGET) \ - if (!current_target.FIELD) \ - current_target.FIELD = (TARGET)->FIELD - - /* Do not add any new INHERITs here. Instead, use the delegation - mechanism provided by make-target-delegates. */ - for (t = target_stack; t; t = t->beneath) - { - INHERIT (to_shortname, t); - INHERIT (to_longname, t); - INHERIT (to_attach_no_wait, t); - INHERIT (to_have_steppable_watchpoint, t); - INHERIT (to_have_continuable_watchpoint, t); - INHERIT (to_has_thread_control, t); - } -#undef INHERIT - - /* Finally, position the target-stack beneath the squashed - "current_target". That way code looking for a non-inherited - target method can quickly and simply find it. */ - current_target.beneath = target_stack; - - if (targetdebug) - setup_target_debug (); -} - /* Push a new target type into the stack of the existing target accessors, possibly superseding some of the existing accessors. @@ -733,17 +626,6 @@ push_target (struct target_ops *t) { struct target_ops **cur; - /* Check magic number. If wrong, it probably means someone changed - the struct definition, but not all the places that initialize one. */ - if (t->to_magic != OPS_MAGIC) - { - fprintf_unfiltered (gdb_stderr, - "Magic number of %s target struct wrong\n", - t->to_shortname); - internal_error (__FILE__, __LINE__, - _("failed internal consistency check")); - } - /* Find the proper stratum to install this target in. */ for (cur = &target_stack; (*cur) != NULL; cur = &(*cur)->beneath) { @@ -768,8 +650,6 @@ push_target (struct target_ops *t) /* We have removed all targets in our stratum, now add the new one. */ t->beneath = (*cur); (*cur) = t; - - update_current_target (); } /* Remove a target_ops vector from the stack, wherever it may be. @@ -804,8 +684,6 @@ unpush_target (struct target_ops *t) (*cur) = (*cur)->beneath; tmp->beneath = NULL; - update_current_target (); - /* Finally close the target. Note we do this after unchaining, so any target method calls from within the target_close implementation don't end up in T anymore. */ @@ -823,7 +701,7 @@ unpush_target_and_assert (struct target_ops *target) { fprintf_unfiltered (gdb_stderr, "pop_all_targets couldn't find target %s\n", - target->to_shortname); + target->shortname ()); internal_error (__FILE__, __LINE__, _("failed internal consistency check")); } @@ -832,7 +710,7 @@ unpush_target_and_assert (struct target_ops *target) void pop_all_targets_above (enum strata above_stratum) { - while ((int) (current_target.to_stratum) > (int) above_stratum) + while ((int) (target_stack->to_stratum) > (int) above_stratum) unpush_target_and_assert (target_stack); } @@ -841,7 +719,7 @@ pop_all_targets_above (enum strata above_stratum) void pop_all_targets_at_and_above (enum strata stratum) { - while ((int) (current_target.to_stratum) >= (int) stratum) + while ((int) (target_stack->to_stratum) >= (int) stratum) unpush_target_and_assert (target_stack); } @@ -858,17 +736,6 @@ target_is_pushed (struct target_ops *t) { struct target_ops *cur; - /* Check magic number. If wrong, it probably means someone changed - the struct definition, but not all the places that initialize one. */ - if (t->to_magic != OPS_MAGIC) - { - fprintf_unfiltered (gdb_stderr, - "Magic number of %s target struct wrong\n", - t->to_shortname); - internal_error (__FILE__, __LINE__, - _("failed internal consistency check")); - } - for (cur = target_stack; cur != NULL; cur = cur->beneath) if (cur == t) return 1; @@ -891,7 +758,7 @@ CORE_ADDR target_translate_tls_address (struct objfile *objfile, CORE_ADDR offset) { volatile CORE_ADDR addr = 0; - struct target_ops *target = ¤t_target; + struct target_ops *target = target_stack; if (gdbarch_fetch_tls_load_module_address_p (target_gdbarch ())) { @@ -905,8 +772,7 @@ target_translate_tls_address (struct objfile *objfile, CORE_ADDR offset) lm_addr = gdbarch_fetch_tls_load_module_address (target_gdbarch (), objfile); - addr = target->to_get_thread_local_address (target, ptid, - lm_addr, offset); + addr = target->get_thread_local_address (ptid, lm_addr, offset); } /* If an error occurred, print TLS related messages here. Otherwise, throw the error to some higher catcher. */ @@ -1064,7 +930,7 @@ done: struct target_section_table * target_get_section_table (struct target_ops *target) { - return (*target->to_get_section_table) (target); + return target->get_section_table (); } /* Find a section containing ADDR. */ @@ -1152,9 +1018,9 @@ raw_memory_xfer_partial (struct target_ops *ops, gdb_byte *readbuf, do { - res = ops->to_xfer_partial (ops, TARGET_OBJECT_MEMORY, NULL, - readbuf, writebuf, memaddr, len, - xfered_len); + res = ops->xfer_partial (TARGET_OBJECT_MEMORY, NULL, + readbuf, writebuf, memaddr, len, + xfered_len); if (res == TARGET_XFER_OK) break; @@ -1164,7 +1030,7 @@ raw_memory_xfer_partial (struct target_ops *ops, gdb_byte *readbuf, /* We want to continue past core files to executables, but not past a running target's memory. */ - if (ops->to_has_all_memory (ops)) + if (ops->has_all_memory ()) break; ops = ops->beneath; @@ -1326,7 +1192,7 @@ memory_xfer_partial (struct target_ops *ops, enum target_object object, shadow handling even though we only end up writing a small subset of it. Cap writes to a limit specified by the target to mitigate this. */ - len = std::min (ops->to_get_memory_xfer_limit (ops), len); + len = std::min (ops->get_memory_xfer_limit (), len); gdb::byte_vector buf (writebuf, writebuf + len); breakpoint_xfer_memory (NULL, buf.data (), writebuf, memaddr, len); @@ -1354,8 +1220,6 @@ target_xfer_partial (struct target_ops *ops, { enum target_xfer_status retval; - gdb_assert (ops->to_xfer_partial != NULL); - /* Transfer is done when LEN is zero. */ if (len == 0) return TARGET_XFER_EOF; @@ -1390,8 +1254,8 @@ target_xfer_partial (struct target_ops *ops, xfered_len); } else - retval = ops->to_xfer_partial (ops, object, annex, readbuf, - writebuf, offset, len, xfered_len); + retval = ops->xfer_partial (object, annex, readbuf, + writebuf, offset, len, xfered_len); if (targetdebug) { @@ -1400,7 +1264,7 @@ target_xfer_partial (struct target_ops *ops, fprintf_unfiltered (gdb_stdlog, "%s:target_xfer_partial " "(%d, %s, %s, %s, %s, %s) = %d, %s", - ops->to_shortname, + ops->shortname (), (int) object, (annex ? annex : "(null)"), host_address_to_string (readbuf), @@ -1460,10 +1324,7 @@ target_xfer_partial (struct target_ops *ops, int target_read_memory (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) { - /* Dispatch to the topmost target, not the flattened current_target. - Memory accesses check target->to_has_(all_)memory, and the - flattened target doesn't inherit those. */ - if (target_read (current_target.beneath, TARGET_OBJECT_MEMORY, NULL, + if (target_read (target_stack, TARGET_OBJECT_MEMORY, NULL, myaddr, memaddr, len) == len) return 0; else @@ -1493,9 +1354,7 @@ target_read_uint32 (CORE_ADDR memaddr, uint32_t *result) int target_read_raw_memory (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) { - /* See comment in target_read_memory about why the request starts at - current_target.beneath. */ - if (target_read (current_target.beneath, TARGET_OBJECT_RAW_MEMORY, NULL, + if (target_read (target_stack, TARGET_OBJECT_RAW_MEMORY, NULL, myaddr, memaddr, len) == len) return 0; else @@ -1508,9 +1367,7 @@ target_read_raw_memory (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) int target_read_stack (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) { - /* See comment in target_read_memory about why the request starts at - current_target.beneath. */ - if (target_read (current_target.beneath, TARGET_OBJECT_STACK_MEMORY, NULL, + if (target_read (target_stack, TARGET_OBJECT_STACK_MEMORY, NULL, myaddr, memaddr, len) == len) return 0; else @@ -1523,9 +1380,7 @@ target_read_stack (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) int target_read_code (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) { - /* See comment in target_read_memory about why the request starts at - current_target.beneath. */ - if (target_read (current_target.beneath, TARGET_OBJECT_CODE_MEMORY, NULL, + if (target_read (target_stack, TARGET_OBJECT_CODE_MEMORY, NULL, myaddr, memaddr, len) == len) return 0; else @@ -1541,9 +1396,7 @@ target_read_code (CORE_ADDR memaddr, gdb_byte *myaddr, ssize_t len) int target_write_memory (CORE_ADDR memaddr, const gdb_byte *myaddr, ssize_t len) { - /* See comment in target_read_memory about why the request starts at - current_target.beneath. */ - if (target_write (current_target.beneath, TARGET_OBJECT_MEMORY, NULL, + if (target_write (target_stack, TARGET_OBJECT_MEMORY, NULL, myaddr, memaddr, len) == len) return 0; else @@ -1559,9 +1412,7 @@ target_write_memory (CORE_ADDR memaddr, const gdb_byte *myaddr, ssize_t len) int target_write_raw_memory (CORE_ADDR memaddr, const gdb_byte *myaddr, ssize_t len) { - /* See comment in target_read_memory about why the request starts at - current_target.beneath. */ - if (target_write (current_target.beneath, TARGET_OBJECT_RAW_MEMORY, NULL, + if (target_write (target_stack, TARGET_OBJECT_RAW_MEMORY, NULL, myaddr, memaddr, len) == len) return 0; else @@ -1573,8 +1424,7 @@ target_write_raw_memory (CORE_ADDR memaddr, const gdb_byte *myaddr, ssize_t len) std::vector target_memory_map (void) { - std::vector result - = current_target.to_memory_map (¤t_target); + std::vector result = target_stack->memory_map (); if (result.empty ()) return result; @@ -1604,13 +1454,13 @@ target_memory_map (void) void target_flash_erase (ULONGEST address, LONGEST length) { - current_target.to_flash_erase (¤t_target, address, length); + target_stack->flash_erase (address, length); } void target_flash_done (void) { - current_target.to_flash_done (¤t_target); + target_stack->flash_done (); } static void @@ -2065,8 +1915,7 @@ target_insert_breakpoint (struct gdbarch *gdbarch, return 1; } - return current_target.to_insert_breakpoint (¤t_target, - gdbarch, bp_tgt); + return target_stack->insert_breakpoint (gdbarch, bp_tgt); } /* See target.h. */ @@ -2086,8 +1935,7 @@ target_remove_breakpoint (struct gdbarch *gdbarch, return 1; } - return current_target.to_remove_breakpoint (¤t_target, - gdbarch, bp_tgt, reason); + return target_stack->remove_breakpoint (gdbarch, bp_tgt, reason); } static void @@ -2102,7 +1950,7 @@ info_target_command (const char *args, int from_tty) for (t = target_stack; t != NULL; t = t->beneath) { - if (!(*t->to_has_memory) (t)) + if (!t->has_memory ()) continue; if ((int) (t->to_stratum) <= (int) dummy_stratum) @@ -2110,9 +1958,9 @@ info_target_command (const char *args, int from_tty) if (has_all_mem) printf_unfiltered (_("\tWhile running this, " "GDB does not access memory from...\n")); - printf_unfiltered ("%s:\n", t->to_longname); - (t->to_files_info) (t); - has_all_mem = (*t->to_has_all_memory) (t); + printf_unfiltered ("%s:\n", t->longname ()); + t->files_info (); + has_all_mem = t->has_all_memory (); } } @@ -2236,7 +2084,7 @@ target_detach (inferior *inf, int from_tty) prepare_for_detach (); - current_target.to_detach (¤t_target, inf, from_tty); + target_stack->detach (inf, from_tty); } void @@ -2247,7 +2095,7 @@ target_disconnect (const char *args, int from_tty) disconnecting. */ remove_breakpoints (); - current_target.to_disconnect (¤t_target, args, from_tty); + target_stack->disconnect (args, from_tty); } /* See target/target.h. */ @@ -2255,7 +2103,7 @@ target_disconnect (const char *args, int from_tty) ptid_t target_wait (ptid_t ptid, struct target_waitstatus *status, int options) { - return (current_target.to_wait) (¤t_target, ptid, status, options); + return target_stack->wait (ptid, status, options); } /* See target.h. */ @@ -2272,13 +2120,13 @@ default_target_wait (struct target_ops *ops, const char * target_pid_to_str (ptid_t ptid) { - return (*current_target.to_pid_to_str) (¤t_target, ptid); + return target_stack->pid_to_str (ptid); } const char * target_thread_name (struct thread_info *info) { - return current_target.to_thread_name (¤t_target, info); + return target_stack->thread_name (info); } struct thread_info * @@ -2286,8 +2134,8 @@ target_thread_handle_to_thread_info (const gdb_byte *thread_handle, int handle_len, struct inferior *inf) { - return current_target.to_thread_handle_to_thread_info - (¤t_target, thread_handle, handle_len, inf); + return target_stack->thread_handle_to_thread_info (thread_handle, + handle_len, inf); } void @@ -2295,7 +2143,7 @@ target_resume (ptid_t ptid, int step, enum gdb_signal signal) { target_dcache_invalidate (); - current_target.to_resume (¤t_target, ptid, step, signal); + target_stack->resume (ptid, step, signal); registers_changed_ptid (ptid); /* We only set the internal executing state here. The user/frontend @@ -2315,7 +2163,7 @@ target_commit_resume (void) if (defer_target_commit_resume) return; - current_target.to_commit_resume (¤t_target); + target_stack->commit_resume (); } /* See target.h. */ @@ -2329,14 +2177,13 @@ make_scoped_defer_target_commit_resume () void target_pass_signals (int numsigs, unsigned char *pass_signals) { - (*current_target.to_pass_signals) (¤t_target, numsigs, pass_signals); + target_stack->pass_signals (numsigs, pass_signals); } void target_program_signals (int numsigs, unsigned char *program_signals) { - (*current_target.to_program_signals) (¤t_target, - numsigs, program_signals); + target_stack->program_signals (numsigs, program_signals); } static int @@ -2354,8 +2201,7 @@ default_follow_fork (struct target_ops *self, int follow_child, int target_follow_fork (int follow_child, int detach_fork) { - return current_target.to_follow_fork (¤t_target, - follow_child, detach_fork); + return target_stack->follow_fork (follow_child, detach_fork); } /* Target wrapper for follow exec hook. */ @@ -2363,7 +2209,7 @@ target_follow_fork (int follow_child, int detach_fork) void target_follow_exec (struct inferior *inf, char *execd_pathname) { - current_target.to_follow_exec (¤t_target, inf, execd_pathname); + target_stack->follow_exec (inf, execd_pathname); } static void @@ -2377,7 +2223,7 @@ void target_mourn_inferior (ptid_t ptid) { gdb_assert (ptid_equal (ptid, inferior_ptid)); - current_target.to_mourn_inferior (¤t_target); + target_stack->mourn_inferior (); /* We no longer need to keep handles on any of the object files. Make sure to release them to avoid unnecessarily locking any @@ -2391,7 +2237,7 @@ target_mourn_inferior (ptid_t ptid) const struct target_desc * target_read_description (struct target_ops *target) { - return target->to_read_description (target); + return target->read_description (); } /* This implements a basic search of memory, reading target memory and @@ -2504,7 +2350,7 @@ default_search_memory (struct target_ops *self, CORE_ADDR *found_addrp) { /* Start over from the top of the target stack. */ - return simple_search_memory (current_target.beneath, + return simple_search_memory (target_stack, start_addr, search_space_len, pattern, pattern_len, found_addrp); } @@ -2521,9 +2367,8 @@ target_search_memory (CORE_ADDR start_addr, ULONGEST search_space_len, const gdb_byte *pattern, ULONGEST pattern_len, CORE_ADDR *found_addrp) { - return current_target.to_search_memory (¤t_target, start_addr, - search_space_len, - pattern, pattern_len, found_addrp); + return target_stack->search_memory (start_addr, search_space_len, + pattern, pattern_len, found_addrp); } /* Look through the currently pushed targets. If none of them will @@ -2541,20 +2386,18 @@ target_require_runnable (void) assume we will still be able to after killing the current one. Either killing and mourning will not pop T, or else find_default_run_target will find it again. */ - if (t->to_create_inferior != NULL) + if (t->can_create_inferior ()) return; /* Do not worry about targets at certain strata that can not create inferiors. Assume they will be pushed again if necessary, and continue to the process_stratum. */ - if (t->to_stratum == thread_stratum - || t->to_stratum == record_stratum - || t->to_stratum == arch_stratum) + if (t->to_stratum > process_stratum) continue; error (_("The \"%s\" target does not support \"run\". " "Try \"help target\" or \"continue\"."), - t->to_shortname); + t->shortname ()); } /* This function is only called if the target is running. In that @@ -2597,7 +2440,7 @@ find_default_run_target (const char *do_mesg) for (i = 0; VEC_iterate (target_ops_p, target_structs, i, t); ++i) { - if (t->to_can_run != delegate_can_run && target_can_run (t)) + if (t->can_run ()) { runable = t; ++count; @@ -2627,9 +2470,9 @@ find_attach_target (void) struct target_ops *t; /* If a target on the current stack can attach, use it. */ - for (t = current_target.beneath; t != NULL; t = t->beneath) + for (t = target_stack; t != NULL; t = t->beneath) { - if (t->to_attach != NULL) + if (t->can_attach ()) break; } @@ -2647,10 +2490,10 @@ find_run_target (void) { struct target_ops *t; - /* If a target on the current stack can attach, use it. */ - for (t = current_target.beneath; t != NULL; t = t->beneath) + /* If a target on the current stack can run, use it. */ + for (t = target_stack; t != NULL; t = t->beneath) { - if (t->to_create_inferior != NULL) + if (t->can_create_inferior ()) break; } @@ -2661,6 +2504,12 @@ find_run_target (void) return t; } +bool +target_ops::info_proc (const char *args, enum info_proc_what what) +{ + return false; +} + /* Implement the "info proc" command. */ int @@ -2671,17 +2520,14 @@ target_info_proc (const char *args, enum info_proc_what what) /* If we're already connected to something that can get us OS related data, use it. Otherwise, try using the native target. */ - if (current_target.to_stratum >= process_stratum) - t = current_target.beneath; - else + t = find_target_at (process_stratum); + if (t == NULL) t = find_default_run_target (NULL); for (; t != NULL; t = t->beneath) { - if (t->to_info_proc != NULL) + if (t->info_proc (args, what)) { - t->to_info_proc (t, args, what); - if (targetdebug) fprintf_unfiltered (gdb_stdlog, "target_info_proc (\"%s\", %d)\n", args, what); @@ -2699,21 +2545,15 @@ find_default_supports_disable_randomization (struct target_ops *self) struct target_ops *t; t = find_default_run_target (NULL); - if (t && t->to_supports_disable_randomization) - return (t->to_supports_disable_randomization) (t); + if (t != NULL) + return t->supports_disable_randomization (); return 0; } int target_supports_disable_randomization (void) { - struct target_ops *t; - - for (t = ¤t_target; t != NULL; t = t->beneath) - if (t->to_supports_disable_randomization) - return t->to_supports_disable_randomization (t); - - return 0; + return target_stack->supports_disable_randomization (); } /* See target/target.h. */ @@ -2721,7 +2561,7 @@ target_supports_disable_randomization (void) int target_supports_multi_process (void) { - return (*current_target.to_supports_multi_process) (¤t_target); + return target_stack->supports_multi_process (); } /* See target.h. */ @@ -2734,9 +2574,8 @@ target_get_osdata (const char *type) /* If we're already connected to something that can get us OS related data, use it. Otherwise, try using the native target. */ - if (current_target.to_stratum >= process_stratum) - t = current_target.beneath; - else + t = find_target_at (process_stratum); + if (t == NULL) t = find_default_run_target ("get OS data"); if (!t) @@ -2769,24 +2608,81 @@ target_thread_address_space (ptid_t ptid) { struct address_space *aspace; - aspace = current_target.to_thread_address_space (¤t_target, ptid); + aspace = target_stack->thread_address_space (ptid); gdb_assert (aspace != NULL); return aspace; } +void +target_ops::open (const char *, int) +{ + gdb_assert_not_reached ("target_ops::open called"); +} + +void +target_ops::close () +{ +} + +bool +target_ops::can_attach () +{ + return 0; +} + +void +target_ops::attach (const char *, int) +{ + gdb_assert_not_reached ("target_ops::attach called"); +} + +bool +target_ops::can_create_inferior () +{ + return 0; +} + +void +target_ops::create_inferior (const char *, const std::string &, + char **, int) +{ + gdb_assert_not_reached ("target_ops::create_inferior called"); +} + +int +target_ops::can_run () +{ + return 0; +} + +int +target_can_run () +{ + struct target_ops *t; + + for (t = target_stack; t != NULL; t = t->beneath) + { + if (t->can_run ()) + return 1; + } + + return 0; +} /* Target file operations. */ static struct target_ops * default_fileio_target (void) { + struct target_ops *t; + /* If we're already connected to something that can perform file I/O, use it. Otherwise, try using the native target. */ - if (current_target.to_stratum >= process_stratum) - return current_target.beneath; - else - return find_default_run_target ("file I/O"); + t = find_target_at (process_stratum); + if (t != NULL) + return t; + return find_default_run_target ("file I/O"); } /* File handle for target file operations. */ @@ -2878,6 +2774,67 @@ fileio_fd_to_fh (int fd) return &fileio_fhandles[fd]; } + +/* Default implementations of file i/o methods. We don't want these + to delegate automatically, because we need to know which target + supported the method, in order to call it directly from within + pread/pwrite, etc. */ + +int +target_ops::fileio_open (struct inferior *inf, const char *filename, + int flags, int mode, int warn_if_slow, + int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return -1; +} + +int +target_ops::fileio_pwrite (int fd, const gdb_byte *write_buf, int len, + ULONGEST offset, int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return -1; +} + +int +target_ops::fileio_pread (int fd, gdb_byte *read_buf, int len, + ULONGEST offset, int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return -1; +} + +int +target_ops::fileio_fstat (int fd, struct stat *sb, int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return -1; +} + +int +target_ops::fileio_close (int fd, int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return -1; +} + +int +target_ops::fileio_unlink (struct inferior *inf, const char *filename, + int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return -1; +} + +gdb::optional +target_ops::fileio_readlink (struct inferior *inf, const char *filename, + int *target_errno) +{ + *target_errno = FILEIO_ENOSYS; + return {}; +} + /* Helper for target_fileio_open and target_fileio_open_warn_if_slow. */ @@ -2890,26 +2847,26 @@ target_fileio_open_1 (struct inferior *inf, const char *filename, for (t = default_fileio_target (); t != NULL; t = t->beneath) { - if (t->to_fileio_open != NULL) - { - int fd = t->to_fileio_open (t, inf, filename, flags, mode, - warn_if_slow, target_errno); + int fd = t->fileio_open (inf, filename, flags, mode, + warn_if_slow, target_errno); - if (fd < 0) - fd = -1; - else - fd = acquire_fileio_fd (t, fd); + if (fd == -1 && *target_errno == FILEIO_ENOSYS) + continue; - if (targetdebug) - fprintf_unfiltered (gdb_stdlog, + if (fd < 0) + fd = -1; + else + fd = acquire_fileio_fd (t, fd); + + if (targetdebug) + fprintf_unfiltered (gdb_stdlog, "target_fileio_open (%d,%s,0x%x,0%o,%d)" " = %d (%d)\n", inf == NULL ? 0 : inf->num, filename, flags, mode, warn_if_slow, fd, fd != -1 ? 0 : *target_errno); - return fd; - } + return fd; } *target_errno = FILEIO_ENOSYS; @@ -2951,8 +2908,8 @@ target_fileio_pwrite (int fd, const gdb_byte *write_buf, int len, else if (fh->target == NULL) *target_errno = EIO; else - ret = fh->target->to_fileio_pwrite (fh->target, fh->target_fd, write_buf, - len, offset, target_errno); + ret = fh->target->fileio_pwrite (fh->target_fd, write_buf, + len, offset, target_errno); if (targetdebug) fprintf_unfiltered (gdb_stdlog, @@ -2977,8 +2934,8 @@ target_fileio_pread (int fd, gdb_byte *read_buf, int len, else if (fh->target == NULL) *target_errno = EIO; else - ret = fh->target->to_fileio_pread (fh->target, fh->target_fd, read_buf, - len, offset, target_errno); + ret = fh->target->fileio_pread (fh->target_fd, read_buf, + len, offset, target_errno); if (targetdebug) fprintf_unfiltered (gdb_stdlog, @@ -3002,8 +2959,7 @@ target_fileio_fstat (int fd, struct stat *sb, int *target_errno) else if (fh->target == NULL) *target_errno = EIO; else - ret = fh->target->to_fileio_fstat (fh->target, fh->target_fd, - sb, target_errno); + ret = fh->target->fileio_fstat (fh->target_fd, sb, target_errno); if (targetdebug) fprintf_unfiltered (gdb_stdlog, @@ -3025,8 +2981,8 @@ target_fileio_close (int fd, int *target_errno) else { if (fh->target != NULL) - ret = fh->target->to_fileio_close (fh->target, fh->target_fd, - target_errno); + ret = fh->target->fileio_close (fh->target_fd, + target_errno); else ret = 0; release_fileio_fd (fd, fh); @@ -3049,19 +3005,18 @@ target_fileio_unlink (struct inferior *inf, const char *filename, for (t = default_fileio_target (); t != NULL; t = t->beneath) { - if (t->to_fileio_unlink != NULL) - { - int ret = t->to_fileio_unlink (t, inf, filename, - target_errno); + int ret = t->fileio_unlink (inf, filename, target_errno); - if (targetdebug) - fprintf_unfiltered (gdb_stdlog, - "target_fileio_unlink (%d,%s)" - " = %d (%d)\n", - inf == NULL ? 0 : inf->num, filename, - ret, ret != -1 ? 0 : *target_errno); - return ret; - } + if (ret == -1 && *target_errno == FILEIO_ENOSYS) + continue; + + if (targetdebug) + fprintf_unfiltered (gdb_stdlog, + "target_fileio_unlink (%d,%s)" + " = %d (%d)\n", + inf == NULL ? 0 : inf->num, filename, + ret, ret != -1 ? 0 : *target_errno); + return ret; } *target_errno = FILEIO_ENOSYS; @@ -3078,20 +3033,20 @@ target_fileio_readlink (struct inferior *inf, const char *filename, for (t = default_fileio_target (); t != NULL; t = t->beneath) { - if (t->to_fileio_readlink != NULL) - { - gdb::optional ret - = t->to_fileio_readlink (t, inf, filename, target_errno); + gdb::optional ret + = t->fileio_readlink (inf, filename, target_errno); - if (targetdebug) - fprintf_unfiltered (gdb_stdlog, - "target_fileio_readlink (%d,%s)" - " = %s (%d)\n", - inf == NULL ? 0 : inf->num, - filename, ret ? ret->c_str () : "(nil)", - ret ? 0 : *target_errno); - return ret; - } + if (!ret.has_value () && *target_errno == FILEIO_ENOSYS) + continue; + + if (targetdebug) + fprintf_unfiltered (gdb_stdlog, + "target_fileio_readlink (%d,%s)" + " = %s (%d)\n", + inf == NULL ? 0 : inf->num, + filename, ret ? ret->c_str () : "(nil)", + ret ? 0 : *target_errno); + return ret; } *target_errno = FILEIO_ENOSYS; @@ -3256,18 +3211,6 @@ default_thread_architecture (struct target_ops *ops, ptid_t ptid) return inf->gdbarch; } -static int -return_zero (struct target_ops *ignore) -{ - return 0; -} - -static int -return_zero_has_execution (struct target_ops *ignore, ptid_t ignore2) -{ - return 0; -} - /* * Find the next target down the stack from the specified target. */ @@ -3285,7 +3228,7 @@ find_target_at (enum strata stratum) { struct target_ops *t; - for (t = current_target.beneath; t != NULL; t = t->beneath) + for (t = target_stack; t != NULL; t = t->beneath) if (t->to_stratum == stratum) return t; @@ -3386,27 +3329,55 @@ dummy_make_corefile_notes (struct target_ops *self, return NULL; } -/* Set up the handful of non-empty slots needed by the dummy target - vector. */ +#include "target-delegates.c" + -static void -init_dummy_target (void) -{ - dummy_target.to_shortname = "None"; - dummy_target.to_longname = "None"; - dummy_target.to_doc = ""; - dummy_target.to_supports_disable_randomization - = find_default_supports_disable_randomization; - dummy_target.to_stratum = dummy_stratum; - dummy_target.to_has_all_memory = return_zero; - dummy_target.to_has_memory = return_zero; - dummy_target.to_has_stack = return_zero; - dummy_target.to_has_registers = return_zero; - dummy_target.to_has_execution = return_zero_has_execution; - dummy_target.to_magic = OPS_MAGIC; - - install_dummy_methods (&dummy_target); +dummy_target::dummy_target () +{ + to_stratum = dummy_stratum; +} + +const char * +dummy_target::shortname () +{ + return "None"; +} + +const char * +dummy_target::longname () +{ + return _("None"); } + +const char * +dummy_target::doc () +{ + return ""; +} + +debug_target::debug_target () +{ + to_stratum = debug_stratum; +} + +const char * +debug_target::shortname () +{ + return beneath->shortname (); +} + +const char * +debug_target::longname () +{ + return beneath->longname (); +} + +const char * +debug_target::doc () +{ + return beneath->doc (); +} + void @@ -3416,8 +3387,7 @@ target_close (struct target_ops *targ) fileio_handles_invalidate_target (targ); - if (targ->to_close != NULL) - targ->to_close (targ); + targ->close (); if (targetdebug) fprintf_unfiltered (gdb_stdlog, "target_close ()\n"); @@ -3426,13 +3396,13 @@ target_close (struct target_ops *targ) int target_thread_alive (ptid_t ptid) { - return current_target.to_thread_alive (¤t_target, ptid); + return target_stack->thread_alive (ptid); } void target_update_thread_list (void) { - current_target.to_update_thread_list (¤t_target); + target_stack->update_thread_list (); } void @@ -3444,7 +3414,7 @@ target_stop (ptid_t ptid) return; } - (*current_target.to_stop) (¤t_target, ptid); + target_stack->stop (ptid); } void @@ -3456,7 +3426,7 @@ target_interrupt () return; } - (*current_target.to_interrupt) (¤t_target); + target_stack->interrupt (); } /* See target.h. */ @@ -3464,7 +3434,7 @@ target_interrupt () void target_pass_ctrlc (void) { - (*current_target.to_pass_ctrlc) (¤t_target); + target_stack->pass_ctrlc (); } /* See target.h. */ @@ -3559,7 +3529,7 @@ target_options_to_string (int target_options) void target_fetch_registers (struct regcache *regcache, int regno) { - current_target.to_fetch_registers (¤t_target, regcache, regno); + target_stack->fetch_registers (regcache, regno); if (targetdebug) regcache->debug_print_register ("target_fetch_registers", regno); } @@ -3570,7 +3540,7 @@ target_store_registers (struct regcache *regcache, int regno) if (!may_write_registers) error (_("Writing to registers is not allowed (regno %d)"), regno); - current_target.to_store_registers (¤t_target, regcache, regno); + target_stack->store_registers (regcache, regno); if (targetdebug) { regcache->debug_print_register ("target_store_registers", regno); @@ -3580,7 +3550,7 @@ target_store_registers (struct regcache *regcache, int regno) int target_core_of_thread (ptid_t ptid) { - return current_target.to_core_of_thread (¤t_target, ptid); + return target_stack->core_of_thread (ptid); } int @@ -3618,15 +3588,14 @@ default_verify_memory (struct target_ops *self, const gdb_byte *data, CORE_ADDR memaddr, ULONGEST size) { /* Start over from the top of the target stack. */ - return simple_verify_memory (current_target.beneath, + return simple_verify_memory (target_stack, data, memaddr, size); } int target_verify_memory (const gdb_byte *data, CORE_ADDR memaddr, ULONGEST size) { - return current_target.to_verify_memory (¤t_target, - data, memaddr, size); + return target_stack->verify_memory (data, memaddr, size); } /* The documentation for this function is in its prototype declaration in @@ -3636,8 +3605,7 @@ int target_insert_mask_watchpoint (CORE_ADDR addr, CORE_ADDR mask, enum target_hw_bp_type rw) { - return current_target.to_insert_mask_watchpoint (¤t_target, - addr, mask, rw); + return target_stack->insert_mask_watchpoint (addr, mask, rw); } /* The documentation for this function is in its prototype declaration in @@ -3647,8 +3615,7 @@ int target_remove_mask_watchpoint (CORE_ADDR addr, CORE_ADDR mask, enum target_hw_bp_type rw) { - return current_target.to_remove_mask_watchpoint (¤t_target, - addr, mask, rw); + return target_stack->remove_mask_watchpoint (addr, mask, rw); } /* The documentation for this function is in its prototype declaration @@ -3657,8 +3624,7 @@ target_remove_mask_watchpoint (CORE_ADDR addr, CORE_ADDR mask, int target_masked_watch_num_registers (CORE_ADDR addr, CORE_ADDR mask) { - return current_target.to_masked_watch_num_registers (¤t_target, - addr, mask); + return target_stack->masked_watch_num_registers (addr, mask); } /* The documentation for this function is in its prototype declaration @@ -3667,7 +3633,7 @@ target_masked_watch_num_registers (CORE_ADDR addr, CORE_ADDR mask) int target_ranged_break_num_registers (void) { - return current_target.to_ranged_break_num_registers (¤t_target); + return target_stack->ranged_break_num_registers (); } /* See target.h. */ @@ -3675,7 +3641,7 @@ target_ranged_break_num_registers (void) struct btrace_target_info * target_enable_btrace (ptid_t ptid, const struct btrace_config *conf) { - return current_target.to_enable_btrace (¤t_target, ptid, conf); + return target_stack->enable_btrace (ptid, conf); } /* See target.h. */ @@ -3683,7 +3649,7 @@ target_enable_btrace (ptid_t ptid, const struct btrace_config *conf) void target_disable_btrace (struct btrace_target_info *btinfo) { - current_target.to_disable_btrace (¤t_target, btinfo); + target_stack->disable_btrace (btinfo); } /* See target.h. */ @@ -3691,7 +3657,7 @@ target_disable_btrace (struct btrace_target_info *btinfo) void target_teardown_btrace (struct btrace_target_info *btinfo) { - current_target.to_teardown_btrace (¤t_target, btinfo); + target_stack->teardown_btrace (btinfo); } /* See target.h. */ @@ -3701,7 +3667,7 @@ target_read_btrace (struct btrace_data *btrace, struct btrace_target_info *btinfo, enum btrace_read_type type) { - return current_target.to_read_btrace (¤t_target, btrace, btinfo, type); + return target_stack->read_btrace (btrace, btinfo, type); } /* See target.h. */ @@ -3709,7 +3675,7 @@ target_read_btrace (struct btrace_data *btrace, const struct btrace_config * target_btrace_conf (const struct btrace_target_info *btinfo) { - return current_target.to_btrace_conf (¤t_target, btinfo); + return target_stack->btrace_conf (btinfo); } /* See target.h. */ @@ -3717,7 +3683,7 @@ target_btrace_conf (const struct btrace_target_info *btinfo) void target_stop_recording (void) { - current_target.to_stop_recording (¤t_target); + target_stack->stop_recording (); } /* See target.h. */ @@ -3725,22 +3691,15 @@ target_stop_recording (void) void target_save_record (const char *filename) { - current_target.to_save_record (¤t_target, filename); + target_stack->save_record (filename); } /* See target.h. */ int -target_supports_delete_record (void) +target_supports_delete_record () { - struct target_ops *t; - - for (t = current_target.beneath; t != NULL; t = t->beneath) - if (t->to_delete_record != delegate_delete_record - && t->to_delete_record != tdefault_delete_record) - return 1; - - return 0; + return target_stack->supports_delete_record (); } /* See target.h. */ @@ -3748,7 +3707,7 @@ target_supports_delete_record (void) void target_delete_record (void) { - current_target.to_delete_record (¤t_target); + target_stack->delete_record (); } /* See target.h. */ @@ -3756,7 +3715,7 @@ target_delete_record (void) enum record_method target_record_method (ptid_t ptid) { - return current_target.to_record_method (¤t_target, ptid); + return target_stack->record_method (ptid); } /* See target.h. */ @@ -3764,7 +3723,7 @@ target_record_method (ptid_t ptid) int target_record_is_replaying (ptid_t ptid) { - return current_target.to_record_is_replaying (¤t_target, ptid); + return target_stack->record_is_replaying (ptid); } /* See target.h. */ @@ -3772,7 +3731,7 @@ target_record_is_replaying (ptid_t ptid) int target_record_will_replay (ptid_t ptid, int dir) { - return current_target.to_record_will_replay (¤t_target, ptid, dir); + return target_stack->record_will_replay (ptid, dir); } /* See target.h. */ @@ -3780,7 +3739,7 @@ target_record_will_replay (ptid_t ptid, int dir) void target_record_stop_replaying (void) { - current_target.to_record_stop_replaying (¤t_target); + target_stack->record_stop_replaying (); } /* See target.h. */ @@ -3788,7 +3747,7 @@ target_record_stop_replaying (void) void target_goto_record_begin (void) { - current_target.to_goto_record_begin (¤t_target); + target_stack->goto_record_begin (); } /* See target.h. */ @@ -3796,7 +3755,7 @@ target_goto_record_begin (void) void target_goto_record_end (void) { - current_target.to_goto_record_end (¤t_target); + target_stack->goto_record_end (); } /* See target.h. */ @@ -3804,7 +3763,7 @@ target_goto_record_end (void) void target_goto_record (ULONGEST insn) { - current_target.to_goto_record (¤t_target, insn); + target_stack->goto_record (insn); } /* See target.h. */ @@ -3812,7 +3771,7 @@ target_goto_record (ULONGEST insn) void target_insn_history (int size, gdb_disassembly_flags flags) { - current_target.to_insn_history (¤t_target, size, flags); + target_stack->insn_history (size, flags); } /* See target.h. */ @@ -3821,7 +3780,7 @@ void target_insn_history_from (ULONGEST from, int size, gdb_disassembly_flags flags) { - current_target.to_insn_history_from (¤t_target, from, size, flags); + target_stack->insn_history_from (from, size, flags); } /* See target.h. */ @@ -3830,7 +3789,7 @@ void target_insn_history_range (ULONGEST begin, ULONGEST end, gdb_disassembly_flags flags) { - current_target.to_insn_history_range (¤t_target, begin, end, flags); + target_stack->insn_history_range (begin, end, flags); } /* See target.h. */ @@ -3838,7 +3797,7 @@ target_insn_history_range (ULONGEST begin, ULONGEST end, void target_call_history (int size, record_print_flags flags) { - current_target.to_call_history (¤t_target, size, flags); + target_stack->call_history (size, flags); } /* See target.h. */ @@ -3846,7 +3805,7 @@ target_call_history (int size, record_print_flags flags) void target_call_history_from (ULONGEST begin, int size, record_print_flags flags) { - current_target.to_call_history_from (¤t_target, begin, size, flags); + target_stack->call_history_from (begin, size, flags); } /* See target.h. */ @@ -3854,7 +3813,7 @@ target_call_history_from (ULONGEST begin, int size, record_print_flags flags) void target_call_history_range (ULONGEST begin, ULONGEST end, record_print_flags flags) { - current_target.to_call_history_range (¤t_target, begin, end, flags); + target_stack->call_history_range (begin, end, flags); } /* See target.h. */ @@ -3862,7 +3821,7 @@ target_call_history_range (ULONGEST begin, ULONGEST end, record_print_flags flag const struct frame_unwind * target_get_unwinder (void) { - return current_target.to_get_unwinder (¤t_target); + return target_stack->get_unwinder (); } /* See target.h. */ @@ -3870,7 +3829,7 @@ target_get_unwinder (void) const struct frame_unwind * target_get_tailcall_unwinder (void) { - return current_target.to_get_tailcall_unwinder (¤t_target); + return target_stack->get_tailcall_unwinder (); } /* See target.h. */ @@ -3878,7 +3837,7 @@ target_get_tailcall_unwinder (void) void target_prepare_to_generate_core (void) { - current_target.to_prepare_to_generate_core (¤t_target); + target_stack->prepare_to_generate_core (); } /* See target.h. */ @@ -3886,16 +3845,9 @@ target_prepare_to_generate_core (void) void target_done_generating_core (void) { - current_target.to_done_generating_core (¤t_target); + target_stack->done_generating_core (); } -static void -setup_target_debug (void) -{ - memcpy (&debug_target, ¤t_target, sizeof debug_target); - - init_debug_target (¤t_target); -} static char targ_desc[] = @@ -3965,7 +3917,9 @@ maintenance_print_target_stack (const char *cmd, int from_tty) for (t = target_stack; t != NULL; t = t->beneath) { - printf_filtered (" - %s (%s)\n", t->to_shortname, t->to_longname); + if (t->to_stratum == debug_stratum) + continue; + printf_filtered (" - %s (%s)\n", t->shortname (), t->longname ()); } } @@ -3975,7 +3929,7 @@ void target_async (int enable) { infrun_async (enable); - current_target.to_async (¤t_target, enable); + target_stack->async (enable); } /* See target.h. */ @@ -3983,7 +3937,7 @@ target_async (int enable) void target_thread_events (int enable) { - current_target.to_thread_events (¤t_target, enable); + target_stack->thread_events (enable); } /* Controls if targets can report that they can/are async. This is @@ -4023,7 +3977,7 @@ maint_show_target_async_command (struct ui_file *file, int from_tty, static int target_always_non_stop_p (void) { - return current_target.to_always_non_stop_p (¤t_target); + return target_stack->always_non_stop_p (); } /* See target.h. */ @@ -4133,59 +4087,13 @@ set_write_memory_permission (const char *args, int from_tty, update_observer_mode (); } -#if GDB_SELF_TEST -namespace selftests { - -static int -test_target_has_registers (target_ops *self) -{ - return 1; -} - -static int -test_target_has_stack (target_ops *self) -{ - return 1; -} - -static int -test_target_has_memory (target_ops *self) -{ - return 1; -} - -static void -test_target_prepare_to_store (target_ops *self, regcache *regs) -{ -} - -static void -test_target_store_registers (target_ops *self, regcache *regs, int regno) -{ -} - -test_target_ops::test_target_ops () - : target_ops {} -{ - to_magic = OPS_MAGIC; - to_stratum = process_stratum; - to_has_memory = test_target_has_memory; - to_has_stack = test_target_has_stack; - to_has_registers = test_target_has_registers; - to_prepare_to_store = test_target_prepare_to_store; - to_store_registers = test_target_store_registers; - - complete_target_initialization (this); -} - -} // namespace selftests -#endif /* GDB_SELF_TEST */ - void initialize_targets (void) { - init_dummy_target (); - push_target (&dummy_target); + the_dummy_target = new dummy_target (); + push_target (the_dummy_target); + + the_debug_target = new debug_target (); add_info ("target", info_target_command, targ_desc); add_info ("files", info_target_command, targ_desc); -- cgit v1.1