Re: [RFC PATCH 02/34] target/riscv: Fix size of trivial CPUArchState fields

[Alistair Francis]

On Thu, Sep 25, 2025 at 7:37 AM Richard Henderson
<richard.hender...@linaro.org> wrote:
>
> On 9/24/25 00:20, Anton Johansson via wrote:
> > +++ b/target/riscv/machine.c
> > @@ -84,13 +84,13 @@ static const VMStateDescription vmstate_hyper = {
> >       .minimum_version_id = 4,
> >       .needed = hyper_needed,
> >       .fields = (const VMStateField[]) {
> > -        VMSTATE_UINTTL(env.hstatus, RISCVCPU),
> > -        VMSTATE_UINTTL(env.hedeleg, RISCVCPU),
> > +        VMSTATE_UINT64(env.hstatus, RISCVCPU),
> > +        VMSTATE_UINT64(env.hedeleg, RISCVCPU),
>
> You can't change these sizes without bumping the version id.
>
> I don't know if riscv really cares about migration stability yet.
> If it does, then you have to jump through more hoops.

Not really, I think a version bump would be enough

Alistair

>
>
> r~
>