[libvirt][PATCH] running libvirt on the latest kvm snapshot

[Daniel P. Berrange]

On Fri, Aug 07, 2009 at 01:57:34PM +0200, Chris Lalancette wrote:
> Anton Protopopov wrote:
> > Hi.
> > 
> > Libvirt tryes to extract kvm version from the string
> >   "QEMU PC emulator version <major>.<minor>.<micro> (kvm-<version>)..."
> > For examle,
> >   # kvm -help | head -1
> >   QEMU PC emulator version 0.10.0 (kvm-85), Copyright (c) 2003-2008
> > Fabrice Bellard
> > 
> > The problem occures if you are trying to run it on the devel version of kvm:
> >   # kvm -help | head -1
> >   QEMU PC emulator version 0.10.0 (kvm-devel), Copyright (c) 2003-2008
> > Fabrice Bellard
> > The virParseNumber() obvioulsy fails on the "devel" string here. I
> > propose to setup
> > `kvm_version' to 1000 (that's enough, I think) in that case, so all
> > checks, such as
> >   if (kvm_version > 74) ...
> > will pass.
> 
> Yeah, I've run into this too.  The problem with just setting kvm_version to a
> huge number is that if you built a kvm development snapshot a long time in the
> past, and you run it now, then you will be turning on features your old
> development snapshot didn't necessarily have.  On the other hand, if you are
> running development snapshots, you get to keep both pieces when things break.
> Maybe we can set the kvm_version number very high like you say, but also dump a
> warning message out to the logfile saying this is a development version so the
> poor sap who has to debug something like this has a fighting chance.  Any other
> opinions?

Fix kvm to include a relevant version number instead of just the 'devel'

eg, Instead of

   QEMU PC emulator version 0.10.0 (kvm-devel), Copyright (c) 2003-2008

have

   QEMU PC emulator version 0.10.0 (kvm-85-devel), Copyright (c) 2003-2008

So that its clear this development release, is the one immediately
following on from the kvm-85 stable release. That would let us 
automatically enable the features upto & including kvm-85 which
would at least make it functional, albeit without letting us use
the new features.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|