Message ID | 5330460F.8020807@greensocs.com |
---|---|
State | New |
Headers | show |
Il 24/03/2014 15:49, Frederic Konrad ha scritto: > --- a/cpus.c > +++ b/cpus.c > @@ -427,6 +427,26 @@ void qemu_clock_warp(QEMUClockType type) > } > } > > +static bool icount_state_needed(void *opaque) > +{ > + return (use_icount != 0); > +} > + > +/* > + * This is a subsection for icount migration. > + */ > +static const VMStateDescription icount_vmstate_timers = { > + .name = "icount", > + .version_id = 2, 1 here. > + .minimum_version_id = 1, > + .minimum_version_id_old = 1, > + .fields = (VMStateField[]) { > + VMSTATE_INT64(qemu_icount_bias, TimersState), > + VMSTATE_INT64(qemu_icount, TimersState), > + VMSTATE_END_OF_LIST() > + } > +}; > + > static const VMStateDescription vmstate_timers = { > .name = "timer", > .version_id = 2, > @@ -437,6 +457,14 @@ static const VMStateDescription vmstate_timers = { > VMSTATE_INT64(dummy, TimersState), > VMSTATE_INT64_V(cpu_clock_offset, TimersState, 2), > VMSTATE_END_OF_LIST() > + }, > + .subsections = (VMStateSubsection[]) { > + { > + .vmsd = &icount_vmstate_timers, > + .needed = icount_state_needed, > + }, { > + /* empty */ > + } > } > }; > > Thanks, > Fred Yes, quite exactly that part from the version_id. If you can test it, this patch would be good for 2.0 too. Paolo
On 24/03/2014 16:42, Paolo Bonzini wrote: > Il 24/03/2014 15:49, Frederic Konrad ha scritto: >> --- a/cpus.c >> +++ b/cpus.c >> @@ -427,6 +427,26 @@ void qemu_clock_warp(QEMUClockType type) >> } >> } >> >> +static bool icount_state_needed(void *opaque) >> +{ >> + return (use_icount != 0); >> +} >> + >> +/* >> + * This is a subsection for icount migration. >> + */ >> +static const VMStateDescription icount_vmstate_timers = { >> + .name = "icount", >> + .version_id = 2, > > 1 here. > >> + .minimum_version_id = 1, >> + .minimum_version_id_old = 1, >> + .fields = (VMStateField[]) { >> + VMSTATE_INT64(qemu_icount_bias, TimersState), >> + VMSTATE_INT64(qemu_icount, TimersState), >> + VMSTATE_END_OF_LIST() >> + } >> +}; >> + >> static const VMStateDescription vmstate_timers = { >> .name = "timer", >> .version_id = 2, >> @@ -437,6 +457,14 @@ static const VMStateDescription vmstate_timers = { >> VMSTATE_INT64(dummy, TimersState), >> VMSTATE_INT64_V(cpu_clock_offset, TimersState, 2), >> VMSTATE_END_OF_LIST() >> + }, >> + .subsections = (VMStateSubsection[]) { >> + { >> + .vmsd = &icount_vmstate_timers, >> + .needed = icount_state_needed, >> + }, { >> + /* empty */ >> + } >> } >> }; >> >> Thanks, >> Fred > > Yes, quite exactly that part from the version_id. If you can test it, > this patch would be good for 2.0 too. > > Paolo > Seems there is a little issue somewhere: Unknown savevm section type 5 I'll try to debug this. Fred
--- a/cpus.c +++ b/cpus.c @@ -427,6 +427,26 @@ void qemu_clock_warp(QEMUClockType type) } } +static bool icount_state_needed(void *opaque) +{ + return (use_icount != 0); +} + +/* + * This is a subsection for icount migration. + */ +static const VMStateDescription icount_vmstate_timers = { + .name = "icount", + .version_id = 2, + .minimum_version_id = 1, + .minimum_version_id_old = 1, + .fields = (VMStateField[]) { + VMSTATE_INT64(qemu_icount_bias, TimersState), + VMSTATE_INT64(qemu_icount, TimersState), + VMSTATE_END_OF_LIST() + } +}; + static const VMStateDescription vmstate_timers = { .name = "timer", .version_id = 2, @@ -437,6 +457,14 @@ static const VMStateDescription vmstate_timers = { VMSTATE_INT64(dummy, TimersState), VMSTATE_INT64_V(cpu_clock_offset, TimersState, 2), VMSTATE_END_OF_LIST() + }, + .subsections = (VMStateSubsection[]) { + { + .vmsd = &icount_vmstate_timers, + .needed = icount_state_needed, + }, { + /* empty */ + } } };