diff options
| author | Steven Stallion <stallion@squareup.com> | 2016-07-20 21:09:24 -0500 |
|---|---|---|
| committer | Paul Fertser <fercerpav@gmail.com> | 2016-12-08 12:34:00 +0000 |
| commit | 50dd7207eaae9d2d1f02e7996135d6257c03b395 (patch) | |
| tree | 11bb8cbfbcbd3ed67b3f4c263df43925dcb85614 /src/rtos/eCos.c | |
| parent | c0e7ccbd87cf491b065bf18bbdb6a73b741c2698 (diff) | |
| download | riscv-openocd-50dd7207eaae9d2d1f02e7996135d6257c03b395.zip riscv-openocd-50dd7207eaae9d2d1f02e7996135d6257c03b395.tar.gz riscv-openocd-50dd7207eaae9d2d1f02e7996135d6257c03b395.tar.bz2 | |
gdb_server: support qXfer:threads:read packet
This patch adds support for the qXfer:threads:read packet. In addition
to providing a more efficient method of updating thread state, recent
versions of GDB (7.11.1 and up) can also report remote thread names.
While thread names are not enabled in this patch due to its limited
applicability at the moment, it can be enabled at a later date with
little effort.
As a part of revamping how threads are presented to GDB, extra info
strings for each of the supported RTOSes were updated to match
conventions present in the GDB source code. For more information, see
remote_threads_extra_info() in remote.c. This results in a much smoother
experience when interacting with GDB.
It is also worth mentioning that use of qXfer:threads:read works around
a number of regressions in older versions of GDB regarding remote thread
display. Trust me, it's great.
Change-Id: I97dd6a93c342ceb9b9d0023b6359db0e5604c6e6
Signed-off-by: Steven Stallion <stallion@squareup.com>
Reviewed-on: http://openocd.zylin.com/3559
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Paul Fertser <fercerpav@gmail.com>
Diffstat (limited to 'src/rtos/eCos.c')
| -rw-r--r-- | src/rtos/eCos.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/rtos/eCos.c b/src/rtos/eCos.c index e38e11f..edc3d8b 100644 --- a/src/rtos/eCos.c +++ b/src/rtos/eCos.c @@ -261,8 +261,8 @@ static int eCos_update_threads(struct rtos *rtos) state_desc = "Unknown state"; rtos->thread_details[tasks_found].extra_info_str = malloc(strlen( - state_desc)+1); - strcpy(rtos->thread_details[tasks_found].extra_info_str, state_desc); + state_desc)+8); + sprintf(rtos->thread_details[tasks_found].extra_info_str, "State: %s", state_desc); rtos->thread_details[tasks_found].exists = true; |