[Qemu-devel] Re: [PATCH 00/22] Refactor and cleaup migration code

[Paolo Bonzini]

On 02/23/2011 01:44 AM, Juan Quintela wrote:
> This series:
> - Fold MigrationState into FdMigrationState (and then rename)
> - Factorize migration statec creation in a single place
> - Make use of MIG_STATE_*, setup through helpers and make them local
> - remove relase&  cancel callbacks (where used only one in same
>    file than defined)
> - get_status() is no more, just access directly to .state
> - current_migration use cleanup, and make variable static
> - max_throotle is gone, now inside current_migration
> - change get_migration_status() to migration_has_finished()
>    and actualize single user.
>
> Please review.

Some changes are a matter of taste, but overall looks very nice!

Reviewed-by: Paolo Bonzini <address@hidden>

Paolo