[PATCH v2 09/50] gdbstub: Always infer gdb_num_core_regs when using XML file

[Philippe Mathieu-Daudé]

Rather correct inferred count over invalid manual one.

Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
---
 include/hw/core/cpu.h | 4 ++--
 gdbstub/gdbstub.c     | 3 +--
 target/alpha/cpu.c    | 1 -
 target/ppc/cpu_init.c | 2 --
 target/sparc/cpu.c    | 1 -
 5 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h
index ef20cb356a6..4bdc9d62a4d 100644
--- a/include/hw/core/cpu.h
+++ b/include/hw/core/cpu.h
@@ -128,9 +128,9 @@ struct SysemuCPUOps;
  * @gdb_adjust_breakpoint: Callback for adjusting the address of a
  *       breakpoint.  Used by AVR to handle a gdb mis-feature with
  *       its Harvard architecture split code and data.
- * @gdb_num_core_regs: Number of core registers accessible to GDB or 0 to infer
- *                     from @gdb_core_xml_file.
  * @gdb_core_xml_file: File name for core registers GDB XML description.
+ * @gdb_num_core_regs: Number of core registers accessible to GDB if no
+ *                     @gdb_core_xml_file available (otherwise inferred).
  * @gdb_get_core_xml_file: Optional callback that returns the file name for
  * the core registers GDB XML description. The returned value is expected to
  * be a simple constant string: the caller will not g_free() it. If this
diff --git a/gdbstub/gdbstub.c b/gdbstub/gdbstub.c
index 474f6712f8d..90f4b95135b 100644
--- a/gdbstub/gdbstub.c
+++ b/gdbstub/gdbstub.c
@@ -593,8 +593,7 @@ void gdb_init_cpu(CPUState *cpu)
         gdb_register_feature(cpu, 0,
                              cc->gdb_read_register, cc->gdb_write_register,
                              feature);
-        assert(!cc->gdb_num_core_regs
-               || cc->gdb_num_core_regs == feature->num_regs);
+        assert(!cc->gdb_num_core_regs);
         cpu->gdb_num_regs = cpu->gdb_num_g_regs = feature->num_regs;
     }
 
diff --git a/target/alpha/cpu.c b/target/alpha/cpu.c
index 1780db7d1e2..e0e13d31e55 100644
--- a/target/alpha/cpu.c
+++ b/target/alpha/cpu.c
@@ -295,7 +295,6 @@ static void alpha_cpu_class_init(ObjectClass *oc, const 
void *data)
     cc->disas_set_info = alpha_cpu_disas_set_info;
 
     cc->tcg_ops = &alpha_tcg_ops;
-    cc->gdb_num_core_regs = 67;
 }
 
 #define DEFINE_ALPHA_CPU_TYPE(base_type, cpu_model, initfn) \
diff --git a/target/ppc/cpu_init.c b/target/ppc/cpu_init.c
index 0b5610d02d5..3d932a5642b 100644
--- a/target/ppc/cpu_init.c
+++ b/target/ppc/cpu_init.c
@@ -7551,8 +7551,6 @@ static void ppc_cpu_class_init(ObjectClass *oc, const 
void *data)
                       MMU_INST_FETCH == 2 && PAGE_READ == 1 &&
                       PAGE_WRITE == 2 && PAGE_EXEC == 4);
 #endif
-
-    cc->gdb_num_core_regs = 70;
     cc->gdb_arch_name = ppc_gdb_arch_name;
 #if defined(TARGET_PPC64)
     cc->gdb_core_xml_file = "power64-core.xml";
diff --git a/target/sparc/cpu.c b/target/sparc/cpu.c
index 3991681d1d1..c11d600dc30 100644
--- a/target/sparc/cpu.c
+++ b/target/sparc/cpu.c
@@ -1092,7 +1092,6 @@ static void sparc_cpu_class_init(ObjectClass *oc, const 
void *data)
 
 #if defined(TARGET_SPARC64) && !defined(TARGET_ABI32)
     cc->gdb_core_xml_file = "sparc64-core.xml";
-    cc->gdb_num_core_regs = 86;
 #else
     cc->gdb_num_core_regs = 72;
 #endif
-- 
2.52.0