From: Wenchao Xia <xiawenc@linux.vnet.ibm.com>
To: Pavel Hrdina <phrdina@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
aliguori@us.ibm.com, quintela@redhat.com, stefanha@gmail.com,
qemu-devel@nongnu.org, pbonzini@redhat.com
Subject: Re: [Qemu-devel] [RFC PATCH 0/4] savevm: save vmsate with fixed size
Date: Wed, 06 Mar 2013 09:25:01 +0800 [thread overview]
Message-ID: <51369AED.7020903@linux.vnet.ibm.com> (raw)
In-Reply-To: <5135DF34.8060508@redhat.com>
于 2013-3-5 20:04, Pavel Hrdina 写道:
> On 03/04/2013 04:04 PM, Wenchao Xia wrote:
>>
>>>> You probably get some improvements of the file size when the migration
>>>> takes a while, depending on how much of the memory actually has to be
>>>> saved. You might however end up with a lot more small writes instead of
>>>> some big ones before, which might hurt performance.
>>>>
>>>> Do you have any data about the resulting performance and file size?
>>>>
>>> ah, an important issue I haven't test, thanks for tipping it, let
>>> me add code for migration to file, and have a test. It also
>>> can be optimized a bit in qemu_fseek(), but IMHO the optimization
>>> for small writes would better goto block layer either in qemu
>>> or underling components in system.
>>>
>> Hi,Juan
>> talking about performance, in migration there are two buffer used,
>> one in qemu_file as static array, and one in migration as dynamic
>> allocated buffer,
>> should they be merged to avoid extra memcpy?
>>
>
> Hi Wenchao,
>
> This is already posted on qemu-devel list,
>
> http://lists.gnu.org/archive/html/qemu-devel/2013-02/msg04119.html
> http://lists.gnu.org/archive/html/qemu-devel/2013-02/msg04222.html
>
> Pavel
>
Great, hope this can be upstream fast, thanks for tipping it.
--
Best Regards
Wenchao Xia
prev parent reply other threads:[~2013-03-06 1:25 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1362038985-19008-1-git-send-email-xiawenc@linux.vnet.ibm.com>
[not found] ` <20130228105035.GE2429@dhcp-200-207.str.redhat.com>
[not found] ` <513013F0.2030007@linux.vnet.ibm.com>
2013-03-01 9:27 ` [Qemu-devel] [RFC PATCH 0/4] savevm: save vmsate with fixed size Kevin Wolf
2013-03-04 15:04 ` Wenchao Xia
2013-03-05 12:04 ` Pavel Hrdina
2013-03-06 1:25 ` Wenchao Xia [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51369AED.7020903@linux.vnet.ibm.com \
--to=xiawenc@linux.vnet.ibm.com \
--cc=aliguori@us.ibm.com \
--cc=kwolf@redhat.com \
--cc=pbonzini@redhat.com \
--cc=phrdina@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
--cc=stefanha@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.