Re: [PATCH] migration: Rename xbzrle_enabled xbzrle_started

[Peter Xu]

On Thu, May 04, 2023 at 01:53:23PM +0200, Juan Quintela wrote:
> Otherwise it is confusing with the function xbzrle_enabled().
> 
> Suggested-by: Daniel P. Berrangé <berrange@redhat.com>
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> ---
>  migration/ram.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/migration/ram.c b/migration/ram.c
> index 43338e1f5b..06015eeb0b 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -388,8 +388,8 @@ struct RAMState {
>      uint64_t xbzrle_pages_prev;
>      /* Amount of xbzrle encoded bytes since the beginning of the period */
>      uint64_t xbzrle_bytes_prev;
> -    /* Start using XBZRLE (e.g., after the first round). */
> -    bool xbzrle_enabled;
> +    /* Are we really  using XBZRLE (e.g., after the first round). */

double spaces

> +    bool xbzrle_started;

Reviewed-by: Peter Xu <peterx@redhat.com>

-- 
Peter Xu