From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42540) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZYtoG-0003iY-7H for qemu-devel@nongnu.org; Mon, 07 Sep 2015 06:38:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZYtoF-0000cU-2Y for qemu-devel@nongnu.org; Mon, 07 Sep 2015 06:38:00 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36398) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZYtoE-0000cF-R6 for qemu-devel@nongnu.org; Mon, 07 Sep 2015 06:37:58 -0400 References: <1440607825-24504-1-git-send-email-ehabkost@redhat.com> From: Paolo Bonzini Message-ID: <55ED6900.5050506@redhat.com> Date: Mon, 7 Sep 2015 12:37:52 +0200 MIME-Version: 1.0 In-Reply-To: <1440607825-24504-1-git-send-email-ehabkost@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] target-i386: Enable "check" mode by default List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost , qemu-devel@nongnu.org Cc: Bandan Das , Igor Mammedov , Jiri Denemark , =?UTF-8?Q?Andreas_F=c3=a4rber?= , libvir-list@redhat.com On 26/08/2015 18:50, Eduardo Habkost wrote: > Current default behavior of QEMU is to silently disable features that > are not supported by the host when a CPU model is requested in the > command-line. This means that in addition to risking breaking guest ABI > by default, we are silent about it. > > I would like to enable "enforce" by default, but this can easily break > existing production systems because of the way libvirt makes assumptions > about CPU models today (this will change in the future, once QEMU > provide a proper interface for checking if a CPU model is runnable). > > But there's no reason we should be silent about it. So, change > target-i386 to enable "check" mode by default so at least we have some > warning printed to stderr (and hopefully logged somewhere) when QEMU > disables a feature that is not supported by the host system. > > Signed-off-by: Eduardo Habkost > --- > target-i386/cpu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) This adds a warning to TCG with the default qemu32/qemu64 CPU models, due to lack of DE implementation in TCG. It can be fixed before release, so this patch is okay. But please remind me to do it, or implement it yourself. :) Paolo