From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=43194 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PulKf-0007t7-Cm for qemu-devel@nongnu.org; Wed, 02 Mar 2011 07:39:10 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PulKd-0005mo-VB for qemu-devel@nongnu.org; Wed, 02 Mar 2011 07:39:09 -0500 Received: from mail-vw0-f45.google.com ([209.85.212.45]:55795) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PulKd-0005md-Rs for qemu-devel@nongnu.org; Wed, 02 Mar 2011 07:39:07 -0500 Received: by vws19 with SMTP id 19so5614142vws.4 for ; Wed, 02 Mar 2011 04:39:07 -0800 (PST) Message-ID: <4D6E3A65.7090502@codemonkey.ws> Date: Wed, 02 Mar 2011 07:39:01 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] Re: [patch 2/3] Add support for live block copy References: <20110222170004.808373778@redhat.com> <4D642181.4080509@codemonkey.ws> <20110222210735.GA9372@amt.cnet> <4D64266A.3060106@codemonkey.ws> <20110222230935.GA11082@amt.cnet> <4D644343.4050800@codemonkey.ws> <4D65051A.6070707@redhat.com> <4D651B20.70405@codemonkey.ws> <4D652852.60505@redhat.com> <4D652F73.3000305@codemonkey.ws> <4D65324A.5080408@redhat.com> <4D65359E.3040008@codemonkey.ws> <4D65416D.8040803@redhat.com> <4D656B97.5030301@codemonkey.ws> <4D661CB8.6010305@redhat.com> <4D667287.9010005@codemonkey.ws> <4D6677BE.2030009@redhat.com> <4D669C46.40909@codemonkey.ws> <4D6A150B.8030205@redhat.com> <4D6A58E0.9020607@codemonkey.ws> <4D6A6E38.4030700@redhat.com> <4D6A8CC9.4090304@codemonkey.ws> <4D6B5EFA.8060106@redhat.com> <4D6B98FD.7020103@codemonkey.ws> <4D6BA16A.2020204@redhat.com> <4D6BDFA1.3000100@redhat.com> <4D6CB556.5060401@redhat.com> In-Reply-To: <4D6CB556.5060401@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: dlaor@redhat.com Cc: Jes.Sorensen@redhat.com, Marcelo Tosatti , Avi Kivity , qemu-devel@nongnu.org On 03/01/2011 03:59 AM, Dor Laor wrote: > On 02/28/2011 08:12 PM, Anthony Liguori wrote: >> >> On Feb 28, 2011 11:47 AM, "Avi Kivity" > > wrote: >> > >> > On 02/28/2011 07:33 PM, Anthony Liguori wrote: >> >> >> >> >> >> > >> >> > You're just ignoring what I've written. >> >> >> >> No, you're just impervious to my subtle attempt to refocus the >> discussion on solving a practical problem. >> >> >> >> There's a lot of good, reasonably straight forward changes we can >> make that have a high return on investment. >> >> >> > >> > Is making qemu the authoritative source of configuration information >> a straightforward change? Is the return on it high? Is the >> investment low? >> >> I think this is where we fundamentally disagree. My position is that >> QEMU is already the authoritative source. Having a state file doesn't >> change anything. >> >> Do a hot unplug of a network device with upstream libvirt with acpiphp >> unloaded, consult libvirt and then consult the monitor to see who has >> the right view of the guests config. >> >> To me, that's the definition of authoritative. >> >> > "No" to all three (ignoring for the moment whether it is good or not, >> which we were debating). >> > >> > >> >> The only suggestion I'm making beyond Marcelo's original patch is >> that we use a structured format and that we make it possible to use the >> same file to solve this problem in multiple places. >> >> >> > >> > No, you're suggesting a lot more than that. >> >> That's exactly what I'm suggesting from a technical perspective. >> >> >> I don't think this creates a fundamental break in how management >> tools interact with QEMU. I don't think introducing RAID support in the >> block layer is a reasonable alternative. >> >> >> >> >> > >> > Why not? >> >> Because its a lot of complexity and code that can go wrong while only >> solving the race for one specific case. Not to mention that we double >> the iop rate. >> >> > Something that avoids the whole state thing altogether: >> > >> > - instead of atomically switching when live copy is done, keep on >> issuing writes to both the origin and the live copy >> > - issue a notification to management >> > - management receives the notification, and issues an atomic blockdev >> switch command >> >> > this is really the RAID-1 solution but without the state file (credit >> Dor). An advantage is that there is no additional latency when trying >> to catch up to the dirty bitmap. >> >> It still suffers from the two generals problem. You cannot solve this >> without making one node reliable and that takes us back to it being >> either QEMU (posted event and state file) or the management tool (sync >> event). > > It is safe w/o a state file by changing the basic live copy algorithm: > > 1. Live copy in progress stage > Once live copy command is issued, a dirty bit map is created for > tracking. There is a single pass over the entire image where we copy > blocks from the src to the dst. > > Write commands for blocks that were already copied will be done > twice for the src and dst. > > Once the full copy single pass ends, we trigger a QMP event that > this stage can end. > > The live copy stage keeps running till the management issue a switch > command. When it will happen, the switch is immediate and no need to > copy additional blocks (but flush pending IOs). > > 2. Management sends a switch command. > Qemu stops the doubling the IO and switches to the destination. > End. Here is where your race is: 2. Management sends a switch command 3. QEMU receives switch command 4. QEMU stops doubling IO and switches to the destination 5. QEMU sends acknowledgement of switch command 6. Management receives acknowledge of switch command 7. Management changes internal state definition to reflect the new destination If QEMU or the management tool crashes after step 4 and before step 6, when the management tool restarts QEMU with the source image, data loss will have occurred (and potentially corruption if a flush had happened). This all boils down to the Two Generals Problem[1]. It's simply not fixable without making one end reliable and that means that someone needs to fsync() something *after* the switchover happens but before the first write happens. That can be QEMU (Avi's RAID proposal and my state file proposal) or it can be the management tool (if we introduce synchronous events). [1] http://en.wikipedia.org/wiki/Two_Generals%27_Problem Regards, Anthony Liguori