On 06/27/2011 08:07 PM, Peter Maydell wrote:
>  +    int ioport_base;
>  +    uint32_t ioport_reg[2];

I think ioport_reg[] needs to go in the VMStateDescription as well.
I don't know enough about the PCI subsystem to know whether that's
also true of ioport_base or whether the the map function is called
again on a vmload.

ioport_reg[1] is never written nor read, so this can be changed to a single uint32_t ioport_addr. Since it will be almost always 0 (either because it was never written, or because E1000_CTRL == 0), a subsection is useful to have.

Paolo

Reply via email to