From b34970d56d2f1d9dbde4dfeb3c9d2e174b04b841 Mon Sep 17 00:00:00 2001 From: drow Date: Tue, 17 Oct 2006 21:55:23 +0000 Subject: [PATCH] gdb/ * gdbint.texinfo (Target Vector Definition): Move most content into Existing Targets. Add a menu. (Existing Targets): New section, moved from Target Vector Definition. Use @subsection. (Managing Execution State): New section. gdb/doc/ * remote-sim.c (gdbsim_kill): Call target_mourn_inferior. (gdbsim_load): Don't bother to adjust inferior_ptid here. (gdbsim_create_inferior): Mark the simulator as running. (gdbsim_open): Don't bother fetching registers. Mark the target as not running. (gdbsim_xfer): When the program is not running, pass memory requests down. (gdbsim_mourn_inferior): Mark the target as not running. * target.c (target_mark_running, target_mark_exited): New. * target.h (target_has_execution): Update the comment. (target_mark_running, target_mark_exited): New prototypes. --- gdb/ChangeLog | 14 +++++++++++++ gdb/doc/ChangeLog | 8 +++++++ gdb/doc/gdbint.texinfo | 57 ++++++++++++++++++++++++++++++++++++++++++++------ gdb/remote-sim.c | 20 +++++++++++++----- gdb/target.c | 50 +++++++++++++++++++++++++++++++++++++++++++ gdb/target.h | 18 +++++++++++----- 6 files changed, 151 insertions(+), 16 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1c5185f838..6dc9002edd 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,17 @@ +2006-10-17 Daniel Jacobowitz + + * remote-sim.c (gdbsim_kill): Call target_mourn_inferior. + (gdbsim_load): Don't bother to adjust inferior_ptid here. + (gdbsim_create_inferior): Mark the simulator as running. + (gdbsim_open): Don't bother fetching registers. Mark + the target as not running. + (gdbsim_xfer): When the program is not running, pass memory + requests down. + (gdbsim_mourn_inferior): Mark the target as not running. + * target.c (target_mark_running, target_mark_exited): New. + * target.h (target_has_execution): Update the comment. + (target_mark_running, target_mark_exited): New prototypes. + 2006-10-17 Joel Brobecker * breakpoint.c (free_valchain): Minor reformatting. diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index 5b0864283f..0d1548ec81 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,3 +1,11 @@ +2006-10-17 Daniel Jacobowitz + + * gdbint.texinfo (Target Vector Definition): Move most + content into Existing Targets. Add a menu. + (Existing Targets): New section, moved from Target Vector + Definition. Use @subsection. + (Managing Execution State): New section. + 2006-10-16 Bob Wilson * gdb.texinfo (ST2000): Use Ctrl- instead of C-. diff --git a/gdb/doc/gdbint.texinfo b/gdb/doc/gdbint.texinfo index d370cb5460..cb77108d70 100644 --- a/gdb/doc/gdbint.texinfo +++ b/gdb/doc/gdbint.texinfo @@ -4453,11 +4453,56 @@ actually exercises control over a process or a serial port. @value{GDBN} includes some 30-40 different target vectors; however, each configuration of @value{GDBN} includes only a few of them. -@section File Targets +@menu +* Managing Execution State:: +* Existing Targets:: +@end menu + +@node Managing Execution State +@section Managing Execution State +@cindex execution state + +A target vector can be completely inactive (not pushed on the target +stack), active but not running (pushed, but not connected to a fully +manifested inferior), or completely active (pushed, with an accessible +inferior). Most targets are only completely inactive or completely +active, but some support persistant connections to a target even +when the target has exited or not yet started. + +For example, connecting to the simulator using @code{target sim} does +not create a running program. Neither registers nor memory are +accessible until @code{run}. Similarly, after @code{kill}, the +program can not continue executing. But in both cases @value{GDBN} +remains connected to the simulator, and target-specific commands +are directed to the simulator. + +A target which only supports complete activation should push itself +onto the stack in its @code{to_open} routine (by calling +@code{push_target}), and unpush itself from the stack in its +@code{to_mourn_inferior} routine (by calling @code{unpush_target}). + +A target which supports both partial and complete activation should +still call @code{push_target} in @code{to_open}, but not call +@code{unpush_target} in @code{to_mourn_inferior}. Instead, it should +call either @code{target_mark_running} or @code{target_mark_exited} +in its @code{to_open}, depending on whether the target is fully active +after connection. It should also call @code{target_mark_running} any +time the inferior becomes fully active (e.g.@: in +@code{to_create_inferior} and @code{to_attach}), and +@code{target_mark_exited} when the inferior becomes inactive (in +@code{to_mourn_inferior}). The target should also make sure to call +@code{target_mourn_inferior} from its @code{to_kill}, to return the +target to inactive state. + +@node Existing Targets +@section Existing Targets +@cindex targets + +@subsection File Targets Both executables and core files have target vectors. -@section Standard Protocol and Remote Stubs +@subsection Standard Protocol and Remote Stubs @value{GDBN}'s file @file{remote.c} talks a serial protocol to code that runs in the target system. @value{GDBN} provides several sample @@ -4502,13 +4547,13 @@ of the debugger/stub. From reading the stub, it's probably not obvious how breakpoints work. They are simply done by deposit/examine operations from @value{GDBN}. -@section ROM Monitor Interface +@subsection ROM Monitor Interface -@section Custom Protocols +@subsection Custom Protocols -@section Transport Layer +@subsection Transport Layer -@section Builtin Simulator +@subsection Builtin Simulator @node Native Debugging diff --git a/gdb/remote-sim.c b/gdb/remote-sim.c index 4e36f54af9..9b432b0b2a 100644 --- a/gdb/remote-sim.c +++ b/gdb/remote-sim.c @@ -383,8 +383,8 @@ gdbsim_kill (void) printf_filtered ("gdbsim_kill\n"); /* There is no need to `kill' running simulator - the simulator is - not running */ - inferior_ptid = null_ptid; + not running. Mourning it is enough. */ + target_mourn_inferior (); } /* Load an executable file into the target process. This is expected to @@ -410,8 +410,6 @@ gdbsim_load (char *args, int fromtty) if (sr_get_debug ()) printf_filtered ("gdbsim_load: prog \"%s\"\n", prog); - inferior_ptid = null_ptid; - /* FIXME: We will print two messages on error. Need error to either not print anything if passed NULL or need another routine that doesn't take any arguments. */ @@ -469,6 +467,7 @@ gdbsim_create_inferior (char *exec_file, char *args, char **env, int from_tty) sim_create_inferior (gdbsim_desc, exec_bfd, argv, env); inferior_ptid = pid_to_ptid (42); + target_mark_running (&gdbsim_ops); insert_breakpoints (); /* Needed to get correct instruction in cache */ clear_proceed_status (); @@ -543,8 +542,12 @@ gdbsim_open (char *args, int from_tty) error (_("unable to create simulator instance")); push_target (&gdbsim_ops); - target_fetch_registers (-1); printf_filtered ("Connected to the simulator.\n"); + + /* There's nothing running after "target sim" or "load"; not until + "run". */ + inferior_ptid = null_ptid; + target_mark_exited (&gdbsim_ops); } /* Does whatever cleanup is required for a target that we are no longer @@ -747,6 +750,12 @@ gdbsim_xfer_inferior_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int write, struct mem_attrib *attrib, struct target_ops *target) { + /* If no program is running yet, then ignore the simulator for + memory. Pass the request down to the next target, hopefully + an exec file. */ + if (!target_has_execution) + return 0; + if (!program_loaded) error (_("No program loaded.")); @@ -802,6 +811,7 @@ gdbsim_mourn_inferior (void) printf_filtered ("gdbsim_mourn_inferior:\n"); remove_breakpoints (); + target_mark_exited (&gdbsim_ops); generic_mourn_inferior (); } diff --git a/gdb/target.c b/gdb/target.c index a757f53474..2ff882cf91 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -649,6 +649,56 @@ update_current_target (void) current_target.beneath = target_stack; } +/* Mark OPS as a running target. This reverses the effect + of target_mark_exited. */ + +void +target_mark_running (struct target_ops *ops) +{ + struct target_ops *t; + + for (t = target_stack; t != NULL; t = t->beneath) + if (t == ops) + break; + if (t == NULL) + internal_error (__FILE__, __LINE__, + "Attempted to mark unpushed target \"%s\" as running", + ops->to_shortname); + + ops->to_has_execution = 1; + ops->to_has_all_memory = 1; + ops->to_has_memory = 1; + ops->to_has_stack = 1; + ops->to_has_registers = 1; + + update_current_target (); +} + +/* Mark OPS as a non-running target. This reverses the effect + of target_mark_running. */ + +void +target_mark_exited (struct target_ops *ops) +{ + struct target_ops *t; + + for (t = target_stack; t != NULL; t = t->beneath) + if (t == ops) + break; + if (t == NULL) + internal_error (__FILE__, __LINE__, + "Attempted to mark unpushed target \"%s\" as running", + ops->to_shortname); + + ops->to_has_execution = 0; + ops->to_has_all_memory = 0; + ops->to_has_memory = 0; + ops->to_has_stack = 0; + ops->to_has_registers = 0; + + update_current_target (); +} + /* Push a new target type into the stack of the existing target accessors, possibly superseding some of the existing accessors. diff --git a/gdb/target.h b/gdb/target.h index 9399e76198..7be4aca5a0 100644 --- a/gdb/target.h +++ b/gdb/target.h @@ -967,11 +967,12 @@ int target_follow_fork (int follow_child); (current_target.to_has_registers) /* Does the target have execution? Can we make it jump (through - hoops), or pop its stack a few times? FIXME: If this is to work that - way, it needs to check whether an inferior actually exists. - remote-udi.c and probably other targets can be the current target - when the inferior doesn't actually exist at the moment. Right now - this just tells us whether this target is *capable* of execution. */ + hoops), or pop its stack a few times? This means that the current + target is currently executing; for some targets, that's the same as + whether or not the target is capable of execution, but there are + also targets which can be current while not executing. In that + case this will become true after target_create_inferior or + target_attach. */ #define target_has_execution \ (current_target.to_has_execution) @@ -1230,6 +1231,13 @@ extern void pop_target (void); extern CORE_ADDR target_translate_tls_address (struct objfile *objfile, CORE_ADDR offset); +/* Mark a pushed target as running or exited, for targets which do not + automatically pop when not active. */ + +void target_mark_running (struct target_ops *); + +void target_mark_exited (struct target_ops *); + /* Struct section_table maps address ranges to file sections. It is mostly used with BFD files, but can be used without (e.g. for handling raw disks, or files not in formats handled by BFD). */ -- 2.11.0