From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: Installing kernel headers in kvm-kmod Date: Thu, 10 Dec 2009 17:01:03 +0200 Message-ID: <4B210D2F.3060607@redhat.com> References: <4B20F27D.3000006@codemonkey.ws> <4B20F914.8020003@siemens.com> <200912101550.22930.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Cc: Jan Kiszka , Anthony Liguori , Marcelo Tosatti , kvm-devel To: Arnd Bergmann Return-path: Received: from mx1.redhat.com ([209.132.183.28]:51198 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751334AbZLJPBC (ORCPT ); Thu, 10 Dec 2009 10:01:02 -0500 In-Reply-To: <200912101550.22930.arnd@arndb.de> Sender: kvm-owner@vger.kernel.org List-ID: On 12/10/2009 04:50 PM, Arnd Bergmann wrote: > On Thursday 10 December 2009, Jan Kiszka wrote: > >> Anthony Liguori wrote: >> >>> QEMU 0.12.0-rc1 does not support KVM >>> https://bugs.launchpad.net/bugs/494500 >>> >>> Boils down to the fact that 1) we don't include kernel headers in qemu >>> (whereas qemu-kvm does) and 2) kvm-kmod does not install those headers >>> on make install. >>> >>> I think we've discussed (2) as being the preferred solution. Does >>> everyone agree with that? Anyone care to volunteer to make the change? :-) >>> >>> >> I've pushed a half-tested approach into kvm-kmod's next branch. Feel >> free to test/fix/enhance it. >> > This would work, but installing to /usr/include/linux/kvm.h will confuse > distro package managers a lot, because that location belongs to the glibc > or libc-linux-headers or some other package already. > > If you want to install the headers from kvm-kmod, I would recommend > doing it in a different path, e.g. /usr/include/kvm-kmod/{linux,asm}. > Maybe even /usr/local/include/kvm-kmod-$version/...., and a symlink /usr/local/include/kvm-kmod. > qemu can then add -I/usr/include/kvm-kmod to it's default include > path and get the kvm-kmod version if that's installed or the distro > version otherwise. > > It may also be useful to do the equivalent of 'make headers_install' > from the kernel, to remove all "#ifdef __KERNEL__" sections and > sparse annotations from the header files, but it should also work > without that. > Well, qemu.git needs __user removed. -- error compiling committee.c: too many arguments to function