[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 11/11] target/arm: generate xml description of our SVE re
From: |
Richard Henderson |
Subject: |
Re: [RFC PATCH 11/11] target/arm: generate xml description of our SVE registers |
Date: |
Mon, 18 Nov 2019 09:46:16 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 11/15/19 6:30 PM, Alex Bennée wrote:
> +int arm_gen_dynamic_svereg_xml(CPUState *cs, int base_reg)
> +{
> + ARMCPU *cpu = ARM_CPU(cs);
> + GString *s = g_string_new(NULL);
> + DynamicGDBXMLInfo *info = &cpu->dyn_svereg_xml;
> + g_autoptr(GString) ts = g_string_new("");
> + g_autoptr(GString) us = g_string_new("");
> + int i, j;
> + info->num = 0;
> + g_string_printf(s, "<?xml version=\"1.0\"?>");
> + g_string_append_printf(s, "<!DOCTYPE target SYSTEM \"gdb-target.dtd\">");
> + g_string_append_printf(s, "<feature name=\"org.qemu.gdb.aarch64.sve\">");
> + /* first define types and the union they belong to */
> + for (i = 0; i < ARRAY_SIZE(vec_lanes); i++) {
> + int count = 128 / vec_lanes[i].size;
> + g_string_printf(ts, "vq%d%s", count, vec_lanes[i].suffix);
> + g_string_append_printf(s, "<vector id=\"%s\" type=\"%s\"
> count=\"%d\"/>",
> + ts->str, vec_lanes[i].gdb_type, count);
> + g_string_append_printf(us, "<field name=\"%s\" type=\"%s\"/>",
> + vec_lanes[i].suffix, ts->str);
> + }
Really? Separate 128-bit registers for each Zreg lane?
Surely that's not what gdb does...
r~
- [RFC PATCH 02/11] gdbstub: stop passing GDBState * around, (continued)
- [RFC PATCH 10/11] target/arm: explicitly encode regnum in our XML, Alex Bennée, 2019/11/15
- [RFC PATCH 09/11] target/arm: prepare for multiple dynamic XMLs, Alex Bennée, 2019/11/15
- [RFC PATCH 11/11] target/arm: generate xml description of our SVE registers, Alex Bennée, 2019/11/15
- Re: [RFC PATCH 11/11] target/arm: generate xml description of our SVE registers,
Richard Henderson <=
- [RFC PATCH 08/11] gdbstub: extend GByteArray to read register helpers, Alex Bennée, 2019/11/15