On 07/14/2017 11:22 AM, Jason Wang wrote:
On 2017年07月13日 13:52, Zhang Chen wrote:
If colo-compare find out the first different packet that means
the following packet almost is different. we needn't do a lot
of checkpoint in this time, so we set the no-need-checkpoint
peroid, default just set 3 second.
Signed-off-by: Zhang Chen <zhangchen.f...@cn.fujitsu.com>
---
net/colo-compare.c | 13 ++++++++++++-
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/net/colo-compare.c b/net/colo-compare.c
index 6d500e1..0f8e198 100644
--- a/net/colo-compare.c
+++ b/net/colo-compare.c
@@ -40,6 +40,9 @@
/* TODO: Should be configurable */
#define REGULAR_PACKET_CHECK_MS 3000
+/* TODO: Should be configurable */
+#define CHECKPOINT_MIN_TIME 3000
+
/*
+ CompareState ++
| |
@@ -455,6 +458,7 @@ static void colo_compare_connection(void *opaque,
void *user_data)
Packet *pkt = NULL;
GList *result = NULL;
int ret;
+ static int64_t checkpoint_time_ms;
Let's avoid static variable here since we support more than one
compare instance.
OK, I will add the "checkpoint_time_ms" to CompareState.
Thanks
Zhang Chen
Thanks
while (!g_queue_is_empty(&conn->primary_list) &&
!g_queue_is_empty(&conn->secondary_list)) {
@@ -494,7 +498,14 @@ static void colo_compare_connection(void
*opaque, void *user_data)
*/
trace_colo_compare_main("packet different");
g_queue_push_tail(&conn->primary_list, pkt);
- /* TODO: colo_notify_checkpoint();*/
+
+ if (pkt->creation_ms - checkpoint_time_ms >
CHECKPOINT_MIN_TIME) {
+ /*
+ * TODO: Notify colo frame to do checkpoint.
+ * colo_compare_inconsistent_notify();
+ */
+ checkpoint_time_ms = pkt->creation_ms;
+ }
break;
}
}
.
--
Thanks
Zhang Chen