From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:48514) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Qmmtw-0006Wr-Oj for qemu-devel@nongnu.org; Fri, 29 Jul 2011 09:14:53 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Qmmtv-00063T-R6 for qemu-devel@nongnu.org; Fri, 29 Jul 2011 09:14:52 -0400 Received: from mail-pz0-f43.google.com ([209.85.210.43]:41883) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Qmmtv-00063H-MK for qemu-devel@nongnu.org; Fri, 29 Jul 2011 09:14:51 -0400 Received: by pzk1 with SMTP id 1so6670627pzk.30 for ; Fri, 29 Jul 2011 06:14:51 -0700 (PDT) Message-ID: <4E32B246.6060001@codemonkey.ws> Date: Fri, 29 Jul 2011 08:14:46 -0500 From: Anthony Liguori MIME-Version: 1.0 References: <1309448777-1447-1-git-send-email-pbonzini@redhat.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC PATCH 0/4] Fix subsection ambiguity in the migration format List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: quintela@redhat.com, qemu-devel@nongnu.org, mst@redhat.com On 07/25/2011 04:10 PM, Paolo Bonzini wrote: > On Thu, Jun 30, 2011 at 17:46, Paolo Bonzini wrote: > I have now tested this series (exactly as sent) both by examining > manually the differences between the two formats on the same guest > state, and by a mix of saves/restores (new on new, 0.14 on new > pc-0.14, new pc-0.14 on 0.14; also the same combinations on RHEL). It > always does what is expected. > > Michael Tsirkin objected that the format should be passed as a > parameter in the migrate command. I kind of agree, however since this > is a real bug you would need to bump the default for new machine > types, and this default would still go in the QEMUMachine struct like > I am doing. So I consider the two settings to be orthogonal. Also, > the alternative requires changes to the whole management stack and if > the default is not changed it imposes a broken format unless you > update the management tools. Clearly much less bang for the buck. > > I think this is ready to go into 0.15. The bug happens when migrating > to 0.14 a pc-0.14 machine created with QEMU 0.15 and which has a > floppy. The media changed subsection is almost always included, and > this causes problems when migrating to 0.14 which didn't have any > subsection for the floppy device. While QEMU support for migration to > old version admittedly depends on luck, this isn't true of certain > downstreams :) which would like to have an unambiguous migration > format. I really hate the idea of changing the migration format moments before the release. Since subsections are optional, can't we take the offending subsections, remove them, bump the section version numbers and make the fields required? That "fixes" this issue temporarily without changing the format and we can change the format for 1.0. Regards, Anthony Liguori > > Paolo > >