michaelh/zephyr
1
0
Fork 0
Code Releases Activity

debug: gdbstub: add arch-specific funcs to read/write registers

Browse source 
This adds architecture-specific functions to read/write registers.
This allows architecture to have a sparse representation of
the register file as not all registers are saved during context
switches. This saves some runtime space, and provides some
flexibility on what architectures can do.

Remove from header the need to define ARCH_GDB_NUM_REGISTERS as
it is no longer used in the common gdbstub code.

Signed-off-by: Daniel Leung <daniel.leung@intel.com>
This commit is contained in:
Daniel Leung 2021-10-14 10:14:50 -07:00 • committed by Anas Nashif
commit e76d385e49
2 changed files with 71 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

73
include/sys/arch_interface.h
View file

@ -819,14 +819,9 @@ static inline void arch_cohere_stacks(struct k_thread *old_thread,
* @{
*/
/**
* @def ARCH_GDB_NUM_REGISTERS
*
* ARCH_GDB_NUM_REGISTERS is architecure specific and
* this symbol must be defined in architecure specific header
*/
#ifdef CONFIG_GDBSTUB
struct gdb_ctx;
/**
* @brief Architecture layer debug start
*
@ -848,6 +843,70 @@ void arch_gdb_continue(void);
*/
void arch_gdb_step(void);
/**
* @brief Read all registers, and outputs as hexadecimal string.
*
* This reads all CPU registers and outputs as hexadecimal string.
* The output string must be parsable by GDB.
*
* @param ctx GDB context
* @param buf Buffer to output hexadecimal string.
* @param buflen Length of buffer.
*
* @return Length of hexadecimal string written.
* Return 0 if error or not supported.
*/
size_t arch_gdb_reg_readall(struct gdb_ctx *ctx, uint8_t *buf, size_t buflen);
/**
* @brief Take a hexadecimal string and update all registers.
*
* This takes in a hexadecimal string as presented from GDB,
* and updates all CPU registers with new values.
*
* @param ctx GDB context
* @param hex Input hexadecimal string.
* @param hexlen Length of hexadecimal string.
*
* @return Length of hexadecimal string parsed.
* Return 0 if error or not supported.
*/
size_t arch_gdb_reg_writeall(struct gdb_ctx *ctx, uint8_t *hex, size_t hexlen);
/**
* @brief Read one register, and outputs as hexadecimal string.
*
* This reads one CPU register and outputs as hexadecimal string.
* The output string must be parsable by GDB.
*
* @param ctx GDB context
* @param buf Buffer to output hexadecimal string.
* @param buflen Length of buffer.
* @param regno Register number
*
* @return Length of hexadecimal string written.
* Return 0 if error or not supported.
*/
size_t arch_gdb_reg_readone(struct gdb_ctx *ctx, uint8_t *buf, size_t buflen,
uint32_t regno);
/**
* @brief Take a hexadecimal string and update one register.
*
* This takes in a hexadecimal string as presented from GDB,
* and updates one CPU registers with new value.
*
* @param ctx GDB context
* @param hex Input hexadecimal string.
* @param hexlen Length of hexadecimal string.
* @param regno Register number
*
* @return Length of hexadecimal string parsed.
* Return 0 if error or not supported.
*/
size_t arch_gdb_reg_writeone(struct gdb_ctx *ctx, uint8_t *hex, size_t hexlen,
uint32_t regno);
#endif
/** @} */