[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v3 3/4] target/arm: Add the XML dynamic generation
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH v3 3/4] target/arm: Add the XML dynamic generation |
Date: |
Thu, 1 Mar 2018 16:46:47 +0000 |
On 28 February 2018 at 11:01, Abdallah Bouassida
<address@hidden> wrote:
> Generate an XML description for the cp-regs.
> Register these regs with the gdb_register_coprocessor().
> Add arm_gdb_get_sysreg() to use it as a callback to read those regs.
>
> Signed-off-by: Abdallah Bouassida <address@hidden>
> ---
> +void arm_register_gdb_regs_for_features(CPUState *cs)
> +{
> + int n;
> +
> + n = arm_gen_dynamic_xml(cs);
> + gdb_register_coprocessor(cs, arm_gdb_get_sysreg, NULL,
> + n, "system-registers.xml", 0);
> +
> +}
Have you tried writing to a sysreg from the debugger without
your patch 4 applied? I suspect you'll find it crashes because
of the NULL pointer you're passing here, and you'll need a
dummy write function.
thanks
-- PMM