[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v15 7/9] Add XBZRLE to ram_save_block and ram_sa
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH v15 7/9] Add XBZRLE to ram_save_block and ram_save_live |
Date: |
Fri, 06 Jul 2012 08:23:26 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
On 07/05/2012 05:04 PM, Eric Blake wrote:
> On 07/05/2012 06:51 AM, Orit Wasserman wrote:
>> In the outgoing migration check to see if the page is cached and
>> changed than send compressed page by using save_xbrle_page function.
>> In the incoming migration check to see if RAM_SAVE_FLAG_XBRLE is set
>> and decompress the page (by using load_xbrle function).
>>
>> Signed-off-by: Benoit Hudzia <address@hidden>
>> Signed-off-by: Petter Svard <address@hidden>
>> Signed-off-by: Aidan Shribman <address@hidden>
>> Signed-off-by: Orit Wasserman <address@hidden>
>> ---
>
>> +static int save_xbzrle_page(QEMUFile *f, uint8_t *current_data,
>> + ram_addr_t current_addr, RAMBlock *block,
>> + ram_addr_t offset, int cont, int stage)
>> +{
>> + int encoded_len = 0, bytes_sent = -1, ret = -1;
>> + XBZRLEHeader hdr = {
>> + .xh_len = 0,
>> + .xh_flags = 0,
>> + };
>> + uint8_t *prev_cached_page;
>> +
>> + /* Stage 1 cache the page and exit.
>> + Stage 2 check to see if page is cached , if not cache the page.
>
> No space before comma in English comments:
>
> s/ ,/,/
>
>> + /* XBZRLE encoding (if there is no overflow) */
>> + encoded_len = xbzrle_encode_buffer(prev_cached_page, current_data,
>> + TARGET_PAGE_SIZE, XBZRLE.encoded_buf,
>> + TARGET_PAGE_SIZE);
>> + if (encoded_len == 0) {
>> + DPRINTF("Unmodifed page skipping\n");
>
> s/Unmodifed/Unmodified/
>
> I mentioned in my v13 review changing this to:
>
> s/Unmodifed page skipping/Skipping unmodified page/
>
> it looks like you didn't pick up my v13 comments yet (and my fault for
> not complaining at v14). As a result, I stopped reviewing here.
>
I'm really sorry I missed this, I will fix it for next version
Thanks,
Orit
- [Qemu-devel] [PATCH v15 5/9] Change ram_save_block to return -1 if there are no more changes, (continued)
- [Qemu-devel] [PATCH v15 5/9] Change ram_save_block to return -1 if there are no more changes, Orit Wasserman, 2012/07/05
- [Qemu-devel] [PATCH v15 6/9] Add xbzrle_encode_buffer and xbzrle_decode_buffer functions, Orit Wasserman, 2012/07/05
- [Qemu-devel] [PATCH v15 3/9] Add cache handling functions, Orit Wasserman, 2012/07/05
- [Qemu-devel] [PATCH v15 2/9] Add XBZRLE documentation, Orit Wasserman, 2012/07/05
- [Qemu-devel] [PATCH v15 7/9] Add XBZRLE to ram_save_block and ram_save_live, Orit Wasserman, 2012/07/05
- [Qemu-devel] [PATCH v15 8/9] Add set_cachesize command, Orit Wasserman, 2012/07/05
- [Qemu-devel] [PATCH v15 9/9] Add XBZRLE statistics, Orit Wasserman, 2012/07/05