On 5/18/21 9:33 AM, Zhang, Chen wrote:
-----Original Message-----
From: Lukas Straub <lukasstra...@web.de>
Sent: Tuesday, May 18, 2021 3:47 AM
To: Rao, Lei <lei....@intel.com>
Cc: Zhang, Chen <chen.zh...@intel.com>; lizhij...@cn.fujitsu.com;
jasow...@redhat.com; quint...@redhat.com; dgilb...@redhat.com;
pbonz...@redhat.com; qemu-devel@nongnu.org
Subject: Re: [PATCH v6 00/10] Fixed some bugs and optimized some codes
for COLO
On Thu, 8 Apr 2021 23:20:46 -0400
leirao <lei....@intel.com> wrote:
From: Rao, Lei <lei....@intel.com>
Changes since v5:
--Replaced g_slice_new calls with g_slice_new0.
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 | 25 +++++++++++-----
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, 129 insertions(+), 37 deletions(-)
I guess Zhang Chen can already take the network-related patches through his
tree.
Sure, I will queue the patch 1-6 and 10.
Hi Dave, can you take the patch 7-9? Or do you think it's more convenient to
take all series?
Hi David,
The 1-6 and 10 already been merged in upstream.
Can you please pick up the rest of this series?
By the way, If you have time, Lei submitted a new series of patch still
need your comments,
Thanks
Chen
Thanks
Chen
Regards,
Lukas Straub
--
