* Samuel Thibault (samuel.thiba...@gnu.org) wrote:
> Hello,
> 
> Dr. David Alan Gilbert (git), on Thu 27 Oct 2016 16:32:17 +0100, wrote:
> > This converts the remaining components, except for the loop, to VMState.
> > 
> > +        /* TODO: Add loop */
> > +        VMSTATE_UINT16_V(ip_id, Slirp, 2),
> > +        VMSTATE_STRUCT_ARRAY(bootp_clients, Slirp, NB_BOOTP_CLIENTS, 3,
> > +                             vmstate_slirp_bootp_client, BOOTPClient),
> 
> I don't understand: doesn't the NB_BOOTP_CLIENTS parameter here
> implement the loop?

That implements one loop; the TODO is to add the loop over slirp->exec_list
that's at the top of slirp_state_save;  once we find a way of adding that
loop then the whole of slirp_state_save/load disappears and we're left
with just the VMStateDescription.

Dave

> Samuel
--
Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK

Reply via email to