<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> <html> <head> <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#ffffff" text="#000000"> 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: <blockquote cite="mid:20091116113044.GD16871@redhat.com" type="cite"> <pre wrap="">On Mon, Nov 16, 2009 at 02:15:28AM -0800, Steve Yarmie wrote</pre> <blockquote type="cite"> <pre wrap=""> + /* Should explicitly enable KVM if + * 1. Guest domain is 'qemu'</pre> </blockquote> </blockquote> <blockquote cite="mid:20091116113044.GD16871@redhat.com" type="cite"> <blockquote type="cite"> <pre wrap="">Surely 'kvm' here </pre> </blockquote> </blockquote> <blockquote cite="mid:20091116113044.GD16871@redhat.com" type="cite"> <blockquote type="cite"> <pre wrap="">+ * 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) </pre> </blockquote> <pre wrap=""> Again, this should be VIR_DOMAIN_VIRT_KVM here - 'qemu' is intended to be pure emulation only Regards, Daniel </pre> </blockquote> </body> </html>