Re: compiling with kvm-kmod

2014-01-24 Thread Jan Kiszka
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 remains 
> undefined in the kvm module.  Below is the patch that I used to fix 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

> 
> -Jonas
> 
> 
> 
> diff --git a/x86/Kbuild b/x86/Kbuild
> index d75b756..637b3b1 100644
> --- a/x86/Kbuild
> +++ b/x86/Kbuild
> @@ -1,7 +1,7 @@
>  obj-m := kvm.o kvm-intel.o kvm-amd.o
>  kvm-objs := 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.o \
> -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 += assigned-dev.o iommu.o
> 
> 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 making use 
>>> Jan's kvm-kmod repository.  I receive the below error when I attempt to 
>>> compile with the most recent version.  My question is simply if this is 
>>> something anyone is aware of or has any suggestions for before I go poking 
>>> around?  
>>>
>>> I am compiling against a 3.13.0 mainline vanilla kernel and am using 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=`pwd` \
>>> LINUXINCLUDE="-I`pwd`/include -I`pwd`/include/uapi 
>>> -Iinclude \
>>>  -Iinclude/uapi -Iarch/x86/include 
>>> -Iarch/x86/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-compat.h:45,
>>>  from :0:
>>> include/linux/gfp.h: In function ‘gfp_zonelist’:
>>> include/linux/gfp.h:272:2: error: implicit declaration of function 
>>> ‘IS_ENABLED’ [-Werror=implicit-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
>>

-- 
Siemens AG, Corporate Technology, CT RTC ITP SES-DE
Corporate Competence Center Embedded Linux
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: compiling with kvm-kmod

2014-01-24 Thread Jonas Pfoh
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 remains undefined 
in the kvm module.  Below is the patch that I used to fix the issue if it is 
helpful.

-Jonas



diff --git a/x86/Kbuild b/x86/Kbuild
index d75b756..637b3b1 100644
--- a/x86/Kbuild
+++ b/x86/Kbuild
@@ -1,7 +1,7 @@
 obj-m := kvm.o kvm-intel.o kvm-amd.o
 kvm-objs := 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.o \
-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 += assigned-dev.o iommu.o

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 making use 
>> Jan's kvm-kmod repository.  I receive the below error when I attempt to 
>> compile with the most recent version.  My question is simply if this is 
>> something anyone is aware of or has any suggestions for before I go poking 
>> around?  
>>
>> I am compiling against a 3.13.0 mainline vanilla kernel and am using 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=`pwd` \
>> LINUXINCLUDE="-I`pwd`/include -I`pwd`/include/uapi -Iinclude 
>> \
>>  -Iinclude/uapi -Iarch/x86/include 
>> -Iarch/x86/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-compat.h:45,
>>  from :0:
>> include/linux/gfp.h: In function ‘gfp_zonelist’:
>> include/linux/gfp.h:272:2: error: implicit declaration of function 
>> ‘IS_ENABLED’ [-Werror=implicit-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
> 
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: compiling with kvm-kmod

2014-01-23 Thread Jan Kiszka
On 2014-01-23 17:34, Jonas Pfoh wrote:
> Hello,
> 
> I am currently working on a project involving KVM and have been making use 
> Jan's kvm-kmod repository.  I receive the below error when I attempt to 
> compile with the most recent version.  My question is simply if this is 
> something anyone is aware of or has any suggestions for before I go poking 
> around?  
> 
> I am compiling against a 3.13.0 mainline vanilla kernel and am using 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=`pwd` \
> LINUXINCLUDE="-I`pwd`/include -I`pwd`/include/uapi -Iinclude \
>  -Iinclude/uapi -Iarch/x86/include 
> -Iarch/x86/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-compat.h:45,
>  from :0:
> include/linux/gfp.h: In function ‘gfp_zonelist’:
> include/linux/gfp.h:272:2: error: implicit declaration of function 
> ‘IS_ENABLED’ [-Werror=implicit-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

-- 
Siemens AG, Corporate Technology, CT RTC ITP SES-DE
Corporate Competence Center Embedded Linux
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


compiling with kvm-kmod

2014-01-23 Thread Jonas Pfoh
Hello,

I am currently working on a project involving KVM and have been making use 
Jan's kvm-kmod repository.  I receive the below error when I attempt to compile 
with the most recent version.  My question is simply if this is something 
anyone is aware of or has any suggestions for before I go poking around?  

I am compiling against a 3.13.0 mainline vanilla kernel and am using 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=`pwd` \
LINUXINCLUDE="-I`pwd`/include -I`pwd`/include/uapi -Iinclude \
 -Iinclude/uapi -Iarch/x86/include 
-Iarch/x86/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-compat.h:45,
 from :0:
include/linux/gfp.h: In function ‘gfp_zonelist’:
include/linux/gfp.h:272:2: error: implicit declaration of function ‘IS_ENABLED’ 
[-Werror=implicit-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
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html