On Sat, Apr 5, 2025 at 1:29 AM Daniel Henrique Barboza
<dbarb...@ventanamicro.com> wrote:
>
> The 'max' CPU includes all available extensions we implement, but at
> this moment it is not rva23s64 compliant due to missing checks that
> the parent profile (rva22s64) does.
>
> Users might expect that the a CPU called 'max' CPU will also compatible
> with our latest S mode profile. Let's make it official.
>
> Signed-off-by: Daniel Henrique Barboza <dbarb...@ventanamicro.com>

Reviewed-by: Alistair Francis <alistair.fran...@wdc.com>

Alistair

> ---
>  target/riscv/tcg/tcg-cpu.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/target/riscv/tcg/tcg-cpu.c b/target/riscv/tcg/tcg-cpu.c
> index 5aef9eef36..cd489ae35b 100644
> --- a/target/riscv/tcg/tcg-cpu.c
> +++ b/target/riscv/tcg/tcg-cpu.c
> @@ -1486,6 +1486,8 @@ static void riscv_init_max_cpu_extensions(Object *obj)
>      if (cpu->cfg.ext_smdbltrp) {
>          isa_ext_update_enabled(cpu, CPU_CFG_OFFSET(ext_smdbltrp), false);
>      }
> +
> +    object_property_set_bool(obj, "rva23s64", true, NULL);
>  }
>
>  static bool riscv_cpu_has_max_extensions(Object *cpu_obj)
> --
> 2.49.0
>
>

Reply via email to