All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dor Laor <dlaor@redhat.com>
To: Stefan Hajnoczi <stefanha@gmail.com>
Cc: Kevin Wolf <kwolf@redhat.com>,
	Anthony Liguori <aliguori@us.ibm.com>,
	Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
	jes sorensen <jes.sorensen@redhat.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	qemu-devel@nongnu.org, Avi Kivity <avi@redhat.com>,
	Adam Litke <agl@us.ibm.com>
Subject: Re: [Qemu-devel] Image streaming and live block copy
Date: Sun, 26 Jun 2011 15:50:19 +0300	[thread overview]
Message-ID: <4E072B0B.4010105@redhat.com> (raw)
In-Reply-To: <BANLkTi=vxpTX82ne_uLoqhm5pbxKkWbUuw@mail.gmail.com>

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.

>
>>   - Run the usual live block copy of src image (master read only OS
>>     template) to the destination.
>>     - Use a -src-read-only flag that will make the copy skip the src
>>       writing.
>>
>> Voila - no duplicate writes, crash recovery works since we reference the
>> original image and we share the code.
>
> So the running guest is using the destination image since the source
> is read-only?

Yes. The image will run on the destination not because of the source in 
RO state but because that is what we look for.

>
> This approach makes sense to me.
>
> Stefan
>
>
>

  reply	other threads:[~2011-06-26 12:50 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 [this message]
2011-06-27  7:48                       ` Kevin Wolf
2011-06-27  9:13                         ` Dor Laor
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=4E072B0B.4010105@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=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 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.