diff options
| author | Antonio Borneo <borneo.antonio@gmail.com> | 2018-08-28 12:29:09 +0200 |
|---|---|---|
| committer | Matthias Welwarsky <matthias@welwarsky.de> | 2018-10-01 20:58:52 +0100 |
| commit | 24654759d5fd92c0dbafac40c8604842dd6d6709 (patch) | |
| tree | 4cec3c49a333e4d71c76c328ea321a3fb983e67d /src/target/target.c | |
| parent | 6823a97beb706a5a3a4b7f813d33a7f3faadf2f0 (diff) | |
gdb_server: avoid gdb server for virtual targets
Virtual targets, like mem_ap, do not or cannot implement the required
functionality to accept a GDB connection. In the case of mem_ap, the
method get_gdb_reg_list() is missing and a following connection from
gdb causes OpenOCD to segfault.
OpenOCD opens a GDB port for each target; it's always possible to
connect, by mistake, GDB to one virtual target.
Add a method to check if the target supports GDB connections (for the
moment just checking if get_gdb_reg_list is implemented).
Skip opening a gdb server for every targets that don't support GDB
connections.
Change-Id: Ia439a43efe1a9adbb1771cd9d252db8ffa32eb9d
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/4676
Tested-by: jenkins
Reviewed-by: Matthias Welwarsky <matthias@welwarsky.de>
Diffstat (limited to 'src/target/target.c')
| -rw-r--r-- | src/target/target.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/target/target.c b/src/target/target.c index 060fbdca..478c39d1 100644 --- a/src/target/target.c +++ b/src/target/target.c @@ -1205,6 +1205,16 @@ int target_get_gdb_reg_list(struct target *target, { return target->type->get_gdb_reg_list(target, reg_list, reg_list_size, reg_class); } + +bool target_supports_gdb_connection(struct target *target) +{ + /* + * based on current code, we can simply exclude all the targets that + * don't provide get_gdb_reg_list; this could change with new targets. + */ + return !!target->type->get_gdb_reg_list; +} + int target_step(struct target *target, int current, target_addr_t address, int handle_breakpoints) { |