[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] ram: Use MigrationStats for statistics
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] ram: Use MigrationStats for statistics |
Date: |
Tue, 6 Jun 2017 16:05:50 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Mon, Jun 05, 2017 at 01:34:45PM +0100, Dr. David Alan Gilbert wrote:
> * 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?
O:-)
Meanwhile, I don't know whether it'll be necessary to remove all the
functions like ram_bytes_transferred(), e.g., it would be just:
uint64_t ram_bytes_transferred(void)
{
- return ram_state.bytes_transferred;
+ return ram_counters.transferred;
}
But I'm okay with either.
>
>
> Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Peter Xu <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
--
Peter Xu
- Re: [Qemu-devel] [PATCH 1/5] ram: Call migration_page_queue_free() at ram_migration_cleanup(), (continued)