[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] ram: Use MigrationStats for statistics
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] ram: Use MigrationStats for statistics |
Date: |
Mon, 5 Jun 2017 13:34:45 +0100 |
User-agent: |
Mutt/1.8.2 (2017-04-18) |
* Juan Quintela (address@hidden) wrote:
> RAM Statistics need to survive migration to make info migrate work, so we
> need to store them outside of RAMState. As we already have an struct
> with those fields, just used them. (MigrationStats and XBZRLECacheStats).
>
> Signed-off-by: Juan Quintela <address@hidden>
Hmm OK; this feels very much like it's the opposite of 180f61f from
March; these variables keep moving around over the last couple of months
- are they going to stay still now?
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
> ---
> migration/migration.c | 33 +++++-----
> migration/ram.c | 179
> ++++++++++++++------------------------------------
> migration/ram.h | 15 +----
> 3 files changed, 68 insertions(+), 159 deletions(-)
>
> diff --git a/migration/migration.c b/migration/migration.c
> index 2c13217..331cab7 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -480,28 +480,28 @@ static void populate_ram_info(MigrationInfo *info,
> MigrationState *s)
> {
> info->has_ram = true;
> info->ram = g_malloc0(sizeof(*info->ram));
> - info->ram->transferred = ram_bytes_transferred();
> + info->ram->transferred = ram_counters.transferred;
> info->ram->total = ram_bytes_total();
> - info->ram->duplicate = dup_mig_pages_transferred();
> + info->ram->duplicate = ram_counters.duplicate;
> /* legacy value. It is not used anymore */
> info->ram->skipped = 0;
> - info->ram->normal = norm_mig_pages_transferred();
> - info->ram->normal_bytes = norm_mig_pages_transferred() *
> + info->ram->normal = ram_counters.normal;
> + info->ram->normal_bytes = ram_counters.normal *
> qemu_target_page_size();
> info->ram->mbps = s->mbps;
> - info->ram->dirty_sync_count = ram_dirty_sync_count();
> - info->ram->postcopy_requests = ram_postcopy_requests();
> + info->ram->dirty_sync_count = ram_counters.dirty_sync_count;
> + info->ram->postcopy_requests = ram_counters.postcopy_requests;
> info->ram->page_size = qemu_target_page_size();
>
> if (migrate_use_xbzrle()) {
> info->has_xbzrle_cache = true;
> info->xbzrle_cache = g_malloc0(sizeof(*info->xbzrle_cache));
> info->xbzrle_cache->cache_size = migrate_xbzrle_cache_size();
> - info->xbzrle_cache->bytes = xbzrle_mig_bytes_transferred();
> - info->xbzrle_cache->pages = xbzrle_mig_pages_transferred();
> - info->xbzrle_cache->cache_miss = xbzrle_mig_pages_cache_miss();
> - info->xbzrle_cache->cache_miss_rate = xbzrle_mig_cache_miss_rate();
> - info->xbzrle_cache->overflow = xbzrle_mig_pages_overflow();
> + info->xbzrle_cache->bytes = xbzrle_counters.bytes;
> + info->xbzrle_cache->pages = xbzrle_counters.pages;
> + info->xbzrle_cache->cache_miss = xbzrle_counters.cache_miss;
> + info->xbzrle_cache->cache_miss_rate =
> xbzrle_counters.cache_miss_rate;
> + info->xbzrle_cache->overflow = xbzrle_counters.overflow;
> }
>
> if (cpu_throttle_active()) {
> @@ -518,10 +518,11 @@ static void populate_ram_info(MigrationInfo *info,
> MigrationState *s)
> }
>
> if (s->state != MIGRATION_STATUS_COMPLETED) {
> - info->ram->remaining_pages = ram_pages_remaining();
> - info->ram->remaining = ram_pages_remaining() *
> +
> + info->ram->remaining_pages = ram_counters.remaining_pages;
> + info->ram->remaining = ram_counters.remaining_pages *
> qemu_target_page_size();
> - info->ram->dirty_pages_rate = ram_dirty_pages_rate();
> + info->ram->dirty_pages_rate = ram_counters.dirty_pages_rate;
> }
> }
>
> @@ -1886,8 +1887,8 @@ static void *migration_thread(void *opaque)
> bandwidth, threshold_size);
> /* if we haven't sent anything, we don't want to recalculate
> 10000 is a small enough number for our purposes */
> - if (ram_dirty_pages_rate() && transferred_bytes > 10000) {
> - s->expected_downtime = ram_dirty_pages_rate() *
> + if (ram_counters.dirty_pages_rate && transferred_bytes > 10000) {
> + s->expected_downtime = ram_counters.dirty_pages_rate *
> qemu_target_page_size() / bandwidth;
> }
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 30519e1..6c48219 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -71,6 +71,8 @@ static inline bool is_zero_range(uint8_t *p, uint64_t size)
> return buffer_is_zero(p, size);
> }
>
> +XBZRLECacheStats xbzrle_counters;
> +
> /* struct contains XBZRLE cache and a static page
> used by the compression */
> static struct {
> @@ -174,8 +176,6 @@ struct RAMState {
> bool ram_bulk_stage;
> /* How many times we have dirty too many pages */
> int dirty_rate_high_cnt;
> - /* How many times we have synchronized the bitmap */
> - uint64_t bitmap_sync_count;
> /* these variables are used for bitmap sync */
> /* last time we did a full bitmap_sync */
> int64_t time_last_bitmap_sync;
> @@ -187,32 +187,8 @@ struct RAMState {
> uint64_t xbzrle_cache_miss_prev;
> /* number of iterations at the beginning of period */
> uint64_t iterations_prev;
> - /* Accounting fields */
> - /* number of zero pages. It used to be pages filled by the same char. */
> - uint64_t zero_pages;
> - /* number of normal transferred pages */
> - uint64_t norm_pages;
> /* Iterations since start */
> uint64_t iterations;
> - /* xbzrle transmitted bytes. Notice that this is with
> - * compression, they can't be calculated from the pages */
> - uint64_t xbzrle_bytes;
> - /* xbzrle transmmited pages */
> - uint64_t xbzrle_pages;
> - /* xbzrle number of cache miss */
> - uint64_t xbzrle_cache_miss;
> - /* xbzrle miss rate */
> - double xbzrle_cache_miss_rate;
> - /* xbzrle number of overflows */
> - uint64_t xbzrle_overflows;
> - /* number of dirty bits in the bitmap */
> - uint64_t migration_dirty_pages;
> - /* total number of bytes transferred */
> - uint64_t bytes_transferred;
> - /* number of dirtied pages in the last second */
> - uint64_t dirty_pages_rate;
> - /* Count of requests incoming from destination */
> - uint64_t postcopy_requests;
> /* protects modification of the bitmap */
> QemuMutex bitmap_mutex;
> /* The RAMBlock used in the last src_page_requests */
> @@ -225,65 +201,7 @@ typedef struct RAMState RAMState;
>
> static RAMState ram_state;
>
> -uint64_t dup_mig_pages_transferred(void)
> -{
> - return ram_state.zero_pages;
> -}
> -
> -uint64_t norm_mig_pages_transferred(void)
> -{
> - return ram_state.norm_pages;
> -}
> -
> -uint64_t xbzrle_mig_bytes_transferred(void)
> -{
> - return ram_state.xbzrle_bytes;
> -}
> -
> -uint64_t xbzrle_mig_pages_transferred(void)
> -{
> - return ram_state.xbzrle_pages;
> -}
> -
> -uint64_t xbzrle_mig_pages_cache_miss(void)
> -{
> - return ram_state.xbzrle_cache_miss;
> -}
> -
> -double xbzrle_mig_cache_miss_rate(void)
> -{
> - return ram_state.xbzrle_cache_miss_rate;
> -}
> -
> -uint64_t xbzrle_mig_pages_overflow(void)
> -{
> - return ram_state.xbzrle_overflows;
> -}
> -
> -uint64_t ram_bytes_transferred(void)
> -{
> - return ram_state.bytes_transferred;
> -}
> -
> -uint64_t ram_pages_remaining(void)
> -{
> - return ram_state.migration_dirty_pages;
> -}
> -
> -uint64_t ram_dirty_sync_count(void)
> -{
> - return ram_state.bitmap_sync_count;
> -}
> -
> -uint64_t ram_dirty_pages_rate(void)
> -{
> - return ram_state.dirty_pages_rate;
> -}
> -
> -uint64_t ram_postcopy_requests(void)
> -{
> - return ram_state.postcopy_requests;
> -}
> +MigrationStats ram_counters;
>
> /* used by the search for pages to send */
> struct PageSearchStatus {
> @@ -510,7 +428,7 @@ static void xbzrle_cache_zero_page(RAMState *rs,
> ram_addr_t current_addr)
> /* We don't care if this fails to allocate a new cache page
> * as long as it updated an old one */
> cache_insert(XBZRLE.cache, current_addr, XBZRLE.zero_target_page,
> - rs->bitmap_sync_count);
> + ram_counters.dirty_sync_count);
> }
>
> #define ENCODING_FLAG_XBZRLE 0x1
> @@ -536,11 +454,12 @@ static int save_xbzrle_page(RAMState *rs, uint8_t
> **current_data,
> int encoded_len = 0, bytes_xbzrle;
> uint8_t *prev_cached_page;
>
> - if (!cache_is_cached(XBZRLE.cache, current_addr, rs->bitmap_sync_count))
> {
> - rs->xbzrle_cache_miss++;
> + if (!cache_is_cached(XBZRLE.cache, current_addr,
> + ram_counters.dirty_sync_count)) {
> + xbzrle_counters.cache_miss++;
> if (!last_stage) {
> if (cache_insert(XBZRLE.cache, current_addr, *current_data,
> - rs->bitmap_sync_count) == -1) {
> + ram_counters.dirty_sync_count) == -1) {
> return -1;
> } else {
> /* update *current_data when the page has been
> @@ -565,7 +484,7 @@ static int save_xbzrle_page(RAMState *rs, uint8_t
> **current_data,
> return 0;
> } else if (encoded_len == -1) {
> trace_save_xbzrle_page_overflow();
> - rs->xbzrle_overflows++;
> + xbzrle_counters.overflow++;
> /* update data in the cache */
> if (!last_stage) {
> memcpy(prev_cached_page, *current_data, TARGET_PAGE_SIZE);
> @@ -586,9 +505,9 @@ static int save_xbzrle_page(RAMState *rs, uint8_t
> **current_data,
> qemu_put_be16(rs->f, encoded_len);
> qemu_put_buffer(rs->f, XBZRLE.encoded_buf, encoded_len);
> bytes_xbzrle += encoded_len + 1 + 2;
> - rs->xbzrle_pages++;
> - rs->xbzrle_bytes += bytes_xbzrle;
> - rs->bytes_transferred += bytes_xbzrle;
> + xbzrle_counters.pages++;
> + xbzrle_counters.bytes += bytes_xbzrle;
> + ram_counters.transferred += bytes_xbzrle;
>
> return 1;
> }
> @@ -630,7 +549,7 @@ static inline bool migration_bitmap_clear_dirty(RAMState
> *rs,
> ret = test_and_clear_bit(page, rb->bmap);
>
> if (ret) {
> - rs->migration_dirty_pages--;
> + ram_counters.remaining_pages--;
> }
> return ret;
> }
> @@ -638,7 +557,7 @@ static inline bool migration_bitmap_clear_dirty(RAMState
> *rs,
> static void migration_bitmap_sync_range(RAMState *rs, RAMBlock *rb,
> ram_addr_t start, ram_addr_t length)
> {
> - rs->migration_dirty_pages +=
> + ram_counters.remaining_pages +=
> cpu_physical_memory_sync_dirty_bitmap(rb, start, length,
> &rs->num_dirty_pages_period);
> }
> @@ -670,7 +589,7 @@ static void migration_bitmap_sync(RAMState *rs)
> int64_t end_time;
> uint64_t bytes_xfer_now;
>
> - rs->bitmap_sync_count++;
> + ram_counters.dirty_sync_count++;
>
> if (!rs->time_last_bitmap_sync) {
> rs->time_last_bitmap_sync = qemu_clock_get_ms(QEMU_CLOCK_REALTIME);
> @@ -694,9 +613,9 @@ static void migration_bitmap_sync(RAMState *rs)
> /* more than 1 second = 1000 millisecons */
> if (end_time > rs->time_last_bitmap_sync + 1000) {
> /* calculate period counters */
> - rs->dirty_pages_rate = rs->num_dirty_pages_period * 1000
> + ram_counters.dirty_pages_rate = rs->num_dirty_pages_period * 1000
> / (end_time - rs->time_last_bitmap_sync);
> - bytes_xfer_now = ram_bytes_transferred();
> + bytes_xfer_now = ram_counters.transferred;
>
> if (migrate_auto_converge()) {
> /* The following detection logic can be refined later. For now:
> @@ -716,13 +635,13 @@ static void migration_bitmap_sync(RAMState *rs)
>
> if (migrate_use_xbzrle()) {
> if (rs->iterations_prev != rs->iterations) {
> - rs->xbzrle_cache_miss_rate =
> - (double)(rs->xbzrle_cache_miss -
> + xbzrle_counters.cache_miss_rate =
> + (double)(xbzrle_counters.cache_miss -
> rs->xbzrle_cache_miss_prev) /
> (rs->iterations - rs->iterations_prev);
> }
> rs->iterations_prev = rs->iterations;
> - rs->xbzrle_cache_miss_prev = rs->xbzrle_cache_miss;
> + rs->xbzrle_cache_miss_prev = xbzrle_counters.cache_miss;
> }
>
> /* reset period counters */
> @@ -731,7 +650,7 @@ static void migration_bitmap_sync(RAMState *rs)
> rs->bytes_xfer_prev = bytes_xfer_now;
> }
> if (migrate_use_events()) {
> - qapi_event_send_migration_pass(rs->bitmap_sync_count, NULL);
> + qapi_event_send_migration_pass(ram_counters.dirty_sync_count, NULL);
> }
> }
>
> @@ -751,11 +670,11 @@ static int save_zero_page(RAMState *rs, RAMBlock
> *block, ram_addr_t offset,
> int pages = -1;
>
> if (is_zero_range(p, TARGET_PAGE_SIZE)) {
> - rs->zero_pages++;
> - rs->bytes_transferred +=
> + ram_counters.duplicate++;
> + ram_counters.transferred +=
> save_page_header(rs, rs->f, block, offset | RAM_SAVE_FLAG_ZERO);
> qemu_put_byte(rs->f, 0);
> - rs->bytes_transferred += 1;
> + ram_counters.transferred += 1;
> pages = 1;
> }
>
> @@ -803,7 +722,7 @@ static int ram_save_page(RAMState *rs, PageSearchStatus
> *pss, bool last_stage)
> ret = ram_control_save_page(rs->f, block->offset,
> offset, TARGET_PAGE_SIZE, &bytes_xmit);
> if (bytes_xmit) {
> - rs->bytes_transferred += bytes_xmit;
> + ram_counters.transferred += bytes_xmit;
> pages = 1;
> }
>
> @@ -814,9 +733,9 @@ static int ram_save_page(RAMState *rs, PageSearchStatus
> *pss, bool last_stage)
> if (ret != RAM_SAVE_CONTROL_NOT_SUPP) {
> if (ret != RAM_SAVE_CONTROL_DELAYED) {
> if (bytes_xmit > 0) {
> - rs->norm_pages++;
> + ram_counters.normal++;
> } else if (bytes_xmit == 0) {
> - rs->zero_pages++;
> + ram_counters.duplicate++;
> }
> }
> } else {
> @@ -842,8 +761,8 @@ static int ram_save_page(RAMState *rs, PageSearchStatus
> *pss, bool last_stage)
>
> /* XBZRLE overflow or normal page */
> if (pages == -1) {
> - rs->bytes_transferred += save_page_header(rs, rs->f, block,
> - offset |
> RAM_SAVE_FLAG_PAGE);
> + ram_counters.transferred +=
> + save_page_header(rs, rs->f, block, offset | RAM_SAVE_FLAG_PAGE);
> if (send_async) {
> qemu_put_buffer_async(rs->f, p, TARGET_PAGE_SIZE,
> migrate_release_ram() &
> @@ -851,9 +770,9 @@ static int ram_save_page(RAMState *rs, PageSearchStatus
> *pss, bool last_stage)
> } else {
> qemu_put_buffer(rs->f, p, TARGET_PAGE_SIZE);
> }
> - rs->bytes_transferred += TARGET_PAGE_SIZE;
> + ram_counters.transferred += TARGET_PAGE_SIZE;
> pages = 1;
> - rs->norm_pages++;
> + ram_counters.normal++;
> }
>
> XBZRLE_cache_unlock();
> @@ -905,7 +824,7 @@ static void flush_compressed_data(RAMState *rs)
> qemu_mutex_lock(&comp_param[idx].mutex);
> if (!comp_param[idx].quit) {
> len = qemu_put_qemu_file(rs->f, comp_param[idx].file);
> - rs->bytes_transferred += len;
> + ram_counters.transferred += len;
> }
> qemu_mutex_unlock(&comp_param[idx].mutex);
> }
> @@ -935,8 +854,8 @@ static int compress_page_with_multi_thread(RAMState *rs,
> RAMBlock *block,
> qemu_cond_signal(&comp_param[idx].cond);
> qemu_mutex_unlock(&comp_param[idx].mutex);
> pages = 1;
> - rs->norm_pages++;
> - rs->bytes_transferred += bytes_xmit;
> + ram_counters.normal++;
> + ram_counters.transferred += bytes_xmit;
> break;
> }
> }
> @@ -976,15 +895,15 @@ static int ram_save_compressed_page(RAMState *rs,
> PageSearchStatus *pss,
> ret = ram_control_save_page(rs->f, block->offset,
> offset, TARGET_PAGE_SIZE, &bytes_xmit);
> if (bytes_xmit) {
> - rs->bytes_transferred += bytes_xmit;
> + ram_counters.transferred += bytes_xmit;
> pages = 1;
> }
> if (ret != RAM_SAVE_CONTROL_NOT_SUPP) {
> if (ret != RAM_SAVE_CONTROL_DELAYED) {
> if (bytes_xmit > 0) {
> - rs->norm_pages++;
> + ram_counters.normal++;
> } else if (bytes_xmit == 0) {
> - rs->zero_pages++;
> + ram_counters.duplicate++;
> }
> }
> } else {
> @@ -1004,8 +923,8 @@ static int ram_save_compressed_page(RAMState *rs,
> PageSearchStatus *pss,
> blen = qemu_put_compression_data(rs->f, p, TARGET_PAGE_SIZE,
> migrate_compress_level());
> if (blen > 0) {
> - rs->bytes_transferred += bytes_xmit + blen;
> - rs->norm_pages++;
> + ram_counters.transferred += bytes_xmit + blen;
> + ram_counters.normal++;
> pages = 1;
> } else {
> qemu_file_set_error(rs->f, blen);
> @@ -1213,7 +1132,7 @@ int ram_save_queue_pages(const char *rbname, ram_addr_t
> start, ram_addr_t len)
> RAMBlock *ramblock;
> RAMState *rs = &ram_state;
>
> - rs->postcopy_requests++;
> + ram_counters.postcopy_requests++;
> rcu_read_lock();
> if (!rbname) {
> /* Reuse last RAMBlock */
> @@ -1401,13 +1320,12 @@ static int ram_find_and_save_block(RAMState *rs, bool
> last_stage)
> void acct_update_position(QEMUFile *f, size_t size, bool zero)
> {
> uint64_t pages = size / TARGET_PAGE_SIZE;
> - RAMState *rs = &ram_state;
>
> if (zero) {
> - rs->zero_pages += pages;
> + ram_counters.duplicate += pages;
> } else {
> - rs->norm_pages += pages;
> - rs->bytes_transferred += size;
> + ram_counters.normal += pages;
> + ram_counters.transferred += size;
> qemu_update_position(f, size);
> }
> }
> @@ -1631,7 +1549,6 @@ static void
> postcopy_chunk_hostpages_pass(MigrationState *ms, bool unsent_pass,
> RAMBlock *block,
> PostcopyDiscardState *pds)
> {
> - RAMState *rs = &ram_state;
> unsigned long *bitmap = block->bmap;
> unsigned long *unsentmap = block->unsentmap;
> unsigned int host_ratio = block->page_size / TARGET_PAGE_SIZE;
> @@ -1724,7 +1641,7 @@ static void
> postcopy_chunk_hostpages_pass(MigrationState *ms, bool unsent_pass,
> * Remark them as dirty, updating the count for any pages
> * that weren't previously dirty.
> */
> - rs->migration_dirty_pages += !test_and_set_bit(page, bitmap);
> + ram_counters.remaining_pages += !test_and_set_bit(page,
> bitmap);
> }
> }
>
> @@ -1932,7 +1849,7 @@ static int ram_state_init(RAMState *rs)
> * Count the total number of pages used by ram blocks not including any
> * gaps due to alignment or unplugs.
> */
> - rs->migration_dirty_pages = ram_bytes_total() >> TARGET_PAGE_BITS;
> + ram_counters.remaining_pages = ram_bytes_total() >> TARGET_PAGE_BITS;
>
> memory_global_dirty_log_start();
> migration_bitmap_sync(rs);
> @@ -2057,7 +1974,7 @@ static int ram_save_iterate(QEMUFile *f, void *opaque)
> ram_control_after_iterate(f, RAM_CONTROL_ROUND);
>
> qemu_put_be64(f, RAM_SAVE_FLAG_EOS);
> - rs->bytes_transferred += 8;
> + ram_counters.transferred += 8;
>
> ret = qemu_file_get_error(f);
> if (ret < 0) {
> @@ -2119,7 +2036,7 @@ static void ram_save_pending(QEMUFile *f, void *opaque,
> uint64_t max_size,
> RAMState *rs = opaque;
> uint64_t remaining_size;
>
> - remaining_size = rs->migration_dirty_pages * TARGET_PAGE_SIZE;
> + remaining_size = ram_counters.remaining_pages * TARGET_PAGE_SIZE;
>
> if (!migration_in_postcopy() &&
> remaining_size < max_size) {
> @@ -2128,7 +2045,7 @@ static void ram_save_pending(QEMUFile *f, void *opaque,
> uint64_t max_size,
> migration_bitmap_sync(rs);
> rcu_read_unlock();
> qemu_mutex_unlock_iothread();
> - remaining_size = rs->migration_dirty_pages * TARGET_PAGE_SIZE;
> + remaining_size = ram_counters.remaining_pages * TARGET_PAGE_SIZE;
> }
>
> /* We can do postcopy, and all the data is postcopiable */
> diff --git a/migration/ram.h b/migration/ram.h
> index 5864470..9eadc8c 100644
> --- a/migration/ram.h
> +++ b/migration/ram.h
> @@ -32,19 +32,10 @@
> #include "qemu-common.h"
> #include "exec/cpu-common.h"
>
> +extern MigrationStats ram_counters;
> +extern XBZRLECacheStats xbzrle_counters;
> +
> int64_t xbzrle_cache_resize(int64_t new_size);
> -uint64_t dup_mig_pages_transferred(void);
> -uint64_t norm_mig_pages_transferred(void);
> -uint64_t xbzrle_mig_bytes_transferred(void);
> -uint64_t xbzrle_mig_pages_transferred(void);
> -uint64_t xbzrle_mig_pages_cache_miss(void);
> -double xbzrle_mig_cache_miss_rate(void);
> -uint64_t xbzrle_mig_pages_overflow(void);
> -uint64_t ram_bytes_transferred(void);
> -uint64_t ram_pages_remaining(void);
> -uint64_t ram_dirty_sync_count(void);
> -uint64_t ram_dirty_pages_rate(void);
> -uint64_t ram_postcopy_requests(void);
> uint64_t ram_bytes_total(void);
>
> void migrate_compress_threads_create(void);
> --
> 2.9.4
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH 1/5] ram: Call migration_page_queue_free() at ram_migration_cleanup(), (continued)