[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/12] migration: fix counting normal page for c
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH 02/12] migration: fix counting normal page for compression |
Date: |
Wed, 13 Jun 2018 16:51:48 +0100 |
User-agent: |
Mutt/1.10.0 (2018-05-17) |
* address@hidden (address@hidden) wrote:
> From: Xiao Guangrong <address@hidden>
>
> The compressed page is not normal page
Is this the right reason?
I think we always increment some counter for a page - so
what gets incremented for a compressed page?
Is the real answer that we do:
ram_save_target_page
control_save_page
compress_page_with_multi_thread
and control_save_page already increments the counter?
Dave
> Signed-off-by: Xiao Guangrong <address@hidden>
> ---
> migration/ram.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 0caf32ab0a..dbf24d8c87 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -1432,7 +1432,6 @@ 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;
> - ram_counters.normal++;
> ram_counters.transferred += bytes_xmit;
> break;
> }
> --
> 2.14.4
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
[Qemu-devel] [PATCH 03/12] migration: fix counting xbzrle cache_miss_rate, guangrong . xiao, 2018/06/04