Re: [PATCH] migration: Rename xbzrle_enabled xbzrle_started

[Juan Quintela]

Peter Xu <peterx@redhat.com> wrote:
> 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

Fixed.

>
>> +    bool xbzrle_started;
>
> Reviewed-by: Peter Xu <peterx@redhat.com>

Thanks.