qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 1/4] net/colo-compare.c: Add checkpoint min peri


From: Li Zhijian
Subject: Re: [Qemu-devel] [PATCH 1/4] net/colo-compare.c: Add checkpoint min period to optimize performance
Date: Fri, 7 Jul 2017 14:06:13 +0800
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1


On 07/07/2017 01:43 PM, Zhang Chen wrote:
If colo-compare find out the first different packet

at this situation, why not trigger the checkpoint, and then flush all primary 
packets and drop secondary packets?


  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 <address@hidden>
---
  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;
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;
          }
      }

--
Best regards.
Li Zhijian (8528)






reply via email to

[Prev in Thread] Current Thread [Next in Thread]