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

[Frank Chang]

Reviewed-by: Frank Chang <frank.chang@sifive.com>

On Thu, Dec 15, 2022 at 11:09 PM Elta <503386372@qq.com> wrote:

Should be cpu->cfg.elen in range [8, 64].

Signed-off-by: Dongxue Zhang <elta.era@gmail.com>

---

 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 d14e95c9dc..1e8032c969 100644

--- a/target/riscv/cpu.c

+++ b/target/riscv/cpu.c

@@ -870,7 +870,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.17.1