mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-02 07:13:54 -06:00
target/arm: Handle AArch64 in TYPE_ARM_CPU gdb_arch_name
Instead of having the TYPE_AARCH64_CPU subclass set CPUClass::gdb_arch_name to a different function, make the TYPE_ARM_CPU implementation of the method handle AArch64. For the moment we make the "is this AArch64?" function test "is the CPU of TYPE_AARCH64_CPU?", so that this produces no behavioural change. When we've moved all the gdbstub related methods across to the base class, we will be able to change this to be "does the CPU have the ARM_FEATURE_AARCH64 feature?". Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Alex Bennée <alex.bennee@linaro.org> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> Message-ID: <20250317142819.900029-4-peter.maydell@linaro.org> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
This commit is contained in:
parent
a1f728ecc9
commit
bae9fb4b01
3 changed files with 9 additions and 6 deletions
|
@ -2649,6 +2649,9 @@ static const gchar *arm_gdb_arch_name(CPUState *cs)
|
|||
ARMCPU *cpu = ARM_CPU(cs);
|
||||
CPUARMState *env = &cpu->env;
|
||||
|
||||
if (arm_gdbstub_is_aarch64(cpu)) {
|
||||
return "aarch64";
|
||||
}
|
||||
if (arm_feature(env, ARM_FEATURE_IWMMXT)) {
|
||||
return "iwmmxt";
|
||||
}
|
||||
|
|
|
@ -813,11 +813,6 @@ static void aarch64_cpu_finalizefn(Object *obj)
|
|||
{
|
||||
}
|
||||
|
||||
static const gchar *aarch64_gdb_arch_name(CPUState *cs)
|
||||
{
|
||||
return "aarch64";
|
||||
}
|
||||
|
||||
static void aarch64_cpu_class_init(ObjectClass *oc, const void *data)
|
||||
{
|
||||
CPUClass *cc = CPU_CLASS(oc);
|
||||
|
@ -825,7 +820,6 @@ static void aarch64_cpu_class_init(ObjectClass *oc, const void *data)
|
|||
cc->gdb_read_register = aarch64_cpu_gdb_read_register;
|
||||
cc->gdb_write_register = aarch64_cpu_gdb_write_register;
|
||||
cc->gdb_core_xml_file = "aarch64-core.xml";
|
||||
cc->gdb_arch_name = aarch64_gdb_arch_name;
|
||||
|
||||
object_class_property_add_bool(oc, "aarch64", aarch64_cpu_get_aarch64,
|
||||
aarch64_cpu_set_aarch64);
|
||||
|
|
|
@ -1826,6 +1826,12 @@ void aarch64_add_pauth_properties(Object *obj);
|
|||
void aarch64_add_sve_properties(Object *obj);
|
||||
void aarch64_add_sme_properties(Object *obj);
|
||||
|
||||
/* Return true if the gdbstub is presenting an AArch64 CPU */
|
||||
static inline bool arm_gdbstub_is_aarch64(ARMCPU *cpu)
|
||||
{
|
||||
return object_dynamic_cast(OBJECT(cpu), TYPE_AARCH64_CPU);
|
||||
}
|
||||
|
||||
/* Read the CONTROL register as the MRS instruction would. */
|
||||
uint32_t arm_v7m_mrs_control(CPUARMState *env, uint32_t secure);
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue