[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ram: Split dirty bitmap by RAMBlock
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH] ram: Split dirty bitmap by RAMBlock |
Date: |
Wed, 26 Apr 2017 12:53:23 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Hailiang Zhang <address@hidden> wrote:
> On 2017/4/26 15:32, Juan Quintela wrote:
>> Both the ram bitmap and the unsent bitmap are split by RAMBlock.
>>
>> Signed-off-by: Juan Quintela <address@hidden>
>>
>> --
>>
>> Fix compilation when DEBUG_POSTCOPY is enabled (thanks Hailiang)
>>
>> Signed-off-by: Juan Quintela <address@hidden>
>> + postcopy_chunk_hostpages_pass(ms, false, block, pds);
>> +
>> + postcopy_discard_send_finish(ms, pds);
>> return 0;
>
> It always return 0 here for postcopy_chunk_hostpages(), so change it to void ?
This was dave code. I assume that he has in mind something where it can
fail?
>> + /* Deal with TPS != HPS and huge pages */
>> + ret = postcopy_chunk_hostpages(ms, block);
>> + if (ret) {
>
> It will never go here, see above.
Yeap, I also noticed.
>> + bitmap_or(unsentmap, unsentmap, bitmap, pages);
>> #ifdef DEBUG_POSTCOPY
>> - ram_debug_dump_bitmap(unsentmap, true);
>> + ram_debug_dump_bitmap(unsentmap, true, pages);
> Indent, you use tab here.
Fixed.
> Others look good to me.
>
> Reviewed-by: zhanghailiang <address@hidden>
Thanks.