[libvirt] [Qemu-devel] [PATCH] target-i386: Enable "check" mode by default

[Eduardo Habkost]

On Mon, Aug 31, 2015 at 02:20:54PM +0200, Igor Mammedov wrote:
> On Wed, 26 Aug 2015 13:50:25 -0300
> Eduardo Habkost <ehabkost at redhat.com> 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 <ehabkost at redhat.com>
[...]
> Reviewed-by: Igor Mammedov <imammedo at redhat.com>

Thanks! Applied to x86 tree.

-- 
Eduardo