From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47168) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yb0PQ-0008B9-EQ for qemu-devel@nongnu.org; Thu, 26 Mar 2015 01:32:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yb0PP-0007GV-Fx for qemu-devel@nongnu.org; Thu, 26 Mar 2015 01:32:48 -0400 Received: from szxga02-in.huawei.com ([119.145.14.65]:41928) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yb0PO-0007G3-NX for qemu-devel@nongnu.org; Thu, 26 Mar 2015 01:32:47 -0400 From: zhanghailiang Date: Thu, 26 Mar 2015 13:29:29 +0800 Message-ID: <1427347774-8960-24-git-send-email-zhang.zhanghailiang@huawei.com> In-Reply-To: <1427347774-8960-1-git-send-email-zhang.zhanghailiang@huawei.com> References: <1427347774-8960-1-git-send-email-zhang.zhanghailiang@huawei.com> MIME-Version: 1.0 Content-Type: text/plain Subject: [Qemu-devel] [RFC PATCH v4 23/28] COLO: Improve checkpoint efficiency by do additional periodic checkpoint List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: lizhijian@cn.fujitsu.com, quintela@redhat.com, yunhong.jiang@intel.com, eddie.dong@intel.com, peter.huangpeng@huawei.com, dgilbert@redhat.com, zhanghailiang , arei.gonglei@huawei.com, amit.shah@redhat.com, Yang Hongyang , david@gibson.dropbear.id.au Besides normal checkpoint which according to the result of net packets comparing, We do additional checkpoint periodically, it will reduce the number of dirty pages when do one checkpoint, if we don't do checkpoint for a long time (This is a special case when the net packets is always consistent). Signed-off-by: zhanghailiang Signed-off-by: Yang Hongyang --- migration/colo.c | 29 +++++++++++++++++++++-------- 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/migration/colo.c b/migration/colo.c index 9ef4554..da5bc5e 100644 --- a/migration/colo.c +++ b/migration/colo.c @@ -10,6 +10,7 @@ * later. See the COPYING file in the top-level directory. */ +#include "qemu/timer.h" #include "sysemu/sysemu.h" #include "migration/migration-colo.h" #include "qemu/error-report.h" @@ -32,6 +33,13 @@ */ #define CHECKPOINT_MIN_PERIOD 100 /* unit: ms */ +/* + * force checkpoint timer: unit ms + * this is large because COLO checkpoint will mostly depend on + * COLO compare module. + */ +#define CHECKPOINT_MAX_PEROID 10000 + enum { COLO_READY = 0x46, @@ -340,14 +348,7 @@ static void *colo_thread(void *opaque) proxy_checkpoint_req = colo_proxy_compare(); if (proxy_checkpoint_req < 0) { goto out; - } else if (!proxy_checkpoint_req) { - /* - * No checkpoint is needed, wait for 1ms and then - * check if we need checkpoint again - */ - g_usleep(1000); - continue; - } else { + } else if (proxy_checkpoint_req) { int64_t interval; current_time = qemu_clock_get_ms(QEMU_CLOCK_HOST); @@ -357,8 +358,20 @@ static void *colo_thread(void *opaque) g_usleep((1000*(CHECKPOINT_MIN_PERIOD - interval))); } DPRINTF("Net packets is not consistent!!!\n"); + goto do_checkpoint; + } + + /* + * No proxy checkpoint is request, wait for 100ms + * and then check if we need checkpoint again. + */ + current_time = qemu_clock_get_ms(QEMU_CLOCK_HOST); + if (current_time - checkpoint_time < CHECKPOINT_MAX_PEROID) { + g_usleep(100000); + continue; } +do_checkpoint: /* start a colo checkpoint */ if (colo_do_checkpoint_transaction(s, colo_control)) { goto out; -- 1.7.12.4