> > > void qemu_spice_display_init_done(void)
> > > {
> > > - spice_display_init_done = true;
> > > + qemu_add_vm_change_state_handler(vm_change_state_handler, NULL);
> > > vm_change_state_handler(NULL, runstate_is_running(), runstate_get());
> >
> > I'd just call qemu_spice_display_start() directly here, the need for
> > runstate_get() goes away then. Otherwise looks good to me.
>
> Hmm, that could work, but it will behave differently as we will start
> spice even if the VM is not running then.
if (runstate_is_running())
qemu_spice_display_start()
take care,
Gerd
