Am 09.06.2011 um 17:35 schrieb Markus Armbruster:
Andreas Färber <andreas.faer...@web.de> writes:
Incorporate ISA VMState. Add "enabled" property.
Could you explain why you need to stick VMSTATE_ISA_DEVICE_V() into
vmstate_isa_serial, but not for the other devices?
I stuck it in all affected devices that have VMState. isa-parallel, as
pointed out elsewhere, currently does not.
What are you referring to?
static const VMStateDescription vmstate_isa_serial = {
.name = "serial",
- .version_id = 3,
+ .version_id = 4,
.minimum_version_id = 2,
.fields = (VMStateField []) {
+ VMSTATE_ISA_DEVICE_V(dev, ISASerialState, 4),
VMSTATE_STRUCT(state, ISASerialState, 0, vmstate_serial,
SerialState),
VMSTATE_END_OF_LIST()
}
If I understand vmstate correctly, this breaks migration new ->
old. Do
we care?
Are you referring to touching vmstate_isa_serial at all, or does the
placement of the new field make the difference here?
Andreas