[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Re: [PATCH qemu] target/riscv/cpu.c: Fix elen check
From: |
Dongxue Zhang |
Subject: |
Re: Re: [PATCH qemu] target/riscv/cpu.c: Fix elen check |
Date: |
Fri, 30 Dec 2022 10:06:05 +0800 |
>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
>
Ok, i will try again, and add users as reviewed-by and signed-off-by.
>> ---
>> 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
>>