Re: [PATCH qemu] target/riscv/cpu.c: Fix elen check

[Alistair Francis]

On Thu, Dec 29, 2022 at 12:34 AM ~elta <elta@git.sr.ht> wrote:
>
> From: Dongxue Zhang <elta.era@gmail.com>
>
> Should be cpu->cfg.elen in range [8, 64].
>
> Signed-off-by: Dongxue Zhang <elta.era@gmail.com>

When sending a new version can you please add any previous reviewed tags.

Alistair

> ---
>  target/riscv/cpu.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/riscv/cpu.c b/target/riscv/cpu.c
> index 6fe176e483..5dc51f7912 100644
> --- a/target/riscv/cpu.c
> +++ b/target/riscv/cpu.c
> @@ -872,7 +872,7 @@ static void riscv_cpu_realize(DeviceState *dev, Error 
> **errp)
>                          "Vector extension ELEN must be power of 2");
>                  return;
>              }
> -            if (cpu->cfg.elen > 64 || cpu->cfg.vlen < 8) {
> +            if (cpu->cfg.elen > 64 || cpu->cfg.elen < 8) {
>                  error_setg(errp,
>                          "Vector extension implementation only supports ELEN "
>                          "in the range [8, 64]");
> --
> 2.34.5
>