From: Anthony Liguori <anthony@codemonkey.ws>
To: "Daniel P. Berrange" <berrange@redhat.com>, qemu-devel@nongnu.org
Cc: Uri Lublin <uril@redhat.com>
Subject: Re: [Qemu-devel] [PATCH] migration: adding migration to/from a file
Date: Tue, 20 Jan 2009 10:24:50 -0600 [thread overview]
Message-ID: <4975FAD2.6050201@codemonkey.ws> (raw)
In-Reply-To: <20090120111530.GJ6004@redhat.com>
Daniel P. Berrange wrote:
> Does it matter if it blocks though ? Migrating to a file is not
> "live" migration anyway,
Yes. The general reason is that blocking in QEMU is evil because it's
single threaded. If you block on IO in QEMU, then everything comes
grinding to a halt. The VM may not be running, but you still want the
GUI to be responsive, the monitor, and VNC to be responsive. This is
particularly import in the context of asynchronous monitor
notifications. savevm may be a long operation and there may be pending
notifications that would get significantly delayed.
If you issue a "stop" before doing live migration, the result is
equivalent to "savevm" from a binary perspective but the monitor will
remain responsive during the whole operation. I'd like to eliminate the
blocking version of savevm entirely but we're currently prevent from
doing this because of the savevm to a disk. The issue is that you
cannot write to the qcow2 snapshot space while other IO operations can
happen. stop'ing the guest would prevent this (obviously) but it would
be nice to support this in a truly live form.
I think we need to adjust the qcow2 snapshot format to support chaining
of snapshot segments. That would allow a truly live save to qcow2.
Regards,
Anthony Liguori
> so if we happen to block the rest of the
> QEMU event loop during course of writing to the file it doesn't seem
> too serious. Or is there some scenario which badly breaks with blocking
> writes even for non-live usage ?
>
> Daniel
>
next prev parent reply other threads:[~2009-01-20 16:25 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-19 0:18 [Qemu-devel] [PATCH] migration: adding migration to/from a file Uri Lublin
2009-01-19 0:55 ` Paul Brook
2009-01-20 11:05 ` Uri Lublin
2009-01-20 14:06 ` Ian Jackson
2009-01-19 15:11 ` Anthony Liguori
2009-01-20 11:15 ` Daniel P. Berrange
2009-01-20 12:15 ` Uri Lublin
2009-01-20 16:24 ` Anthony Liguori [this message]
2009-01-20 11:32 ` Uri Lublin
2009-01-20 11:53 ` Daniel P. Berrange
2009-01-20 16:26 ` Anthony Liguori
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=4975FAD2.6050201@codemonkey.ws \
--to=anthony@codemonkey.ws \
--cc=berrange@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=uril@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).