> > > 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