[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 00/11] migration: cleanup migration_thread()
From: |
Peter Xu |
Subject: |
[Qemu-devel] [PATCH 00/11] migration: cleanup migration_thread() |
Date: |
Wed, 3 Jan 2018 13:40:32 +0800 |
Firstly this series is something as a first attempt of me to cleanup
some migration code. It may not be a good idea, but I still think it
worth a try, so I posted it. Let me know if any of you don't like it,
so I can stop. At least after the series the migration_thread()
function can be far shorter and much easier for first-time readers
AFAICT.
For this single function, the most complexity part is quite a lot of
local variables crossly referenced everywhere, and during the cleanup
I do think the COLO part is hacky too.
There can be some functional changes too in the future:
- cancel migration as cleanup of QEMU quit
- add migration locks to protect migration internal states
- ...
But for this single series, there is still no functional change yet.
Please have a look. Any feedback is welcomed. Thanks,
Peter Xu (11):
migration: assert colo instead of check
migration: qemu_savevm_state_cleanup() in cleanup
migration: remove "enable_colo" var
migration: split use of MigrationState.total_time
migration: move vm_old_running into global state
migration: introduce vm_down_start_time
migration: introduce migrate_calculate_complete
migration: use switch at the end of migration
migration: cleanup stats update into function
migration: major cleanup for migrate iterations
migration: put the finish part into a new function
migration/migration.c | 289 ++++++++++++++++++++++++++++----------------------
migration/migration.h | 26 ++++-
2 files changed, 187 insertions(+), 128 deletions(-)
--
2.14.3
- [Qemu-devel] [PATCH 00/11] migration: cleanup migration_thread(),
Peter Xu <=
- [Qemu-devel] [PATCH 01/11] migration: assert colo instead of check, Peter Xu, 2018/01/03
- [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Juan Quintela, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Juan Quintela, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Juan Quintela, 2018/01/03
- Re: [Qemu-devel] [PATCH 02/11] migration: qemu_savevm_state_cleanup() in cleanup, Peter Xu, 2018/01/03