[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 00/10] Fixed some bugs and optimized some codes for COLO
From: |
Lukas Straub |
Subject: |
Re: [PATCH v5 00/10] Fixed some bugs and optimized some codes for COLO |
Date: |
Sun, 4 Apr 2021 12:22:11 +0200 |
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
--
pgp9HSRTmnM6q.pgp
Description: OpenPGP digital signature
- RE: [PATCH v5 04/10] Remove migrate_set_block_enabled in checkpoint, (continued)
- [PATCH v5 05/10] Add a function named packet_new_nocopy for COLO., leirao, 2021/04/01
- [PATCH v5 06/10] Add the function of colo_compare_cleanup, leirao, 2021/04/01
- [PATCH v5 07/10] Reset the auto-converge counter at every checkpoint., leirao, 2021/04/01
- [PATCH v5 08/10] Reduce the PVM stop time during Checkpoint, leirao, 2021/04/01
- [PATCH v5 09/10] Add the function of colo_bitmap_clear_dirty, leirao, 2021/04/01
- [PATCH v5 10/10] Fixed calculation error of pkt->header_size in fill_pkt_tcp_info(), leirao, 2021/04/01
- Re: [PATCH v5 00/10] Fixed some bugs and optimized some codes for COLO,
Lukas Straub <=