[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH V2 1/5] target/riscv: Add basic vmstate description of CPU
From: |
Richard Henderson |
Subject: |
Re: [PATCH V2 1/5] target/riscv: Add basic vmstate description of CPU |
Date: |
Wed, 14 Oct 2020 08:45:26 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 |
On 10/14/20 3:21 AM, Jiangyifei wrote:
>> Would this be a good time to expand mstatus to uint64_t instead of
>> target_ulong so that it can be saved as one unit and reduce some ifdefs in
>> the
>> code base?
>>
>> Similarly with some of the other status registers that are two halved for
>> riscv32.
>
> I agree with you that it should be rearranged.
> But I hope this series will focus on achieving migration.
> Can I send another patch to rearrange it later?
Well, that changes the bit layout for migration.
While we could bump the version number, it seemed
easier to change the representation first.
r~
[PATCH V2 5/5] target/riscv: Add sifive_plic vmstate, Yifei Jiang, 2020/10/10
[PATCH V2 4/5] target/riscv: Add V extension state description, Yifei Jiang, 2020/10/10
[PATCH V2 3/5] target/riscv: Add H extension state description, Yifei Jiang, 2020/10/10
[PATCH V2 2/5] target/riscv: Add PMP state description, Yifei Jiang, 2020/10/10