[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 09/31] ram: Move dup_pages into RAMState
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 09/31] ram: Move dup_pages into RAMState |
Date: |
Thu, 16 Mar 2017 12:27:12 +0000 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
* Juan Quintela (address@hidden) wrote:
> Once there rename it to its actual meaning, zero_pages.
>
> Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
> ---
> migration/ram.c | 26 +++++++++++++++-----------
> 1 file changed, 15 insertions(+), 11 deletions(-)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 6cdad06..059e9f1 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -159,6 +159,9 @@ struct RAMState {
> uint64_t xbzrle_cache_miss_prev;
> /* number of iterations at the beggining 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;
> };
> typedef struct RAMState RAMState;
>
> @@ -166,7 +169,6 @@ static RAMState ram_state;
>
> /* accounting for migration statistics */
> typedef struct AccountingInfo {
> - uint64_t dup_pages;
> uint64_t skipped_pages;
> uint64_t norm_pages;
> uint64_t iterations;
> @@ -186,12 +188,12 @@ static void acct_clear(void)
>
> uint64_t dup_mig_bytes_transferred(void)
> {
> - return acct_info.dup_pages * TARGET_PAGE_SIZE;
> + return ram_state.zero_pages * TARGET_PAGE_SIZE;
> }
>
> uint64_t dup_mig_pages_transferred(void)
> {
> - return acct_info.dup_pages;
> + return ram_state.zero_pages;
> }
>
> uint64_t skipped_mig_bytes_transferred(void)
> @@ -718,13 +720,14 @@ static void migration_bitmap_sync(RAMState *rs)
> * @p: pointer to the page
> * @bytes_transferred: increase it with the number of transferred bytes
> */
> -static int save_zero_page(QEMUFile *f, RAMBlock *block, ram_addr_t offset,
> +static int save_zero_page(RAMState *rs, QEMUFile *f, RAMBlock *block,
> + ram_addr_t offset,
> uint8_t *p, uint64_t *bytes_transferred)
> {
> int pages = -1;
>
> if (is_zero_range(p, TARGET_PAGE_SIZE)) {
> - acct_info.dup_pages++;
> + rs->zero_pages++;
> *bytes_transferred += save_page_header(f, block,
> offset |
> RAM_SAVE_FLAG_COMPRESS);
> qemu_put_byte(f, 0);
> @@ -797,11 +800,11 @@ static int ram_save_page(RAMState *rs, MigrationState
> *ms, QEMUFile *f,
> if (bytes_xmit > 0) {
> acct_info.norm_pages++;
> } else if (bytes_xmit == 0) {
> - acct_info.dup_pages++;
> + rs->zero_pages++;
> }
> }
> } else {
> - pages = save_zero_page(f, block, offset, p, bytes_transferred);
> + pages = save_zero_page(rs, f, block, offset, p, bytes_transferred);
> if (pages > 0) {
> /* Must let xbzrle know, otherwise a previous (now 0'd) cached
> * page would be stale
> @@ -973,7 +976,7 @@ static int ram_save_compressed_page(RAMState *rs,
> MigrationState *ms,
> if (bytes_xmit > 0) {
> acct_info.norm_pages++;
> } else if (bytes_xmit == 0) {
> - acct_info.dup_pages++;
> + rs->zero_pages++;
> }
> }
> } else {
> @@ -985,7 +988,7 @@ static int ram_save_compressed_page(RAMState *rs,
> MigrationState *ms,
> */
> if (block != rs->last_sent_block) {
> flush_compressed_data(f);
> - pages = save_zero_page(f, block, offset, p, bytes_transferred);
> + pages = save_zero_page(rs, f, block, offset, p,
> bytes_transferred);
> if (pages == -1) {
> /* Make sure the first page is sent out before other pages */
> bytes_xmit = save_page_header(f, block, offset |
> @@ -1006,7 +1009,7 @@ static int ram_save_compressed_page(RAMState *rs,
> MigrationState *ms,
> }
> } else {
> offset |= RAM_SAVE_FLAG_CONTINUE;
> - pages = save_zero_page(f, block, offset, p, bytes_transferred);
> + pages = save_zero_page(rs, f, block, offset, p,
> bytes_transferred);
> if (pages == -1) {
> pages = compress_page_with_multi_thread(f, block, offset,
> bytes_transferred);
> @@ -1428,7 +1431,7 @@ void acct_update_position(QEMUFile *f, size_t size,
> bool zero)
> {
> uint64_t pages = size / TARGET_PAGE_SIZE;
> if (zero) {
> - acct_info.dup_pages += pages;
> + ram_state.zero_pages += pages;
> } else {
> acct_info.norm_pages += pages;
> bytes_transferred += size;
> @@ -1941,6 +1944,7 @@ static int ram_save_init_globals(RAMState *rs)
>
> rs->dirty_rate_high_cnt = 0;
> rs->bitmap_sync_count = 0;
> + rs->zero_pages = 0;
> migration_bitmap_sync_init(rs);
> qemu_mutex_init(&migration_bitmap_mutex);
>
> --
> 2.9.3
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH 01/31] ram: move more fields into RAMState, (continued)
- [Qemu-devel] [PATCH 07/31] ram: Move xbzrle_cache_miss_prev into RAMState, Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 10/31] ram: Remove unused dump_mig_dbytes_transferred(), Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 12/31] ram: Move norm_pages to RAMState, Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 09/31] ram: Move dup_pages into RAMState, Juan Quintela, 2017/03/15
- Re: [Qemu-devel] [PATCH 09/31] ram: Move dup_pages into RAMState,
Dr. David Alan Gilbert <=
- [Qemu-devel] [PATCH 13/31] ram: Remove norm_mig_bytes_transferred, Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 14/31] ram: Move iterations into RAMState, Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 11/31] ram: Remove unused pages_skiped variable, Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 15/31] ram: Move xbzrle_bytes into RAMState, Juan Quintela, 2017/03/15
- [Qemu-devel] [PATCH 19/31] ram: move xbzrle_overflows into RAMState, Juan Quintela, 2017/03/15