Skip to content

Commit

Permalink
gdbstub: Remove gdb_has_xml variable
Browse files Browse the repository at this point in the history
GDB has XML support since 6.7 which was released in 2007.
It's time to remove support for old GDB versions without XML support.

Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Message-Id: <20230912224107.29669-12-akihiko.odaki@daynix.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20231009164104.369749-19-alex.bennee@linaro.org>
  • Loading branch information
akihikodaki authored and stsquad committed Oct 11, 2023
1 parent 8e6d3ea commit 213316d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 25 deletions.
15 changes: 0 additions & 15 deletions gdbstub/gdbstub.c
Original file line number Diff line number Diff line change
Expand Up @@ -349,11 +349,6 @@ static CPUState *gdb_get_cpu(uint32_t pid, uint32_t tid)
}
}

bool gdb_has_xml(void)
{
return !!gdb_get_cpu_process(gdbserver_state.g_cpu)->target_xml;
}

static const char *get_feature_xml(const char *p, const char **newp,
GDBProcess *process)
{
Expand Down Expand Up @@ -1086,11 +1081,6 @@ static void handle_set_reg(GArray *params, void *user_ctx)
{
int reg_size;

if (!gdb_get_cpu_process(gdbserver_state.g_cpu)->target_xml) {
gdb_put_packet("");
return;
}

if (params->len != 2) {
gdb_put_packet("E22");
return;
Expand All @@ -1107,11 +1097,6 @@ static void handle_get_reg(GArray *params, void *user_ctx)
{
int reg_size;

if (!gdb_get_cpu_process(gdbserver_state.g_cpu)->target_xml) {
gdb_put_packet("");
return;
}

if (!params->len) {
gdb_put_packet("E14");
return;
Expand Down
2 changes: 0 additions & 2 deletions gdbstub/internals.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@ enum {
typedef struct GDBProcess {
uint32_t pid;
bool attached;

/* If gdb sends qXfer:features:read:target.xml this will be populated */
char *target_xml;
} GDBProcess;

Expand Down
8 changes: 0 additions & 8 deletions include/exec/gdbstub.h
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,6 @@ int gdbserver_start(const char *port_or_device);

void gdb_set_stop_cpu(CPUState *cpu);

/**
* gdb_has_xml() - report of gdb supports modern target descriptions
*
* This will report true if the gdb negotiated qXfer:features:read
* target descriptions.
*/
bool gdb_has_xml(void);

/* in gdbstub-xml.c, generated by scripts/feature_to_c.py */
extern const GDBFeature gdb_static_features[];

Expand Down

0 comments on commit 213316d

Please sign in to comment.