[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/9] Migration: Remove copying of guest ram p
From: |
Orit Wasserman |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/9] Migration: Remove copying of guest ram pages |
Date: |
Thu, 21 Mar 2013 19:35:17 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
On 03/21/2013 07:12 PM, Paolo Bonzini wrote:
> Il 21/03/2013 17:05, Orit Wasserman ha scritto:
>> In migration all data is copied to a static buffer in QEMUFile,
>> this hurts our network bandwidth and CPU usage especially with large guests.
>> We switched to iovec for storing different buffers to send (even a byte
>> field is
>> considered as a buffer) and use writev to send the iovec.
>> writev was chosen (as apposed to sendmsg) because it supprts non socket fds.
>>
>> Guest memory pages are not copied by calling a new function
>> qemu_put_buffer_no_copy.
>> The page header data and device state data are still copied into the static
>> buffer. This data consists of a lot of bytes and integer fields and the
>> static
>> buffer is used to store it during batching.
>> Another improvement is changing qemu_putbe64/32/16 to create a single
>> buffer instead of several byte sized buffer.
>>
>> git repository: git://github.com/oritwas/qemu.git sendv_v2
>>
>> Change from v2:
>> Always send data for the iovec even if writev_buffer is not implemented.
>> Coalesce adjacent iovecs to create one big buffer from small adjacent buffer.
>>
>> Changes from v1:
>> Use iov_send for socket.
>> Make writev_buffer optional and if it is not implemented use put_buffer
>
> I didn't review it too closely, so I don't feel like giving my
> Reviewed-by. Still, I must say it is a very nice and simple solution.
> Kudos!
>
> Just one question: patch 6 is not strictly needed anymore now that you
> have coalescing, no? Should Juan leave it out?
true, Juan had a small fix so I'm sending v4 I will remove it.
>
> Paolo
>
- Re: [Qemu-devel] [PATCH v3 9/9] coalesce adjacent iovecs, (continued)
Re: [Qemu-devel] [PATCH v3 9/9] coalesce adjacent iovecs, Juan Quintela, 2013/03/21
[Qemu-devel] [PATCH v3 8/9] Use qemu_put_buffer_no_copy for guest memory pages, Orit Wasserman, 2013/03/21
Re: [Qemu-devel] [PATCH v3 0/9] Migration: Remove copying of guest ram pages, Paolo Bonzini, 2013/03/21
- Re: [Qemu-devel] [PATCH v3 0/9] Migration: Remove copying of guest ram pages,
Orit Wasserman <=
Re: [Qemu-devel] [PATCH v3 0/9] Migration: Remove copying of guest ram pages, Juan Quintela, 2013/03/21