diff options
author | Erhan Kurubas <erhan.kurubas@espressif.com> | 2023-01-21 20:10:21 +0100 |
---|---|---|
committer | Antonio Borneo <borneo.antonio@gmail.com> | 2023-01-28 15:50:50 +0000 |
commit | 59bc761d56f9ce9a76c917e267a2c37f1a680611 (patch) | |
tree | fbb01f1fb4e4d0640a2dbc6c3253614c486ab3d5 | |
parent | dfbbfac4d72e247e8094a49c8573b2f49689b6d5 (diff) | |
download | riscv-openocd-59bc761d56f9ce9a76c917e267a2c37f1a680611.zip riscv-openocd-59bc761d56f9ce9a76c917e267a2c37f1a680611.tar.gz riscv-openocd-59bc761d56f9ce9a76c917e267a2c37f1a680611.tar.bz2 |
rtos: add custom stack read function
This is optional field for the targets which has to implement
their custom stack read function.
Signed-off-by: Erhan Kurubas <erhan.kurubas@espressif.com>
Change-Id: Icbc9ed66a052fc2cc0ef67e3ec4d85ab0c2c1b94
Reviewed-on: https://review.openocd.org/c/openocd/+/7442
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
-rw-r--r-- | src/rtos/rtos.c | 5 | ||||
-rw-r--r-- | src/rtos/rtos.h | 7 |
2 files changed, 11 insertions, 1 deletions
diff --git a/src/rtos/rtos.c b/src/rtos/rtos.c index 2e76b50..dfa158d 100644 --- a/src/rtos/rtos.c +++ b/src/rtos/rtos.c @@ -632,7 +632,10 @@ int rtos_generic_stack_read(struct target *target, if (stacking->stack_growth_direction == 1) address -= stacking->stack_registers_size; - retval = target_read_buffer(target, address, stacking->stack_registers_size, stack_data); + if (stacking->read_stack) + retval = stacking->read_stack(target, address, stacking, stack_data); + else + retval = target_read_buffer(target, address, stacking->stack_registers_size, stack_data); if (retval != ERROR_OK) { free(stack_data); LOG_ERROR("Error reading stack frame from thread"); diff --git a/src/rtos/rtos.h b/src/rtos/rtos.h index ace5751..9128c16 100644 --- a/src/rtos/rtos.h +++ b/src/rtos/rtos.h @@ -102,6 +102,13 @@ struct rtos_register_stacking { const struct rtos_register_stacking *stacking, target_addr_t stack_ptr); const struct stack_register_offset *register_offsets; + /* Optional field for targets which may have to implement their own stack read function. + * Because stack format can be weird or stack data needed to be edited before passing to the gdb. + */ + int (*read_stack)(struct target *target, + int64_t stack_ptr, + const struct rtos_register_stacking *stacking, + uint8_t *stack_data); }; #define GDB_THREAD_PACKET_NOT_CONSUMED (-40) |