From: Dor Laor <dlaor@redhat.com>
To: Kevin Wolf <kwolf@redhat.com>
Cc: Anthony Liguori <aliguori@us.ibm.com>,
Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
jes sorensen <jes.sorensen@redhat.com>,
Stefan Hajnoczi <stefanha@gmail.com>,
Marcelo Tosatti <mtosatti@redhat.com>,
qemu-devel@nongnu.org, Orit Wasserman <owasserm@redhat.com>,
Avi Kivity <avi@redhat.com>, Adam Litke <agl@us.ibm.com>
Subject: Re: [Qemu-devel] Image streaming and live block copy
Date: Mon, 27 Jun 2011 12:13:10 +0300 [thread overview]
Message-ID: <4E0849A6.4070009@redhat.com> (raw)
In-Reply-To: <4E0835EB.9010505@redhat.com>
On 06/27/2011 10:48 AM, Kevin Wolf wrote:
> Am 26.06.2011 14:50, schrieb Dor Laor:
>> On 06/24/2011 12:28 PM, Stefan Hajnoczi wrote:
>>> On Sun, Jun 19, 2011 at 5:02 PM, Dor Laor<dlaor@redhat.com> wrote:
>>>> On 06/18/2011 12:17 PM, Stefan Hajnoczi wrote:
>>>>>
>>>>> On Sat, Jun 18, 2011 at 10:15 AM, Stefan Hajnoczi<stefanha@gmail.com>
>>>>> wrote:
>>>>>>
>>>>>> On Fri, Jun 17, 2011 at 1:31 PM, Marcelo Tosatti<mtosatti@redhat.com>
>>>>>> wrote:
>>>>>>>
>>>>>>> On Thu, Jun 16, 2011 at 04:30:18PM +0100, Stefan Hajnoczi wrote:
>>>>>>>>
>>>>>>>> On Thu, Jun 16, 2011 at 11:52:43AM -0300, Marcelo Tosatti wrote:
>>>>>>>> This approach does not use the backing file feature?
>>>>>>>>
>>>>>>>>> blkstream block driver:
>>>>>>>>>
>>>>>>>>> - Maintain in memory whether given block is allocated in local image,
>>>>>>>>> if not, read from remote, write to local. Set block as local.
>>>>>>>>> Local and remote simply two block drivers from image streaming driver
>>>>>>>>> POV.
>>>>>>>>> - Once all blocks are local, notify mgmt so it can switch to local
>>>>>>>>> copy.
>>>>>>>>> - Writes are mirrored to source and destination, minding guest writes
>>>>>>>>> over copy writes.
>>>>>>>>
>>>>>>>> We open the remote file read-only for image streaming and do not want
>>>>>>>> to
>>>>>>>> mirror writes.
>>>>>>>
>>>>>>> Why not? Is there any disadvantage of mirroring writes?
>>>>>>
>>>>>> Think of the use case with a Fedora master image over NFS. You want a
>>>>>> local clone of that master image and use the stream command to copy
>>>>>> the data from the master image into the local clone.
>>>>>>
>>>>>> You cannot modify that master image because other VMs are using it too
>>>>>> and/or you want to be able to clone new VMs from it in the future.
>>>>>
>>>>> BTW the workaround is to create two local images:
>>>>> 1. Local clone with master image as a backing file. This is the live
>>>>> block copy source image.
>>>>> 2. Local image without a backing file. This is the live block copy
>>>>> destination image.
>>>>>
>>>>> But this is not very elegant. Writes get mirrored so that crash recovery
>>>>> works.
>>>>
>>>> There is an easier work around for image streaming using live block copy
>>>> (mirror approach):
>>>> - Create the dst VM as an empty new COW image of the src (even over
>>>> the non shared storage, use some protocol tag for the src location
>>>> like nbd://original_path/src_file_name)
>>>
>>> Migration and non-shared storage has come up a few times in this
>>> discussion. But both live block copy and image streaming need access
>>> to source and destination - they do not have explicit non-shared
>>> storage support. I think non-shared and using nbd:// is orthogonal to
>>> the discussion. Just want to check that you agree and I haven't
>>> missed something?
>>
>> You're right, I was mainly trying to be as general as possible.
>
> I think there is one important point to consider for using NBD: You
> always see a single image on the NBD client, which could in fact have a
> backing file chain on the source. So bdrv_is_allocated() doesn't work
> over NBD, which becomes interesting when you want to share a backing
> file with the new copy.
What is we'll use iscsi? Will the client have a matching iscsi verb to
detect whether a certain block is in fact a unallocated and we'll be
able to use this for our benefit? Of course it will make the non shared
storage case more complex.
>
> Kevin
>
>
next prev parent reply other threads:[~2011-06-27 9:13 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-14 18:18 [Qemu-devel] [PATCH 00/13] QED image streaming Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 01/13] qemu-config: }, { -> }, { to please checkpatch.pl Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 02/13] block: add -drive copy-on-read=on|off Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 03/13] qed: replace is_write with flags field Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 04/13] qed: extract qed_start_allocating_write() Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 05/13] qed: make qed_aio_write_alloc() reusable Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 06/13] qed: add support for copy-on-read Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 07/13] qed: avoid deadlock on emulated synchronous I/O Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 08/13] qerror: add qerror_from_args() to create qerror objects Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 09/13] block: add bdrv_aio_copy_backing() Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 10/13] qmp: add QMP support for stream commands Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 11/13] block: add -drive stream=on|off Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 12/13] qed: intelligent streaming implementation Stefan Hajnoczi
2011-06-14 18:18 ` [Qemu-devel] [PATCH 13/13] trace: trace bdrv_aio_readv/writev error paths Stefan Hajnoczi
2011-06-15 10:46 ` [Qemu-devel] [PATCH 00/13] QED image streaming Philipp Hahn
2011-06-15 12:18 ` Stefan Hajnoczi
2011-06-16 12:35 ` [Qemu-devel] Image streaming and live block copy (was: [PATCH 00/13] QED image streaming) Kevin Wolf
2011-06-16 12:49 ` [Qemu-devel] Image streaming and live block copy Avi Kivity
2011-06-16 13:08 ` Kevin Wolf
2011-06-16 13:38 ` Avi Kivity
2011-06-16 14:52 ` Marcelo Tosatti
2011-06-16 15:30 ` Stefan Hajnoczi
2011-06-17 12:31 ` Marcelo Tosatti
2011-06-18 9:15 ` Stefan Hajnoczi
2011-06-18 9:17 ` Stefan Hajnoczi
2011-06-19 16:02 ` Dor Laor
2011-06-24 9:28 ` Stefan Hajnoczi
2011-06-26 12:50 ` Dor Laor
2011-06-27 7:48 ` Kevin Wolf
2011-06-27 9:13 ` Dor Laor [this message]
2011-06-17 13:54 ` Marcelo Tosatti
2011-06-17 8:36 ` Kevin Wolf
2011-06-17 8:57 ` Stefan Hajnoczi
2011-06-17 9:22 ` Kevin Wolf
2011-06-17 10:11 ` Stefan Hajnoczi
2011-06-17 12:21 ` Anthony Liguori
2011-06-17 13:04 ` Marcelo Tosatti
2011-06-17 13:50 ` Marcelo Tosatti
2011-06-16 13:10 ` Anthony Liguori
2011-06-16 13:50 ` Kevin Wolf
2011-06-16 14:38 ` [Qemu-devel] Image streaming and live block copy (was: [PATCH 00/13] QED image streaming) Marcelo Tosatti
2011-06-16 14:55 ` Marcelo Tosatti
2011-06-17 8:21 ` [Qemu-devel] Image streaming and live block copy Kevin Wolf
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=4E0849A6.4070009@redhat.com \
--to=dlaor@redhat.com \
--cc=agl@us.ibm.com \
--cc=aliguori@us.ibm.com \
--cc=avi@redhat.com \
--cc=jes.sorensen@redhat.com \
--cc=kwolf@redhat.com \
--cc=mtosatti@redhat.com \
--cc=owasserm@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
--cc=stefanha@linux.vnet.ibm.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).