[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V3 1/4] net/colo-compare.c: Add checkpoint min perio
From: |
Zhang Chen |
Subject: |
[Qemu-devel] [PATCH V3 1/4] net/colo-compare.c: Add checkpoint min period to optimize performance |
Date: |
Wed, 19 Jul 2017 15:29:56 +0800 |
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 <address@hidden>
---
net/colo-compare.c | 21 ++++++++++++++++++++-
1 file changed, 20 insertions(+), 1 deletion(-)
diff --git a/net/colo-compare.c b/net/colo-compare.c
index ca67c68..5313e74 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 ++
| |
@@ -75,6 +78,9 @@ typedef struct CompareState {
SocketReadState sec_rs;
bool vnet_hdr;
+ /* Record the last checkpoint time */
+ int64_t checkpoint_time_ms;
+
/* connection list: the connections belonged to this NIC could be found
* in this list.
* element type: Connection
@@ -507,7 +513,19 @@ 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 - s->checkpoint_time_ms >
+ CHECKPOINT_MIN_TIME) {
+ /*
+ * TODO: Notify colo frame to do checkpoint.
+ * colo_compare_inconsistent_notify();
+ *
+ * TODO: Reset s->checkpoint_time_ms after finish
+ * checkpoint(when colo-compare get notify from colo-frame,
+ * in another independent patch).
+ */
+ s->checkpoint_time_ms = pkt->creation_ms;
+ }
break;
}
}
@@ -803,6 +821,7 @@ static void colo_compare_complete(UserCreatable *uc, Error
**errp)
colo_compare_thread, s,
QEMU_THREAD_JOINABLE);
compare_id++;
+ s->checkpoint_time_ms = 0;
return;
}
--
2.7.4