Thanks for pointing that out. I made the changes you suggested.
An updated patch is attached.

btw - I'm using qemu-kvm-0.11.0, but this patch would be relevant when running
any version of qemu/qemu-kvm that was released after the upstream merge of the
kvm userspace..

-Steve

Daniel P. Berrange wrote:

On Mon, Nov 16, 2009 at 02:15:28AM -0800, Steve Yarmie wrote

 
+    /* Should explicitly enable KVM if
+     * 1. Guest domain is 'qemu'

Surely 'kvm' here

+     * 2. The qemu binary has the -enable-kvm flag
+     * NOTE: user must be responsible to load the kvm modules
+     */
+    if ((qemuCmdFlags & QEMUD_CMD_FLAG_ENABLE_KVM) &&
+        def->virtType == VIR_DOMAIN_VIRT_QEMU)


Again, this should be VIR_DOMAIN_VIRT_KVM here - 'qemu' is intended
to be pure emulation only


Regards,
Daniel