From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:53991) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QofZy-0001WG-Dr for qemu-devel@nongnu.org; Wed, 03 Aug 2011 13:50:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QofZx-0000RM-Ea for qemu-devel@nongnu.org; Wed, 03 Aug 2011 13:50:02 -0400 Received: from mail-pz0-f43.google.com ([209.85.210.43]:46901) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QofZx-0000R6-62 for qemu-devel@nongnu.org; Wed, 03 Aug 2011 13:50:01 -0400 Received: by pzk1 with SMTP id 1so2197530pzk.16 for ; Wed, 03 Aug 2011 10:50:00 -0700 (PDT) Message-ID: <4E398A44.8060507@codemonkey.ws> Date: Wed, 03 Aug 2011 12:49:56 -0500 From: Anthony Liguori MIME-Version: 1.0 References: <1312326516-10117-1-git-send-email-aliguori@us.ibm.com> <4E389258.6040007@us.ibm.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] migration: remove subsections in fdc and rtl8139 and bump versions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: quintela@redhat.com Cc: Paolo Bonzini , Anthony Liguori , qemu-devel@nongnu.org On 08/03/2011 04:00 AM, Juan Quintela wrote: > Anthony Liguori wrote: >> So my thinking is to be a bit more conservative. If we bump the >> version number for 0.15.0, we make sure that we don't allow new -> old >> migration. We will break old -> new migration, but we can fix that >> (including in the stable series) by adding special handling of the >> previous version. >> >> Fixing new->old is the critical bit here. We can resolve old->new as >> a stable update. > > We are making something that is incompatible. Indeed. > If we don't care about > breaking 0.14 -> 0.15 migration. Just add Paolo version, and drop > altogether the old protocol. It will give us exactly the same result, > new versions work, old versions fail. I don't have a problem with Paolo's new protocol. In fact, I'm strong in favor of applying it to master. But I don't like the idea of adding a new migration protocol with no testing in master before putting it in a release. >> >> This series was just too late for 0.15. I can close to suggesting >> that we delay 0.15 in order to give this time to be tested thoroughly >> but I think my proposal is a reasonable compromise. > > I think it is anything except reasonable. From my point on view (and I > am biased), it is the equivalent of finding a corner case broken on > qcow2 and make all _OLD_ qcow2 images unreadable. It will work, but it > is anything except reasonable. As said, everything uses an fdc. > Furthermore, the _two_ things that you change don't matter in the big > scheme of things. The subsections that fail are the ones that can > appear in the middle of another section. The ones that appear at the > end of a real section work perfectly well with this protocol (a.k.a. the > ones that are broken are IDE), floppy and rtl8139 are ok with current > protocol. I can certainly limit the change to IDE if we think machine, floppy, and rtl8139 are safe. My main concern is fixing the corruption during migration for the release. Once that is fixed, we can revisit compatibility for the stable branch (by introducing a compatibility path for the older version). Regards, Anthony Liguori > > Later, Juan. >