From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38028) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XW5nP-0005Zj-5X for qemu-devel@nongnu.org; Mon, 22 Sep 2014 11:45:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XW5nG-00024a-3k for qemu-devel@nongnu.org; Mon, 22 Sep 2014 11:44:59 -0400 Received: from mail-wg0-x22b.google.com ([2a00:1450:400c:c00::22b]:52059) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XW5nF-00023Q-TE for qemu-devel@nongnu.org; Mon, 22 Sep 2014 11:44:50 -0400 Received: by mail-wg0-f43.google.com with SMTP id y10so3268705wgg.14 for ; Mon, 22 Sep 2014 08:44:43 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <542043EA.6060104@redhat.com> Date: Mon, 22 Sep 2014 17:44:42 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1411310339-27733-1-git-send-email-alex@alex.org.uk> <1411310339-27733-3-git-send-email-alex@alex.org.uk> <20140922113655.GI14882@redhat.com> <20140922114247.GE31563@redhat.com> <878ulb7k54.fsf@blackfin.pond.sub.org> In-Reply-To: <878ulb7k54.fsf@blackfin.pond.sub.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [libvirt] [PATCH v3 2/2] Add configure option --enable-pc-1-0-qemu-kvm List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster , "Daniel P. Berrange" Cc: Ryan Harper , "quintela@redhat.com" , Libvirt , Serge Hallyn , Alexander Graf , qemu-devel@nongnu.org, "Michael S. Tsirkin" , Alex Bligh , Amit Shah , =?windows-1252?Q?Andreas_F=E4rber?= , "Serge E. Hallyn" Il 22/09/2014 17:24, Markus Armbruster ha scritto: > You're right. However, this particular horse left the barn a long time > ago: the pc-* machine types differ in qemu-kvm and upstream QEMU. > > Sure, when qemu-kvm was merged back into QEMU, its machine type variants > were dropped. But they live on in various downstreams that just like > QEMU had to pick between compatibility with upstream QEMU and qemu-kvm, > but unlike QEMU picked compatibility with qemu-kvm. And fixing this should have been downstream's job. Fedora did it at the time, RHEL7 did it. Ubuntu didn't, and probably neither did Debian. This patch singles out pc-1.0 just because it used to be the default in Ubuntu 12.04. So basically it's making upstream carry the burden of a decision of the Ubuntu folks. It's understandable that Alex disagrees with the decision, but nevertheless it's not something that upstream should agree with. Also, another horse that has left the barn: it's already too late to apply this patch to upstream Ubuntu. If you do that, any machine created with 12.04 and reused with 14.04 will fail to migrate to another 14.04 machine that includes this patch, as I understand it. So as things stand, I don't see a reason to apply this patch upstream. Paolo