From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=36507 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ps2qW-0002lr-3N for qemu-devel@nongnu.org; Tue, 22 Feb 2011 19:44:49 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Ps2qU-0005GL-R5 for qemu-devel@nongnu.org; Tue, 22 Feb 2011 19:44:47 -0500 Received: from mx1.redhat.com ([209.132.183.28]:5787) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Ps2qU-0005EP-Iu for qemu-devel@nongnu.org; Tue, 22 Feb 2011 19:44:46 -0500 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p1N0ihZP025880 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 22 Feb 2011 19:44:44 -0500 From: Juan Quintela Date: Wed, 23 Feb 2011 01:44:14 +0100 Message-Id: Subject: [Qemu-devel] [PATCH 00/22] Refactor and cleaup migration code List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org 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. Later, Juan. Juan Quintela (22): migration: Make *start_outgoing_migration return FdMigrationState migration: Use FdMigrationState instead of MigrationState when possible migration: Fold MigrationState into FdMigrationState migration: Rename FdMigrationState MigrationState migration: Refactor MigrationState creation migration: Make all posible migration functions static migration: move migrate_create_state to do_migrate migration: Check that migration is active before cancel it migration: Introduce MIG_STATE_NONE migration: Refactor and simplify error checking in migrate_fd_put_ready migration: Introduce migrate_fd_completed() for consistenncy migration: Use migrate_fd_error() in last place that set status to ERROR migration: Our release callback was just free migration: Remove get_status() accessor migration: Remove migration cancel() callback migration: Move exported functions to the end of the file migration: use global variable directly migration: another case of global variable assigned to local one migration: convert current_migration from pointer to struct migration: Use bandwidth_limit directly migration: Export a function that tells if the migration has finished correctly migration: Make state definitions local migration-exec.c | 39 +---- migration-fd.c | 42 ++----- migration-tcp.c | 41 ++---- migration-unix.c | 40 ++---- migration.c | 399 ++++++++++++++++++++++++++---------------------------- migration.h | 85 ++---------- ui/spice-core.c | 4 +- 7 files changed, 238 insertions(+), 412 deletions(-) -- 1.7.4