Il ven 13 dic 2024, 10:01 Zhao Liu <zhao1....@intel.com> ha scritto:

> I found vmstate_array_of_pointer_to_struct missed a `info` field, and I
> could submit a patch to fix this nit next week (along with other cleanup
> you and other miantainers suggested for HPET).
>

I wouldn't worry too much about VMState, it's not in its final form and
anything that isn't needed to pass qtests can wait.

Paolo


> for which my idea is to implement a trait on
> > types to retrieve a basic VMStateField (for example something like
> > vmstate_uint32 would become an implementation of the VMState trait on
> u32).
>
> This makes sense.
>
> > Then you'd write something like "vmstate_of!(Type,
> > field).with_version_id(2)" (i.e. vmstate_of retrieves the basic field and
> > fills in the offset, then you apply more changes on top).  But that may
> take
> > a while, and I think it cannot be done without the const_refs_to_static
> > feature, which is only stable in 1.83.0.
>
> I also like this idea!
>
> Thanks,
> Zhao
>
>
>

Reply via email to