From: Jan Kiszka <jan.kiszka@siemens.com>
To: Liran Schour <LIRANS@il.ibm.com>
Cc: "qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: [Qemu-devel] Re: [PATCH 2/3 v5] Block live migration
Date: Thu, 26 Nov 2009 18:05:48 +0100 [thread overview]
Message-ID: <4B0EB56C.2040504@siemens.com> (raw)
In-Reply-To: <OF4420BC14.A7407910-ONC225767A.0056688D-C225767A.0057093C@il.ibm.com>
Liran Schour wrote:
> Jan Kiszka <jan.kiszka@siemens.com> wrote on 26/11/2009 15:53:49:
>
>
>>> + qemu_get_buffer(f, buf,
>>> + BLOCK_SIZE);
>>> + if(bs != NULL) {
>>> +
>>> + bdrv_write(bs, (addr >> SECTOR_BITS),
>>> + buf, block_mig_state->sectors_per_block);
>> This synchronous write-back translates appears to be the reason for an
>> unusable migration (or restore from snapshot) speed: about 100 KB/s here
>> (ie. 22h for a rather small 8G guest :( ). Did you already try to
>> improve this situation?
>
> I have seen this behavior, but it seems that there is a very big difference
> in the performance if the new block device is based on an allocated file
> already (try the same migration to an already allocated file in the
> requested size) I am trying to figure out why we see this behavior.(any
> ideas?)
Yes, much faster, more than 6 MB/s. Not really impressing, but that's
now likely due to the tiny block size.
That 4K also made the unallocated write so awfully slow as the image
file had to be continuously extended by this amount.
> Anyway we can turn the writes to be async but we have to synchronize all
> destinations writes before completing the migration and moving the guest to
> the destination. When the guest starts to run on the destination all writes
> should be finished, so anyhow we need to wait synchronously to the writes.
> I will look on this further next week.
Well, we actually need these changes:
1. Reasonable block size (if we cannot overcome the cluster size, we
need to coalesce blocks on write out)
2. Async write, throttled by the sync read, or both asynchronously.
Unfortunately, qemu's aio does not work yet at the point we need
it...
I will also continue to dig into this.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
next prev parent reply other threads:[~2009-11-26 17:06 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-02 13:40 [Qemu-devel] [PATCH 2/3 v5] Block live migration lirans
2009-11-24 22:55 ` [Qemu-devel] " Jan Kiszka
2009-11-26 7:13 ` Liran Schour
2009-11-26 7:39 ` Jan Kiszka
2009-11-26 13:53 ` Jan Kiszka
2009-11-26 15:50 ` Liran Schour
2009-11-26 17:05 ` Jan Kiszka [this message]
2009-11-26 17:24 ` Jan Kiszka
2009-11-30 11:49 ` Liran Schour
2009-11-30 12:01 ` Jan Kiszka
2009-11-26 18:06 ` Pierre Riteau
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=4B0EB56C.2040504@siemens.com \
--to=jan.kiszka@siemens.com \
--cc=LIRANS@il.ibm.com \
--cc=qemu-devel@nongnu.org \
/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.