From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: Re: compiling with kvm-kmod Date: Fri, 24 Jan 2014 14:03:07 +0100 Message-ID: <52E2648B.7090207@siemens.com> References: <52E14495.50501@sec.in.tum.de> <52E15D28.3060101@siemens.com> <52E25EFB.2000505@sec.in.tum.de> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE To: Jonas Pfoh , "kvm@vger.kernel.org" Return-path: Received: from thoth.sbs.de ([192.35.17.2]:51339 "EHLO thoth.sbs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752480AbaAXNDS (ORCPT ); Fri, 24 Jan 2014 08:03:18 -0500 In-Reply-To: <52E25EFB.2000505@sec.in.tum.de> Sender: kvm-owner@vger.kernel.org List-ID: On 2014-01-24 13:39, Jonas Pfoh wrote: > Thanks for the quick fix, I found that the modules build, but I still= have a problem loading the module due to the fact that kvm_vfio_ops re= mains undefined in the kvm module. Below is the patch that I used to f= ix the issue if it is helpful. Yes, indeed - can you send it as proper patch (subject, brief reason, signed-off)? Then I could merge directly. Thanks, Jan >=20 > -Jonas >=20 > ---- >=20 > diff --git a/x86/Kbuild b/x86/Kbuild > index d75b756..637b3b1 100644 > --- a/x86/Kbuild > +++ b/x86/Kbuild > @@ -1,7 +1,7 @@ > obj-m :=3D kvm.o kvm-intel.o kvm-amd.o > kvm-objs :=3D kvm_main.o x86.o mmu.o emulate.o irq.o i8259.o pmu.o \ > lapic.o ioapic.o preempt.o i8254.o coalesced_mmio.o irq_comm= =2Eo \ > - eventfd.o compat-x86.o async_pf.o cpuid.o irqchip.o \ > + eventfd.o compat-x86.o async_pf.o cpuid.o irqchip.o vfio.o\ > ../external-module-compat.o > ifeq ($(CONFIG_IOMMU_API)$(CONFIG_PCI),yy) > kvm-objs +=3D assigned-dev.o iommu.o >=20 > On 23.01.2014 19:19, Jan Kiszka wrote: >> On 2014-01-23 17:34, Jonas Pfoh wrote: >>> Hello, >>> >>> I am currently working on a project involving KVM and have been mak= ing use Jan's kvm-kmod repository. I receive the below error when I at= tempt to compile with the most recent version. My question is simply i= f this is something anyone is aware of or has any suggestions for befor= e I go poking around? =20 >>> >>> I am compiling against a 3.13.0 mainline vanilla kernel and am usin= g the master branch (3d923a3) of Jan's kvm-kmod repo which seems to be = syncing from kvm commit 7650b68. >>> >>> Thanks and regards, >>> Jonas Pfoh >>> >>> make -C /lib/modules/3.13.0/build M=3D`pwd` \ >>> LINUXINCLUDE=3D"-I`pwd`/include -I`pwd`/include/uap= i -Iinclude \ >>> -Iinclude/uapi -Iarch/x86/include -Iarch/x= 86/include/uapi \ >>> -Iinclude/generated/uapi -Iarch/x86/include= /generated \ >>> -Iarch/x86/include/generated/uapi \ >>> -I`pwd`/include-compat -I`pwd`/x86 \ >>> -include include/generated/autoconf.h \ >>> -include `pwd`/x86/external-module-compat.h= " \ >>> "$@" >>> make[1]: Entering directory `/usr/src/linux-3.13' >>> CC [M] /local/repos/kvm-kmod/x86/svm.o >>> In file included from include/linux/device.h:29:0, >>> from include/linux/node.h:17, >>> from include/linux/cpu.h:16, >>> from /local/repos/kvm-kmod/x86/../external-module-= compat-comm.h:15, >>> from /local/repos/kvm-kmod/x86/external-module-com= pat.h:45, >>> from :0: >>> include/linux/gfp.h: In function =91gfp_zonelist=92: >>> include/linux/gfp.h:272:2: error: implicit declaration of function = =91IS_ENABLED=92 [-Werror=3Dimplicit-function-declaration] >>> if (IS_ENABLED(CONFIG_NUMA) && unlikely(flags & __GFP_THISNODE)) >>> ^ >>> cc1: some warnings being treated as errors >>> make[3]: *** [/local/repos/kvm-kmod/x86/svm.o] Error 1 >>> make[2]: *** [/local/repos/kvm-kmod/x86] Error 2 >>> make[1]: *** [_module_/local/repos/kvm-kmod] Error 2 >>> make[1]: Leaving directory `/usr/src/linux-3.13' >>> make: *** [all] Error 2 >> >> That's likely a kernel issue: not all required headers are pulled by= gfp.h. >> >> It's worked around now with 2b06046. >> >> Thanks, >> Jan >> --=20 Siemens AG, Corporate Technology, CT RTC ITP SES-DE Corporate Competence Center Embedded Linux