Peter Xu <pet...@redhat.com> wrote:
> On Thu, Mar 23, 2017 at 09:44:59PM +0100, Juan Quintela wrote:
>> Signed-off-by: Juan Quintela <quint...@redhat.com>
>> Reviewed-by: Dr. David Alan Gilbert <dgilb...@redhat.com>
>> ---
>> migration/ram.c | 20 +++++++++++---------
>> 1 file changed, 11 insertions(+), 9 deletions(-)
>>
>> diff --git a/migration/ram.c b/migration/ram.c
>> index f811e81..5881805 100644
>> --- a/migration/ram.c
>> +++ b/migration/ram.c
>> @@ -154,6 +154,9 @@ struct RAMState {
>> int dirty_rate_high_cnt;
>> /* How many times we have synchronized the bitmap */
>> uint64_t bitmap_sync_count;
>> + /* this variables are used for bitmap sync */
>
> s/this/These/?
>
>> + /* last time we did a full bitmap_sync */
>> + int64_t start_time;
>
> Not sure whether it'll be a good chance we rename this variable in
> this patch to make it a less-generic name, like: bm_sync_start? But
> again, this is nicpicking and totally optional.
I changed it to time_last_bitmap_sync.
Thanks, Juan.
>
> With the typo fixed, please add:
>
> Reviewed-by: Peter Xu <pet...@redhat.com>
>
> Thanks,
>
> -- peterx
