Re: [PATCH-for-8.0 0/2] target/arm/gdbstub: Fix builds when TCG is disabled

[Philippe Mathieu-Daudé]

On 28/3/23 15:20, Philippe Mathieu-Daudé wrote:
> On 28/3/23 11:54, Peter Maydell wrote:
> > On Wed, 22 Mar 2023 at 14:29, Philippe Mathieu-Daudé 
> > <philmd@linaro.org> wrote:
> > > Fix when building QEMU configured with --disable-tcg:
> > >
> > >    Undefined symbols for architecture arm64:
> > >      "_arm_v7m_get_sp_ptr", referenced from:
> > >          _m_sysreg_get in target_arm_gdbstub.c.o
> > >      "_arm_v7m_mrs_control", referenced from:
> > >          _arm_gdb_get_m_systemreg in target_arm_gdbstub.c.o
> > >      "_pauth_ptr_mask", referenced from:
> > >          _aarch64_gdb_get_pauth_reg in target_arm_gdbstub64.c.o
> > >    ld: symbol(s) not found for architecture arm64
> > >    clang: error: linker command failed with exit code 1 (use -v to 
> > > see invocation)
> > >
> > > Philippe Mathieu-Daudé (2):
> > >    target/arm/gdbstub: Restrict aarch64_gdb_get_pauth_reg() to 
> > > CONFIG_TCG
> > >    target/arm/gdbstub: Only advertise M-profile features if TCG 
> > > available
> >
> > I've applied patch 2 to target-arm.next; thanks.
>
> If you only take #2, then you need to squash this from #1:
>
> -- >8 --
> diff --git a/target/arm/gdbstub.c b/target/arm/gdbstub.c
> @@ -21,6 +21,7 @@
>   #include "cpu.h"
>   #include "exec/gdbstub.h"
>   #include "gdbstub/helpers.h"
> +#include "sysemu/tcg.h"
>   #include "internals.h"
>   #include "cpregs.h"
> ---
>
> I can respin if it eases your workflow.

Posted as:
20230328133054.6553-2-philmd@linaro.org/">https://lore.kernel.org/qemu-devel/20230328133054.6553-2-philmd@linaro.org/

Along with rth's suggestion as another patch.