[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gdbstub: Have syscall_complete/[gs]et_reg to target agnostic
From: |
Alex Bennée |
Subject: |
Re: [PATCH] gdbstub: Have syscall_complete/[gs]et_reg to target agnostic typedefs |
Date: |
Wed, 14 Dec 2022 16:09:28 +0000 |
User-agent: |
mu4e 1.9.6; emacs 29.0.60 |
Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> On 14/12/22 15:36, Philippe Mathieu-Daudé wrote:
>> Prototypes using gdb_syscall_complete_cb() or gdb_?et_reg_cb()
>> don't depend on "cpu.h", thus are not target-specific.
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> ---
>> include/exec/gdbstub.h | 7 ++++---
>> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> Subject should be:
> "gdbstub: Make syscall_complete/[gs]et_reg target-agnostic typedefs"
Queued to gdbstub/next, thanks.
>
>> diff --git a/include/exec/gdbstub.h b/include/exec/gdbstub.h
>> index f667014888..1636fb3841 100644
>> --- a/include/exec/gdbstub.h
>> +++ b/include/exec/gdbstub.h
>> @@ -71,9 +71,6 @@ struct gdb_timeval {
>> uint64_t tv_usec; /* microsecond */
>> } QEMU_PACKED;
>> -#ifdef NEED_CPU_H
>> -#include "cpu.h"
>> -
>> typedef void (*gdb_syscall_complete_cb)(CPUState *cpu, uint64_t ret, int
>> err);
>> /**
>> @@ -126,6 +123,7 @@ int gdb_handlesig(CPUState *, int);
>> void gdb_signalled(CPUArchState *, int);
>> void gdbserver_fork(CPUState *);
>> #endif
>> +
>> /* Get or set a register. Returns the size of the register. */
>> typedef int (*gdb_get_reg_cb)(CPUArchState *env, GByteArray *buf, int reg);
>> typedef int (*gdb_set_reg_cb)(CPUArchState *env, uint8_t *buf, int reg);
>> @@ -133,6 +131,9 @@ void gdb_register_coprocessor(CPUState *cpu,
>> gdb_get_reg_cb get_reg, gdb_set_reg_cb
>> set_reg,
>> int num_regs, const char *xml, int g_pos);
>> +#ifdef NEED_CPU_H
>> +#include "cpu.h"
>> +
>> /*
>> * The GDB remote protocol transfers values in target byte order. As
>> * the gdbstub may be batching up several register values we always
--
Alex Bennée
Virtualisation Tech Lead @ Linaro