Re: [PATCH 2/9] gdbstub: move enums into separate header

[Pierrick Bouvier <pierrick.bouvier@linaro.org>]

On 6/12/24 08:35, Alex Bennée wrote:
> This is an experiment to further reduce the amount we throw into the
> exec headers. It might not be as useful as I initially thought because
> just under half of the users also need gdbserver_start().
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   include/exec/gdbstub.h    |  9 ---------
>   include/gdbstub/enums.h   | 21 +++++++++++++++++++++
>   accel/hvf/hvf-accel-ops.c |  2 +-
>   accel/kvm/kvm-all.c       |  2 +-
>   accel/tcg/tcg-accel-ops.c |  2 +-
>   gdbstub/user.c            |  1 +
>   monitor/hmp-cmds.c        |  3 ++-
>   system/vl.c               |  1 +
>   target/arm/hvf/hvf.c      |  2 +-
>   target/arm/hyp_gdbstub.c  |  2 +-
>   target/arm/kvm.c          |  2 +-
>   target/i386/kvm/kvm.c     |  2 +-
>   target/ppc/kvm.c          |  2 +-
>   target/s390x/kvm/kvm.c    |  2 +-
>   14 files changed, 34 insertions(+), 19 deletions(-)
>   create mode 100644 include/gdbstub/enums.h
> 
> diff --git a/include/exec/gdbstub.h b/include/exec/gdbstub.h
> index 008a92198a..1bd2c4ec2a 100644
> --- a/include/exec/gdbstub.h
> +++ b/include/exec/gdbstub.h
> @@ -1,15 +1,6 @@
>   #ifndef GDBSTUB_H
>   #define GDBSTUB_H
>   
> -#define DEFAULT_GDBSTUB_PORT "1234"
> -
> -/* GDB breakpoint/watchpoint types */
> -#define GDB_BREAKPOINT_SW        0
> -#define GDB_BREAKPOINT_HW        1
> -#define GDB_WATCHPOINT_WRITE     2
> -#define GDB_WATCHPOINT_READ      3
> -#define GDB_WATCHPOINT_ACCESS    4
> -
>   typedef struct GDBFeature {
>       const char *xmlname;
>       const char *xml;
> diff --git a/include/gdbstub/enums.h b/include/gdbstub/enums.h
> new file mode 100644
> index 0000000000..c4d54a1d08
> --- /dev/null
> +++ b/include/gdbstub/enums.h
> @@ -0,0 +1,21 @@
> +/*
> + * gdbstub enums
> + *
> + * Copyright (c) 2024 Linaro Ltd
> + *
> + * SPDX-License-Identifier: GPL-2.0-or-later
> + */
> +
> +#ifndef GDBSTUB_ENUMS_H
> +#define GDBSTUB_ENUMS_H
> +
> +#define DEFAULT_GDBSTUB_PORT "1234"
> +
> +/* GDB breakpoint/watchpoint types */
> +#define GDB_BREAKPOINT_SW        0
> +#define GDB_BREAKPOINT_HW        1
> +#define GDB_WATCHPOINT_WRITE     2
> +#define GDB_WATCHPOINT_READ      3
> +#define GDB_WATCHPOINT_ACCESS    4
> +
> +#endif /* GDBSTUB_ENUMS_H */
> diff --git a/accel/hvf/hvf-accel-ops.c b/accel/hvf/hvf-accel-ops.c
> index b2a37a2229..ac08cfb9f3 100644
> --- a/accel/hvf/hvf-accel-ops.c
> +++ b/accel/hvf/hvf-accel-ops.c
> @@ -52,7 +52,7 @@
>   #include "qemu/main-loop.h"
>   #include "exec/address-spaces.h"
>   #include "exec/exec-all.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "sysemu/cpus.h"
>   #include "sysemu/hvf.h"
>   #include "sysemu/hvf_int.h"
> diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c
> index 009b49de44..5680cd157e 100644
> --- a/accel/kvm/kvm-all.c
> +++ b/accel/kvm/kvm-all.c
> @@ -27,7 +27,7 @@
>   #include "hw/pci/msi.h"
>   #include "hw/pci/msix.h"
>   #include "hw/s390x/adapter.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "sysemu/kvm_int.h"
>   #include "sysemu/runstate.h"
>   #include "sysemu/cpus.h"
> diff --git a/accel/tcg/tcg-accel-ops.c b/accel/tcg/tcg-accel-ops.c
> index 1433e38f40..3c19e68a79 100644
> --- a/accel/tcg/tcg-accel-ops.c
> +++ b/accel/tcg/tcg-accel-ops.c
> @@ -35,7 +35,7 @@
>   #include "exec/exec-all.h"
>   #include "exec/hwaddr.h"
>   #include "exec/tb-flush.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   
>   #include "hw/core/cpu.h"
>   
> diff --git a/gdbstub/user.c b/gdbstub/user.c
> index edeb72efeb..e34b58b407 100644
> --- a/gdbstub/user.c
> +++ b/gdbstub/user.c
> @@ -18,6 +18,7 @@
>   #include "exec/gdbstub.h"
>   #include "gdbstub/syscalls.h"
>   #include "gdbstub/user.h"
> +#include "gdbstub/enums.h"
>   #include "hw/core/cpu.h"
>   #include "trace.h"
>   #include "internals.h"
> diff --git a/monitor/hmp-cmds.c b/monitor/hmp-cmds.c
> index ea79148ee8..067152589b 100644
> --- a/monitor/hmp-cmds.c
> +++ b/monitor/hmp-cmds.c
> @@ -15,8 +15,9 @@
>   
>   #include "qemu/osdep.h"
>   #include "exec/address-spaces.h"
> -#include "exec/gdbstub.h"
>   #include "exec/ioport.h"
> +#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "monitor/hmp.h"
>   #include "qemu/help_option.h"
>   #include "monitor/monitor-internal.h"
> diff --git a/system/vl.c b/system/vl.c
> index a3eede5fa5..cfcb674425 100644
> --- a/system/vl.c
> +++ b/system/vl.c
> @@ -68,6 +68,7 @@
>   #include "sysemu/numa.h"
>   #include "sysemu/hostmem.h"
>   #include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "qemu/timer.h"
>   #include "chardev/char.h"
>   #include "qemu/bitmap.h"
> diff --git a/target/arm/hvf/hvf.c b/target/arm/hvf/hvf.c
> index 45e2218be5..ef9bc42738 100644
> --- a/target/arm/hvf/hvf.c
> +++ b/target/arm/hvf/hvf.c
> @@ -33,7 +33,7 @@
>   #include "trace/trace-target_arm_hvf.h"
>   #include "migration/vmstate.h"
>   
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   
>   #define MDSCR_EL1_SS_SHIFT  0
>   #define MDSCR_EL1_MDE_SHIFT 15
> diff --git a/target/arm/hyp_gdbstub.c b/target/arm/hyp_gdbstub.c
> index ebde2899cd..f120d55caa 100644
> --- a/target/arm/hyp_gdbstub.c
> +++ b/target/arm/hyp_gdbstub.c
> @@ -12,7 +12,7 @@
>   #include "qemu/osdep.h"
>   #include "cpu.h"
>   #include "internals.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   
>   /* Maximum and current break/watch point counts */
>   int max_hw_bps, max_hw_wps;
> diff --git a/target/arm/kvm.c b/target/arm/kvm.c
> index 7cf5cf31de..70f79eda33 100644
> --- a/target/arm/kvm.c
> +++ b/target/arm/kvm.c
> @@ -31,7 +31,7 @@
>   #include "hw/pci/pci.h"
>   #include "exec/memattrs.h"
>   #include "exec/address-spaces.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "hw/boards.h"
>   #include "hw/irq.h"
>   #include "qapi/visitor.h"
> diff --git a/target/i386/kvm/kvm.c b/target/i386/kvm/kvm.c
> index 912f5d5a6b..a666129f41 100644
> --- a/target/i386/kvm/kvm.c
> +++ b/target/i386/kvm/kvm.c
> @@ -38,7 +38,7 @@
>   #include "hyperv.h"
>   #include "hyperv-proto.h"
>   
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "qemu/host-utils.h"
>   #include "qemu/main-loop.h"
>   #include "qemu/ratelimit.h"
> diff --git a/target/ppc/kvm.c b/target/ppc/kvm.c
> index 005f2239f3..2c3932200b 100644
> --- a/target/ppc/kvm.c
> +++ b/target/ppc/kvm.c
> @@ -39,7 +39,7 @@
>   #include "migration/qemu-file-types.h"
>   #include "sysemu/watchdog.h"
>   #include "trace.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "exec/memattrs.h"
>   #include "exec/ram_addr.h"
>   #include "sysemu/hostmem.h"
> diff --git a/target/s390x/kvm/kvm.c b/target/s390x/kvm/kvm.c
> index 1b494ecc20..94181d9281 100644
> --- a/target/s390x/kvm/kvm.c
> +++ b/target/s390x/kvm/kvm.c
> @@ -40,7 +40,7 @@
>   #include "sysemu/hw_accel.h"
>   #include "sysemu/runstate.h"
>   #include "sysemu/device_tree.h"
> -#include "exec/gdbstub.h"
> +#include "gdbstub/enums.h"
>   #include "exec/ram_addr.h"
>   #include "trace.h"
>   #include "hw/s390x/s390-pci-inst.h"

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>