| Message ID | 1617263249-54501-1-git-send-email-lei.rao@intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Fixed some bugs and optimized some codes for COLO | expand |
On Thu, 1 Apr 2021 15:47:19 +0800 leirao <lei.rao@intel.com> wrote: > From: Rao,Lei <lei.rao@intel.com> > > Changes since v4: > --Replaced qemu_mutex_lock calls with QEMU_LOCK_GUARD in colo_bitmap_clear_dirty. > --Modify some minor issues about variable definition. > --Add some performance test data in the commit message. > > Changes since v3: > --Remove cpu_throttle_stop from mig_throttle_counter_reset. > > Changes since v2: > --Add a function named packet_new_nocopy. > --Continue to optimize the function of colo_flush_ram_cache. > > Changes since v1: > --Reset the state of the auto-converge counters at every checkpoint instead of directly disabling. > --Treat the filter_send function returning zero as a normal case. > > The series of patches include: > Fixed some bugs of qemu crash. > Optimized some code to reduce the time of checkpoint. > Remove some unnecessary code to improve COLO. > > Rao, Lei (10): > Remove some duplicate trace code. > Fix the qemu crash when guest shutdown during checkpoint > Optimize the function of filter_send > Remove migrate_set_block_enabled in checkpoint > Add a function named packet_new_nocopy for COLO. > Add the function of colo_compare_cleanup > Reset the auto-converge counter at every checkpoint. > Reduce the PVM stop time during Checkpoint > Add the function of colo_bitmap_clear_dirty > Fixed calculation error of pkt->header_size in fill_pkt_tcp_info() > > migration/colo.c | 10 +++---- > migration/migration.c | 4 +++ > migration/ram.c | 83 +++++++++++++++++++++++++++++++++++++++++++++++++-- > migration/ram.h | 1 + > net/colo-compare.c | 25 +++++++--------- > net/colo-compare.h | 1 + > net/colo.c | 23 ++++++++++++++ > net/colo.h | 1 + > net/filter-mirror.c | 8 ++--- > net/filter-rewriter.c | 3 +- > net/net.c | 4 +++ > softmmu/runstate.c | 1 + > 12 files changed, 135 insertions(+), 29 deletions(-) > Looks good to me and works well in my tests: Reviewed-by: Lukas Straub <lukasstraub2@web.de> Tested-by: Lukas Straub <lukasstraub2@web.de> Regards, Lukas Straub --
From: Rao,Lei <lei.rao@intel.com> Changes since v4: --Replaced qemu_mutex_lock calls with QEMU_LOCK_GUARD in colo_bitmap_clear_dirty. --Modify some minor issues about variable definition. --Add some performance test data in the commit message. Changes since v3: --Remove cpu_throttle_stop from mig_throttle_counter_reset. Changes since v2: --Add a function named packet_new_nocopy. --Continue to optimize the function of colo_flush_ram_cache. Changes since v1: --Reset the state of the auto-converge counters at every checkpoint instead of directly disabling. --Treat the filter_send function returning zero as a normal case. The series of patches include: Fixed some bugs of qemu crash. Optimized some code to reduce the time of checkpoint. Remove some unnecessary code to improve COLO. Rao, Lei (10): Remove some duplicate trace code. Fix the qemu crash when guest shutdown during checkpoint Optimize the function of filter_send Remove migrate_set_block_enabled in checkpoint Add a function named packet_new_nocopy for COLO. Add the function of colo_compare_cleanup Reset the auto-converge counter at every checkpoint. Reduce the PVM stop time during Checkpoint Add the function of colo_bitmap_clear_dirty Fixed calculation error of pkt->header_size in fill_pkt_tcp_info() migration/colo.c | 10 +++---- migration/migration.c | 4 +++ migration/ram.c | 83 +++++++++++++++++++++++++++++++++++++++++++++++++-- migration/ram.h | 1 + net/colo-compare.c | 25 +++++++--------- net/colo-compare.h | 1 + net/colo.c | 23 ++++++++++++++ net/colo.h | 1 + net/filter-mirror.c | 8 ++--- net/filter-rewriter.c | 3 +- net/net.c | 4 +++ softmmu/runstate.c | 1 + 12 files changed, 135 insertions(+), 29 deletions(-)