[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] arch_init: Count the total number of pages by u
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH] arch_init: Count the total number of pages by using helper function |
Date: |
Thu, 12 Mar 2015 10:59:04 +0000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
* zhanghailiang (address@hidden) wrote:
> There is already a helper function ram_bytes_total(), we can use it to
> help counting the total number of pages used by ram blocks.
>
> Signed-off-by: zhanghailiang <address@hidden>
> ---
> arch_init.c | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/arch_init.c b/arch_init.c
> index 691b5e2..b22e38e 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -845,13 +845,7 @@ static int ram_save_setup(QEMUFile *f, void *opaque)
> * Count the total number of pages used by ram blocks not including any
> * gaps due to alignment or unplugs.
> */
> - migration_dirty_pages = 0;
> - QLIST_FOREACH_RCU(block, &ram_list.blocks, next) {
> - uint64_t block_pages;
> -
> - block_pages = block->used_length >> TARGET_PAGE_BITS;
> - migration_dirty_pages += block_pages;
> - }
> + migration_dirty_pages = ram_bytes_total() >> TARGET_PAGE_BITS;
Yes, I think Markus had suggested something similar when I did the
original fix.
Thanks,
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
>
> memory_global_dirty_log_start();
> migration_bitmap_sync();
> --
> 1.7.12.4
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK