Re: [Qemu-devel] [PATCH] define qemukvm-1.2 machine type

[Eduardo Habkost]

On Thu, Dec 13, 2012 at 03:35:03PM -0600, Anthony Liguori wrote:
> Eduardo Habkost <address@hidden> writes:
> 
> > On Thu, Dec 13, 2012 at 09:35:13AM +0100, Paolo Bonzini wrote:
> >> Il 12/12/2012 23:39, Marcelo Tosatti ha scritto:
> >> > 
> >> > To allow migration from qemu-kvm-1.2, where cirrus device has 16 
> >> > megabytes
> >> > of RAM.
> >> > 
> >> > Signed-off-by: Marcelo Tosatti <address@hidden>
> >> 
> >> A similar patch would be needed for all machine types though, no?
> >> 
> >> I think distros that used to ship qemu-kvm should just change the
> >> default just like for the acpi_piix4.c change.
> >
> > Maybe we could provide a --with-qemu-kvm-compat configure flag to
> > them?
> 
> I think that defeats the purpose of a single binary.
> 
> I think it would be better for the distros to have a qemu-kvm script
> that was:
> 
> /usr/libexec/qemu-kvm:
> 
> #!/bin/sh
> 
> qemu-system-x86_64 -enable-qemu-kvm-compat "$@"

That would be even better. I proposed a configure flag because I
understood (maybe incorrectly) that Paolo proposed a build-time default
change.

-- 
Eduardo