mirror of
https://github.com/espressif/esp-idf.git
synced 2024-10-05 20:47:46 -04:00
0adf0f85dd
This commit fixes an issue with gdbstub, where it would list threads with TIDs 1 to N in qfThreadInfo/qsThreadInfo responses, and then would tell GDB that the current TID is 0 in the qC response. This caused an assertion failure in GDB, because it couldn't find the thread structure corresponding to TID 0: src/gdb/gdb/thread.c:93: internal-error: thread_info* inferior_thread(): Assertion `tp' failed. The issue was caused by the logic of qfThreadInfo/qsThreadInfo. If the "paniced" task index was 1, the code would report it in the response to qfThreadInfo, and then mistakenly skip task with index 0 in qsThreadInfo, due to the use of pre-increment instead of a post-increment. With that issue fixed, GDB assertion doesn't happen anymore. However the code contained a deeper problem, which manifested itself in the fact that GDB would incorrectly show task index 0 as the current task, after the above fix. Previous version of the code assumed that when GDB requests the thread list, it uses the first thread returned by the target as the "default" thread, and subsequently shows the user that the program is stopped in that thread. This assumption was incorrect. In fact, after connecting to a remote target, GDB obtains information about the "default" or "current" thread from two sources: 1. the 'thread' special register indicated in the status response ($T00thread;00000001#ee) 2. if the target has only sent the plain stop response ($T00#ee), GDB would ask for the current thread using a qC packet. With that in mind, it is not necessary to report the paniced task as the first task in qfThreadInfo response. We can simply returns the tasks in their natural order, and then indicate the current task in the qS packet response. However even that change does not fully resolve the issues with task list. The previous version of this code also incorrectly interpreted the meaning of GDB TIDs -1 and 0. When GDB sends an "Hg0" command early in the connection process, it doesn't expect the server to set task 0 as the current task, as the code assumed. Rather, it tells the server to "set any (arbitrary) task as the current one", and the most logical thing to do for the server that is already in "stopped" state is to keep the current task selection. Since TID 0 has a special meaning in GDB remote protocol, gdbstub code is now modified to map task indices (which start from 0) to GDB TIDs. GDB TIDs are arbitrary, and for simplicity we keep the same order and start counting them from 1. The summary of all the above changes is: 1. Use "task index + 1" as the TID reported to GDB 2. Report the tasks in natural order; don't complicate the code to make the paniced task first in the list. 3. Centralize modification of 'current_task_index' and 'regfile' in the new 'set_active_task' function, to improve encapsulation.
387 lines
13 KiB
C
387 lines
13 KiB
C
// Copyright 2015-2019 Espressif Systems (Shanghai) PTE LTD
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
#include <string.h>
|
|
#include "esp_gdbstub.h"
|
|
#include "esp_gdbstub_common.h"
|
|
#include "sdkconfig.h"
|
|
|
|
|
|
#ifdef CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
static inline int gdb_tid_to_task_index(int tid);
|
|
static inline int task_index_to_gdb_tid(int tid);
|
|
static void init_task_info(void);
|
|
static void find_paniced_task_index(void);
|
|
static void set_active_task(size_t index);
|
|
static int handle_task_commands(unsigned char *cmd, int len);
|
|
#endif
|
|
|
|
static void send_reason(void);
|
|
|
|
|
|
static esp_gdbstub_scratch_t s_scratch;
|
|
|
|
|
|
void esp_gdbstub_panic_handler(esp_gdbstub_frame_t *frame)
|
|
{
|
|
#ifndef CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
esp_gdbstub_frame_to_regfile(frame, &s_scratch.regfile);
|
|
#else
|
|
if (s_scratch.state == GDBSTUB_STARTED) {
|
|
/* We have re-entered GDB Stub. Try disabling task support. */
|
|
s_scratch.state = GDBSTUB_TASK_SUPPORT_DISABLED;
|
|
/* Flush any pending GDB packet (this creates a garbage value) */
|
|
esp_gdbstub_send_end();
|
|
} else if (s_scratch.state == GDBSTUB_NOT_STARTED) {
|
|
s_scratch.state = GDBSTUB_STARTED;
|
|
/* Save the paniced frame and get the list of tasks */
|
|
memcpy(&s_scratch.paniced_frame, frame, sizeof(*frame));
|
|
init_task_info();
|
|
find_paniced_task_index();
|
|
/* Current task is the paniced task */
|
|
if (s_scratch.paniced_task_index == GDBSTUB_CUR_TASK_INDEX_UNKNOWN) {
|
|
set_active_task(0);
|
|
} else {
|
|
set_active_task(s_scratch.paniced_task_index);
|
|
}
|
|
}
|
|
#endif // CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
|
|
esp_gdbstub_target_init();
|
|
s_scratch.signal = esp_gdbstub_get_signal(frame);
|
|
send_reason();
|
|
while (true) {
|
|
unsigned char *cmd;
|
|
size_t size;
|
|
int res = esp_gdbstub_read_command(&cmd, &size);
|
|
if (res > 0) {
|
|
/* character received instead of a command */
|
|
continue;
|
|
}
|
|
if (res == GDBSTUB_ST_ERR) {
|
|
esp_gdbstub_send_str_packet("E01");
|
|
continue;
|
|
}
|
|
res = esp_gdbstub_handle_command(cmd, size);
|
|
if (res == GDBSTUB_ST_ERR) {
|
|
esp_gdbstub_send_str_packet(NULL);
|
|
}
|
|
}
|
|
}
|
|
|
|
|
|
static void send_reason(void)
|
|
{
|
|
esp_gdbstub_send_start();
|
|
esp_gdbstub_send_char('T');
|
|
esp_gdbstub_send_hex(s_scratch.signal, 8);
|
|
esp_gdbstub_send_end();
|
|
}
|
|
|
|
static uint32_t gdbstub_hton(uint32_t i)
|
|
{
|
|
return __builtin_bswap32(i);
|
|
}
|
|
|
|
/** Send all registers to gdb */
|
|
static void handle_g_command(const unsigned char* cmd, int len)
|
|
{
|
|
uint32_t *p = (uint32_t *) &s_scratch.regfile;
|
|
esp_gdbstub_send_start();
|
|
for (int i = 0; i < sizeof(s_scratch.regfile) / sizeof(*p); ++i) {
|
|
esp_gdbstub_send_hex(gdbstub_hton(*p++), 32);
|
|
}
|
|
esp_gdbstub_send_end();
|
|
}
|
|
|
|
/** Receive register values from gdb */
|
|
static void handle_G_command(const unsigned char* cmd, int len)
|
|
{
|
|
uint32_t *p = (uint32_t *) &s_scratch.regfile;
|
|
for (int i = 0; i < sizeof(s_scratch.regfile) / sizeof(*p); ++i) {
|
|
*p++ = gdbstub_hton(esp_gdbstub_gethex(&cmd, 32));
|
|
}
|
|
esp_gdbstub_send_str_packet("OK");
|
|
}
|
|
|
|
/** Read memory to gdb */
|
|
static void handle_m_command(const unsigned char* cmd, int len)
|
|
{
|
|
intptr_t addr = (intptr_t) esp_gdbstub_gethex(&cmd, -1);
|
|
cmd++;
|
|
uint32_t size = esp_gdbstub_gethex(&cmd, -1);
|
|
|
|
if (esp_gdbstub_readmem(addr) < 0 || esp_gdbstub_readmem(addr + size - 1) < 0) {
|
|
esp_gdbstub_send_str_packet("E01");
|
|
return;
|
|
}
|
|
|
|
esp_gdbstub_send_start();
|
|
for (int i = 0; i < size; ++i) {
|
|
int b = esp_gdbstub_readmem(addr++);
|
|
esp_gdbstub_send_hex(b, 8);
|
|
}
|
|
esp_gdbstub_send_end();
|
|
}
|
|
|
|
/** Handle a command received from gdb */
|
|
int esp_gdbstub_handle_command(unsigned char *cmd, int len)
|
|
{
|
|
unsigned char *data = cmd + 1;
|
|
if (cmd[0] == 'g')
|
|
{
|
|
handle_g_command(data, len - 1);
|
|
} else if (cmd[0] == 'G') {
|
|
/* receive content for all registers from gdb */
|
|
handle_G_command(data, len - 1);
|
|
} else if (cmd[0] == 'm') {
|
|
/* read memory to gdb */
|
|
handle_m_command(data, len - 1);
|
|
} else if (cmd[0] == '?') {
|
|
/* Reply with stop reason */
|
|
send_reason();
|
|
#if CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
} else if (s_scratch.state != GDBSTUB_TASK_SUPPORT_DISABLED) {
|
|
return handle_task_commands(cmd, len);
|
|
#endif // CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
} else {
|
|
/* Unrecognized command */
|
|
return GDBSTUB_ST_ERR;
|
|
}
|
|
return GDBSTUB_ST_OK;
|
|
}
|
|
|
|
/* Everything below is related to the support for listing FreeRTOS tasks as threads in GDB */
|
|
|
|
#ifdef CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
|
|
/* Some terminology related to task/thread indices:
|
|
*
|
|
* "task index" — Index used here in gdbstub.c to enumberate all the tasks.
|
|
* Range is from 0 to <number of tasks> - 1.
|
|
* The order is defined by uxTaskGetSnapshotAll.
|
|
*
|
|
* "GDB TID" — Thread ID used by GDB internally and in the remote protocol.
|
|
* IDs of real threads can be any positive values other than 0.
|
|
* For example, OpenOCD uses the TCB pointer (cast to a 32-bit int) as GDB TID.
|
|
* Values 0 and -1, when used in place of TID in the remote protocol
|
|
* have special meanings:
|
|
* -1: indicates all threads, may be used in 'Hc' command
|
|
* 0: indicates an arbitrary process or thread (GDB client doesn't care, server decides)
|
|
*
|
|
* Since GDB TIDs are arbitrary, except that 0 is reserved, we set them using a simple rule:
|
|
* GDB TID = task index + 1.
|
|
* The two functions below perform the conversions between the kinds of IDs.
|
|
*/
|
|
static inline int gdb_tid_to_task_index(int tid)
|
|
{
|
|
return tid - 1;
|
|
}
|
|
|
|
static inline int task_index_to_gdb_tid(int tid)
|
|
{
|
|
return tid + 1;
|
|
}
|
|
|
|
static void init_task_info(void)
|
|
{
|
|
unsigned tcb_size;
|
|
s_scratch.task_count = uxTaskGetSnapshotAll(s_scratch.tasks, GDBSTUB_TASKS_NUM, &tcb_size);
|
|
}
|
|
|
|
static bool get_task_handle(size_t index, TaskHandle_t *handle)
|
|
{
|
|
if (index >= s_scratch.task_count) {
|
|
return false;
|
|
}
|
|
*handle = (TaskHandle_t) s_scratch.tasks[index].pxTCB;
|
|
return true;
|
|
}
|
|
|
|
/** Get the index of the task running on the current CPU, and save the result */
|
|
static void find_paniced_task_index(void)
|
|
{
|
|
TaskHandle_t cur_handle = xTaskGetCurrentTaskHandleForCPU(xPortGetCoreID());
|
|
TaskHandle_t handle;
|
|
for (int i = 0; i < s_scratch.task_count; i++) {
|
|
if (get_task_handle(i, &handle) && cur_handle == handle) {
|
|
s_scratch.paniced_task_index = i;
|
|
return;
|
|
}
|
|
}
|
|
s_scratch.paniced_task_index = GDBSTUB_CUR_TASK_INDEX_UNKNOWN;
|
|
}
|
|
|
|
/** Select the current task and update the regfile */
|
|
static void set_active_task(size_t index)
|
|
{
|
|
if (index == s_scratch.paniced_task_index) {
|
|
/* Have to get the registers from exception frame */
|
|
esp_gdbstub_frame_to_regfile(&s_scratch.paniced_frame, &s_scratch.regfile);
|
|
} else {
|
|
/* Get the registers from TCB.
|
|
* FIXME: for the task currently running on the other CPU, extracting the registers from TCB
|
|
* isn't valid. Need to use some IPC mechanism to obtain the registers of the other CPU.
|
|
*/
|
|
TaskHandle_t handle = NULL;
|
|
get_task_handle(index, &handle);
|
|
if (handle != NULL) {
|
|
esp_gdbstub_tcb_to_regfile(handle, &s_scratch.regfile);
|
|
}
|
|
}
|
|
s_scratch.current_task_index = index;
|
|
}
|
|
|
|
/** H command sets the "current task" for the purpose of further commands */
|
|
static void handle_H_command(const unsigned char* cmd, int len)
|
|
{
|
|
const char *ret = "OK";
|
|
if (cmd[1] == 'g') {
|
|
cmd += 2;
|
|
int requested_tid = esp_gdbstub_gethex(&cmd, -1);
|
|
int requested_task_index = gdb_tid_to_task_index(requested_tid);
|
|
if (requested_tid == 0) {
|
|
/* 0 means "arbitrary process or thread", keep the current thread */
|
|
} else if (requested_task_index >= s_scratch.task_count ||
|
|
requested_tid == -1) { /* -1 means "all threads", which doesn't make sense for "Hg" */
|
|
ret = "E00";
|
|
} else {
|
|
set_active_task(requested_task_index);
|
|
}
|
|
esp_gdbstub_send_str_packet(ret);
|
|
} else if (cmd[1] == 'c') {
|
|
/* Select the task for "continue" and "step" operations. No-op in post-mortem mode. */
|
|
/* Note that the argument may be -1 to indicate "all threads" or 0 to indicate "any thread" */
|
|
esp_gdbstub_send_str_packet(ret);
|
|
} else {
|
|
esp_gdbstub_send_str_packet(NULL);
|
|
}
|
|
}
|
|
|
|
/** qC returns the current thread ID */
|
|
static void handle_qC_command(const unsigned char* cmd, int len)
|
|
{
|
|
esp_gdbstub_send_start();
|
|
esp_gdbstub_send_str("QC");
|
|
esp_gdbstub_send_hex(task_index_to_gdb_tid(s_scratch.current_task_index), 32);
|
|
esp_gdbstub_send_end();
|
|
}
|
|
|
|
/** T command checks if the task is alive.
|
|
* Since GDB isn't going to ask about the tasks which haven't been listed by q*ThreadInfo,
|
|
* and the state of tasks can not change (no stepping allowed), simply return "OK" here.
|
|
*/
|
|
static void handle_T_command(const unsigned char* cmd, int len)
|
|
{
|
|
esp_gdbstub_send_str_packet("OK");
|
|
}
|
|
|
|
/** Called by qfThreadInfo and qsThreadInfo handlers */
|
|
static void send_single_thread_info(int task_index)
|
|
{
|
|
esp_gdbstub_send_start();
|
|
esp_gdbstub_send_str("m");
|
|
esp_gdbstub_send_hex(task_index_to_gdb_tid(task_index), 32);
|
|
esp_gdbstub_send_end();
|
|
}
|
|
|
|
/** qfThreadInfo requests the start of the thread list, qsThreadInfo (below) is repeated to
|
|
* get the subsequent threads.
|
|
*/
|
|
static void handle_qfThreadInfo_command(const unsigned char* cmd, int len)
|
|
{
|
|
assert(s_scratch.task_count > 0); /* There should be at least one task */
|
|
send_single_thread_info(0);
|
|
s_scratch.thread_info_index = 1;
|
|
}
|
|
|
|
static void handle_qsThreadInfo_command(const unsigned char* cmd, int len)
|
|
{
|
|
int task_index = s_scratch.thread_info_index;
|
|
if (task_index == s_scratch.task_count) {
|
|
/* No more tasks */
|
|
esp_gdbstub_send_str_packet("l");
|
|
return;
|
|
}
|
|
send_single_thread_info(s_scratch.thread_info_index);
|
|
s_scratch.thread_info_index++;
|
|
}
|
|
|
|
/** qThreadExtraInfo requests the thread name */
|
|
static void handle_qThreadExtraInfo_command(const unsigned char* cmd, int len)
|
|
{
|
|
cmd += sizeof("qThreadExtraInfo,") - 1;
|
|
int task_index = gdb_tid_to_task_index(esp_gdbstub_gethex(&cmd, -1));
|
|
TaskHandle_t handle;
|
|
if (!get_task_handle(task_index, &handle)) {
|
|
esp_gdbstub_send_str_packet("E01");
|
|
return;
|
|
}
|
|
esp_gdbstub_send_start();
|
|
const char* task_name = pcTaskGetTaskName(handle);
|
|
while (*task_name) {
|
|
esp_gdbstub_send_hex(*task_name, 8);
|
|
task_name++;
|
|
}
|
|
/** TODO: add "Running" or "Suspended" and "CPU0" or "CPU1" */
|
|
esp_gdbstub_send_end();
|
|
}
|
|
|
|
bool command_name_matches(const char* pattern, const unsigned char* ucmd, int len)
|
|
{
|
|
const char* cmd = (const char*) ucmd;
|
|
const char* end = cmd + len;
|
|
for (; *pattern && cmd < end; ++cmd, ++pattern) {
|
|
if (*pattern == '?') {
|
|
continue;
|
|
}
|
|
if (*pattern != *cmd) {
|
|
return false;
|
|
}
|
|
}
|
|
return *pattern == 0 && (cmd == end || *cmd == ',');
|
|
}
|
|
|
|
/** Handle all the thread-related commands */
|
|
static int handle_task_commands(unsigned char *cmd, int len)
|
|
{
|
|
if (cmd[0] == 'H') {
|
|
/* Continue with task */
|
|
handle_H_command(cmd, len);
|
|
} else if (cmd[0] == 'T') {
|
|
/* Task alive check */
|
|
handle_T_command(cmd, len);
|
|
} else if (cmd[0] == 'q') {
|
|
if (command_name_matches("qfThreadInfo", cmd, len)) {
|
|
handle_qfThreadInfo_command(cmd, len);
|
|
} else if (command_name_matches("qsThreadInfo", cmd, len)) {
|
|
handle_qsThreadInfo_command(cmd, len);
|
|
} else if (command_name_matches("qC", cmd, len)) {
|
|
handle_qC_command(cmd, len);
|
|
} else if (command_name_matches("qThreadExtraInfo", cmd, len)) {
|
|
handle_qThreadExtraInfo_command(cmd, len);
|
|
} else {
|
|
/* Unrecognized command */
|
|
return GDBSTUB_ST_ERR;
|
|
}
|
|
} else {
|
|
/* Unrecognized command */
|
|
return GDBSTUB_ST_ERR;
|
|
}
|
|
return GDBSTUB_ST_OK;
|
|
}
|
|
|
|
#endif // CONFIG_ESP_GDBSTUB_SUPPORT_TASKS
|
|
|