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


Reply via email to