On Mon, Oct 27, 2025 at 04:41:11PM +0100, Juraj Marcin wrote:
[...]
> @@ -1752,6 +1756,7 @@ bool migration_in_postcopy(void)
> MigrationState *s = migrate_get_current();
>
> switch (s->state) {
> + case MIGRATION_STATUS_POSTCOPY_DEVICE:
> case MIGRATION_STATUS_POSTCOPY_ACTIVE:
> case MIGRATION_STATUS_POSTCOPY_PAUSED:
> case MIGRATION_STATUS_POSTCOPY_RECOVER_SETUP:
> @@ -1853,6 +1858,8 @@ int migrate_init(MigrationState *s, Error **errp)
> memset(&mig_stats, 0, sizeof(mig_stats));
> migration_reset_vfio_bytes_transferred();
>
> + s->postcopy_package_loaded = false;
Better reset postcopy_package_loaded_event (in case of continuous
postcopy-device failures)?
Other than that, looks all good, thanks!
Reviewed-by: Peter Xu <[email protected]>
> +
> return 0;
> }
--
Peter Xu