[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v11 5/6] migration/dirtyrate: move init step of calculation to ma
From: |
huangy81 |
Subject: |
[PATCH v11 5/6] migration/dirtyrate: move init step of calculation to main thread |
Date: |
Tue, 29 Jun 2021 21:39:17 +0800 |
From: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
since main thread may "query dirty rate" at any time, it's better
to move init step into main thead so that synchronization overhead
between "main" and "get_dirtyrate" can be reduced.
Signed-off-by: Hyman Huang(黄勇) <huangy81@chinatelecom.cn>
Message-Id:
<109f8077518ed2f13068e3bfb10e625e964780f1.1624040308.git.huangy81@chinatelecom.cn>
Reviewed-by: Peter Xu <peterx@redhat.com>
---
migration/dirtyrate.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/migration/dirtyrate.c b/migration/dirtyrate.c
index a9bdd60..b8f61cc 100644
--- a/migration/dirtyrate.c
+++ b/migration/dirtyrate.c
@@ -380,7 +380,6 @@ void *get_dirtyrate_thread(void *arg)
{
struct DirtyRateConfig config = *(struct DirtyRateConfig *)arg;
int ret;
- int64_t start_time;
rcu_register_thread();
ret = dirtyrate_set_state(&CalculatingState, DIRTY_RATE_STATUS_UNSTARTED,
@@ -390,9 +389,6 @@ void *get_dirtyrate_thread(void *arg)
return NULL;
}
- start_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME) / 1000;
- init_dirtyrate_stat(start_time, config);
-
calculate_dirtyrate(config);
ret = dirtyrate_set_state(&CalculatingState, DIRTY_RATE_STATUS_MEASURING,
@@ -411,6 +407,7 @@ void qmp_calc_dirty_rate(int64_t calc_time, bool
has_sample_pages,
static struct DirtyRateConfig config;
QemuThread thread;
int ret;
+ int64_t start_time;
/*
* If the dirty rate is already being measured, don't attempt to start.
@@ -451,6 +448,10 @@ void qmp_calc_dirty_rate(int64_t calc_time, bool
has_sample_pages,
config.sample_period_seconds = calc_time;
config.sample_pages_per_gigabytes = sample_pages;
config.mode = DIRTY_RATE_MEASURE_MODE_PAGE_SAMPLING;
+
+ start_time = qemu_clock_get_ms(QEMU_CLOCK_REALTIME) / 1000;
+ init_dirtyrate_stat(start_time, config);
+
qemu_thread_create(&thread, "get_dirtyrate", get_dirtyrate_thread,
(void *)&config, QEMU_THREAD_DETACHED);
}
--
1.8.3.1
- [PATCH v11 0/6] support dirtyrate at the granualrity of vcpu, huangy81, 2021/06/29
- [PATCH v11 1/6] KVM: introduce dirty_pages and kvm_dirty_ring_enabled, huangy81, 2021/06/29
- [PATCH v11 2/6] memory: make global_dirty_tracking a bitmask, huangy81, 2021/06/29
- [PATCH v11 3/6] migration/dirtyrate: introduce struct and adjust DirtyRateStat, huangy81, 2021/06/29
- [PATCH v11 5/6] migration/dirtyrate: move init step of calculation to main thread,
huangy81 <=
- [PATCH v11 4/6] migration/dirtyrate: adjust order of registering thread, huangy81, 2021/06/29
- [PATCH v11 6/6] migration/dirtyrate: implement dirty-ring dirtyrate calculation, huangy81, 2021/06/29