Re: [PATCH v4 06/17] target/riscv: setup everything so that riscv128-softmmu compiles

[Richard Henderson]

On 10/25/21 5:28 AM, Frédéric Pétrot wrote:
> This patch is kind of a mess because several files have to be slightly
> modified to allow for a new target. In the current status, we have done
> our best to have RV64 and RV128 under the same RV64 umbrella, but there
> is still work to do to have a single executable for both.
>
> Once this patch applied, adding risc128-sofmmu to --target-list produces
> a (no so useful yet) executable.
>
> Signed-off-by: Frédéric Pétrot<frederic.petrot@univ-grenoble-alpes.fr>
> Co-authored-by: Fabien Portas<fabien.portas@grenoble-inp.org>
> ---
>   configs/devices/riscv128-softmmu/default.mak | 17 +++++++++++++++
>   configs/targets/riscv128-softmmu.mak         |  5 +++++
>   include/disas/dis-asm.h                      |  1 +
>   include/hw/riscv/sifive_cpu.h                |  3 +++
>   target/riscv/cpu-param.h                     |  5 +++++
>   target/riscv/cpu.h                           |  3 +++
>   disas/riscv.c                                |  5 +++++
>   target/riscv/cpu.c                           | 22 ++++++++++++++++++--
>   target/riscv/gdbstub.c                       |  8 +++++++
>   target/riscv/insn_trans/trans_rvd.c.inc      | 12 +++++------
>   target/riscv/insn_trans/trans_rvf.c.inc      |  6 +++---
>   target/riscv/Kconfig                         |  3 +++
>   12 files changed, 79 insertions(+), 11 deletions(-)
>   create mode 100644 configs/devices/riscv128-softmmu/default.mak
>   create mode 100644 configs/targets/riscv128-softmmu.mak

Can we start with adding -cpu rv128 to the existing riscv64-softmmu executable, before we 
go so far as to add yet another executable?


r~