From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34085) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xafnx-0003dT-G3 for qemu-devel@nongnu.org; Sun, 05 Oct 2014 03:00:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xafnn-0007mf-GC for qemu-devel@nongnu.org; Sun, 05 Oct 2014 03:00:29 -0400 Received: from mail-wg0-x22c.google.com ([2a00:1450:400c:c00::22c]:41070) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xafnn-0007lb-8J for qemu-devel@nongnu.org; Sun, 05 Oct 2014 03:00:19 -0400 Received: by mail-wg0-f44.google.com with SMTP id y10so4265541wgg.27 for ; Sun, 05 Oct 2014 00:00:17 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <5430EC7B.2070009@redhat.com> Date: Sun, 05 Oct 2014 09:00:11 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1411414496-46245-1-git-send-email-alex@alex.org.uk> <1411414496-46245-2-git-send-email-alex@alex.org.uk> <20140928153058.GA4994@redhat.com> <87fvfavrho.fsf@blackfin.pond.sub.org> In-Reply-To: <87fvfavrho.fsf@blackfin.pond.sub.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH v4] Add machine parameter qemu-kvm-migration for live migrate compatibility with qemu-kvm List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster , Alex Bligh Cc: Ryan Harper , Serge Hallyn , "quintela@redhat.com" , Libvirt , Serge Hallyn , "qemu-devel@nongnu.org" , Alexander Graf , "Michael S. Tsirkin" , Cole Robinson , Amit Shah , Bruce Rogers , =?windows-1252?Q?Andreas_F=E4rber?= , "Serge E. Hallyn" Il 29/09/2014 09:02, Markus Armbruster ha scritto: >> If you were just objecting to the fact that pc-1.0 was made to >> be an alias of either one or the other at compile time, simply >> drop the second patch of the v2 patchset. I was objecting to making pc-1.0 special. There's nothing special in pc-1.0, other machine types also had differences between qemu-kvm and qemu. And I do not think that upstream has any reason to make pc-1.0 special. So, if Ubuntu is okay with breaking pc-1.0 migration from 14.04-old to 14.04-new, the right thing to do is simply that Ubuntu makes its pc-1.0 machine type the qemu-kvm one. No new machine types, no aliases, no anything. For upstream, the option is acceptable because that one applies just as well to other types than 1.0. Other distros included 0.15 or 1.2, and can use the option as well. >> If we have a new machine type, I don't /think/ I need the early_init >> thing at all (I may be wrong about that). You can add a new property to the machine, and do the early_init work in the property setter, I think. > I also prefer a new machine type. > > Ideally, the management application understands that there are two > incompatibile versions QEMU (upstream and old qemu-kvm), and how to map > their machine types to current QEMU's. That would mean patching the Ubuntu libvirt, right? At this point it's simpler to just patch Ubuntu QEMU to do what you write here: > If that's not practical, then downstream can still alias the machine > types around to make things just work in the most important downstream > scenarios. The most important upstream scenario is QEMU <-> QEMU, of > course. I'm not sure why this patch is of any interest upstream... Paolo