On Tue, Oct 10, 2023 at 2:48 AM Alex Bennée <alex.ben...@linaro.org> wrote:
>
> From: Akihiko Odaki <akihiko.od...@daynix.com>
>
> 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.od...@daynix.com>
> Message-Id: <20230912224107.29669-12-akihiko.od...@daynix.com>
> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com>
Alistair
> ---
> gdbstub/internals.h | 2 --
> include/exec/gdbstub.h | 8 --------
> gdbstub/gdbstub.c | 15 ---------------
> 3 files changed, 25 deletions(-)
>
> diff --git a/gdbstub/internals.h b/gdbstub/internals.h
> index fee243081f..7128c4aa85 100644
> --- a/gdbstub/internals.h
> +++ b/gdbstub/internals.h
> @@ -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;
>
> diff --git a/include/exec/gdbstub.h b/include/exec/gdbstub.h
> index 705be2c5d7..1a01c35f8e 100644
> --- a/include/exec/gdbstub.h
> +++ b/include/exec/gdbstub.h
> @@ -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[];
>
> diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
> index 3dc847f835..62608a5389 100644
> --- a/gdbstub/gdbstub.c
> +++ b/gdbstub/gdbstub.c
> @@ -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)
> {
> @@ -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;
> @@ -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;
> --
> 2.39.2
>
>
