On Fri, Jan 12, 2024 at 07:05:03AM -0800, Steve Sistare wrote:
> bool migration_in_incoming_postcopy(void)
> diff --git a/ui/spice-core.c b/ui/spice-core.c
> index b3cd229..e43a93f 100644
> --- a/ui/spice-core.c
> +++ b/ui/spice-core.c
> @@ -580,7 +580,7 @@ static int migration_state_notifier(NotifierWithReturn
> *notifier,
> if (migration_in_setup(s)) {
> spice_server_migrate_start(spice_server);
> } else if (migration_has_finished(s) ||
> - migration_in_postcopy_after_devices(s)) {
> + migration_in_postcopy_after_devices()) {
This can be a reply also to your other email: my previous suggestion of
using PRECOPY_DONE should apply here, where we can convert this chunk into:
} else if (event == MIG_EVENT_PRECOPY_DONE) {...}
Because PRECOPY_DONE should also cover the notification from
postcopy_start(), then we can drop migration_in_postcopy_after_devices()
completely, I think.
--
Peter Xu
