kvm-29 and linux 2.6.22.1-cfs-v19

kvm-29 and linux 2.6.22.1-cfs-v19

[Chris Clayton]

Hi,

I'm building kvm-29 on a system that is running linux-2.6.22.1-cfs-v19. When 
the modules are built I get warnings about set_64bit:

  CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/vmx-debug.o
  CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm_main.o
  CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.o
/home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c: In function 'set_shadow_pte':
/home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c:202: warning: implicit 
declaration of function 'set_64bit'
  CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/x86_emulate.o
  CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/anon_inodes.o
  LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.o
  LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.o
  LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.o
  Building modules, stage 2.
  MODPOST 3 modules
WARNING: "set_64bit" [/home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko] 
undefined!
  CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.mod.o
  CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.mod.o
  CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.mod.o
  LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.ko
  LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.ko
  LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko
m
Because set_64bit is undefined, kvm.ko (and kvm-intel.ko) can't be loaded.

I've found the set_64bit in the kernel sources - it's a macro in 
asm-i386/cmpxchg.h. The problem is that the macro(and some realted macros and 
definitions are guarded by "#ifdef  CONFIG_X86_CMPXCHG64". Since that isn't 
defined for my kernel, set_64bit is not defined either.

If I comment the guarding #ifdef (and its partner #endif) kvm builds fine and 
everything seems to work OK.- the modules load and Damn Small Linux and 
FreeBSD run fine.

Any advice would be much appreciated and more than happy to test patches.

I'm not subscribed, so please cc me.

Thanks

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

Re: kvm-29 and linux 2.6.22.1-cfs-v19

[Anthony Liguori]

Chris Clayton wrote:
> Hi,
>
> I'm building kvm-29 on a system that is running linux-2.6.22.1-cfs-v19. When 
> the modules are built I get warnings about set_64bit:
>
>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/vmx-debug.o
>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm_main.o
>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.o
> /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c: In function 'set_shadow_pte':
> /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c:202: warning: implicit 
> declaration of function 'set_64bit'
>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/x86_emulate.o
>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/anon_inodes.o
>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.o
>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.o
>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.o
>   Building modules, stage 2.
>   MODPOST 3 modules
> WARNING: "set_64bit" [/home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko] 
> undefined!
>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.mod.o
>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.mod.o
>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.mod.o
>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.ko
>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.ko
>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko
> m
> Because set_64bit is undefined, kvm.ko (and kvm-intel.ko) can't be loaded.
>
> I've found the set_64bit in the kernel sources - it's a macro in 
> asm-i386/cmpxchg.h. The problem is that the macro(and some realted macros and 
> definitions are guarded by "#ifdef  CONFIG_X86_CMPXCHG64". Since that isn't 
> defined for my kernel, set_64bit is not defined either.
>   

KVM depends on CONFIG_X86_CMPXCHG64 which also depends on CONFIG_X86_PAE.

Regards,

Anthony Liguori

> If I comment the guarding #ifdef (and its partner #endif) kvm builds fine and 
> everything seems to work OK.- the modules load and Damn Small Linux and 
> FreeBSD run fine.
>
> Any advice would be much appreciated and more than happy to test patches.
>
> I'm not subscribed, so please cc me.
>
> Thanks
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> kvm-devel mailing list
> kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> https://lists.sourceforge.net/lists/listinfo/kvm-devel
>
>   


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

Re: kvm-29 and linux 2.6.22.1-cfs-v19

[Chris Clayton]

On Saturday 14 July 2007, Anthony Liguori wrote:
> Chris Clayton wrote:
> > Hi,
> >
> > I'm building kvm-29 on a system that is running linux-2.6.22.1-cfs-v19.
> > When the modules are built I get warnings about set_64bit:
> >
> >   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/vmx-debug.o
> >   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm_main.o
> >   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.o
> > /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c: In function
> > 'set_shadow_pte': /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c:202:
> > warning: implicit declaration of function 'set_64bit'
> >   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/x86_emulate.o
> >   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/anon_inodes.o
> >   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.o
> >   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.o
> >   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.o
> >   Building modules, stage 2.
> >   MODPOST 3 modules
> > WARNING: "set_64bit" [/home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko]
> > undefined!
> >   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.mod.o
> >   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.mod.o
> >   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.mod.o
> >   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.ko
> >   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.ko
> >   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko
> > m
> > Because set_64bit is undefined, kvm.ko (and kvm-intel.ko) can't be
> > loaded.
> >
> > I've found the set_64bit in the kernel sources - it's a macro in
> > asm-i386/cmpxchg.h. The problem is that the macro(and some realted macros
> > and definitions are guarded by "#ifdef  CONFIG_X86_CMPXCHG64". Since that
> > isn't defined for my kernel, set_64bit is not defined either.
>
> KVM depends on CONFIG_X86_CMPXCHG64 which also depends on CONFIG_X86_PAE.
>

To select that I have to specify (in make menuconfig) that I have more than 4 
gigabytes of RAM, when actually I have 2 gigabytes. It's the only way I can 
find to get CONFIG_X86_PAE selected, but means I must ignore the help text 
offered around CONFIG_HIGHMEM4G/64G.

Have I got that right, please? 

Thanks

Chris

> Regards,
>
> Anthony Liguori
>
> > If I comment the guarding #ifdef (and its partner #endif) kvm builds fine
> > and everything seems to work OK.- the modules load and Damn Small Linux
> > and FreeBSD run fine.
> >
> > Any advice would be much appreciated and more than happy to test patches.
> >
> > I'm not subscribed, so please cc me.
> >
> > Thanks
> >
> > -------------------------------------------------------------------------
> > This SF.net email is sponsored by DB2 Express
> > Download DB2 Express C - the FREE version of DB2 express and take
> > control of your XML. No limits. Just data. Click to get it now.
> > http://sourceforge.net/powerbar/db2/
> > _______________________________________________
> > kvm-devel mailing list
> > kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
> > https://lists.sourceforge.net/lists/listinfo/kvm-devel



-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

Re: kvm-29 and linux 2.6.22.1-cfs-v19

[Anthony Liguori]

Chris Clayton wrote:
> On Saturday 14 July 2007, Anthony Liguori wrote:
>   
>> Chris Clayton wrote:
>>     
>>> Hi,
>>>
>>> I'm building kvm-29 on a system that is running linux-2.6.22.1-cfs-v19.
>>> When the modules are built I get warnings about set_64bit:
>>>
>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/vmx-debug.o
>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm_main.o
>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.o
>>> /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c: In function
>>> 'set_shadow_pte': /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c:202:
>>> warning: implicit declaration of function 'set_64bit'
>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/x86_emulate.o
>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/anon_inodes.o
>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.o
>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.o
>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.o
>>>   Building modules, stage 2.
>>>   MODPOST 3 modules
>>> WARNING: "set_64bit" [/home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko]
>>> undefined!
>>>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.mod.o
>>>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.mod.o
>>>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.mod.o
>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.ko
>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.ko
>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko
>>> m
>>> Because set_64bit is undefined, kvm.ko (and kvm-intel.ko) can't be
>>> loaded.
>>>
>>> I've found the set_64bit in the kernel sources - it's a macro in
>>> asm-i386/cmpxchg.h. The problem is that the macro(and some realted macros
>>> and definitions are guarded by "#ifdef  CONFIG_X86_CMPXCHG64". Since that
>>> isn't defined for my kernel, set_64bit is not defined either.
>>>       
>> KVM depends on CONFIG_X86_CMPXCHG64 which also depends on CONFIG_X86_PAE.
>>
>>     
>
> To select that I have to specify (in make menuconfig) that I have more than 4 
> gigabytes of RAM, when actually I have 2 gigabytes. It's the only way I can 
> find to get CONFIG_X86_PAE selected, but means I must ignore the help text 
> offered around CONFIG_HIGHMEM4G/64G.
>
> Have I got that right, please? 
>   

Enabling HIGHMEM64G enables the use of PAE.  Any processor that support 
virtualization supports PAE and there's very little harm to enabling it 
even if you have less than 4G of memory.

Regards,

Anthony Liguori

> Thanks
>
> Chris
>
>   
>> Regards,
>>
>> Anthony Liguori
>>
>>     
>>> If I comment the guarding #ifdef (and its partner #endif) kvm builds fine
>>> and everything seems to work OK.- the modules load and Damn Small Linux
>>> and FreeBSD run fine.
>>>
>>> Any advice would be much appreciated and more than happy to test patches.
>>>
>>> I'm not subscribed, so please cc me.
>>>
>>> Thanks
>>>
>>> -------------------------------------------------------------------------
>>> This SF.net email is sponsored by DB2 Express
>>> Download DB2 Express C - the FREE version of DB2 express and take
>>> control of your XML. No limits. Just data. Click to get it now.
>>> http://sourceforge.net/powerbar/db2/
>>> _______________________________________________
>>> kvm-devel mailing list
>>> kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
>>> https://lists.sourceforge.net/lists/listinfo/kvm-devel
>>>       
>
>
>
>   


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

Re: kvm-29 and linux 2.6.22.1-cfs-v19

[Avi Kivity]

Anthony Liguori wrote:
> Chris Clayton wrote:
>   
>> On Saturday 14 July 2007, Anthony Liguori wrote:
>>   
>>     
>>> Chris Clayton wrote:
>>>     
>>>       
>>>> Hi,
>>>>
>>>> I'm building kvm-29 on a system that is running linux-2.6.22.1-cfs-v19.
>>>> When the modules are built I get warnings about set_64bit:
>>>>
>>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/vmx-debug.o
>>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm_main.o
>>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.o
>>>> /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c: In function
>>>> 'set_shadow_pte': /home/users/chris/rpm/BUILD/kvm-29/kernel/mmu.c:202:
>>>> warning: implicit declaration of function 'set_64bit'
>>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/x86_emulate.o
>>>>   CC [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/anon_inodes.o
>>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.o
>>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.o
>>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.o
>>>>   Building modules, stage 2.
>>>>   MODPOST 3 modules
>>>> WARNING: "set_64bit" [/home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko]
>>>> undefined!
>>>>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.mod.o
>>>>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.mod.o
>>>>   CC      /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.mod.o
>>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-amd.ko
>>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm-intel.ko
>>>>   LD [M]  /home/users/chris/rpm/BUILD/kvm-29/kernel/kvm.ko
>>>> m
>>>> Because set_64bit is undefined, kvm.ko (and kvm-intel.ko) can't be
>>>> loaded.
>>>>
>>>> I've found the set_64bit in the kernel sources - it's a macro in
>>>> asm-i386/cmpxchg.h. The problem is that the macro(and some realted macros
>>>> and definitions are guarded by "#ifdef  CONFIG_X86_CMPXCHG64". Since that
>>>> isn't defined for my kernel, set_64bit is not defined either.
>>>>       
>>>>         
>>> KVM depends on CONFIG_X86_CMPXCHG64 which also depends on CONFIG_X86_PAE.
>>>
>>>     
>>>       
>> To select that I have to specify (in make menuconfig) that I have more than 4 
>> gigabytes of RAM, when actually I have 2 gigabytes. It's the only way I can 
>> find to get CONFIG_X86_PAE selected, but means I must ignore the help text 
>> offered around CONFIG_HIGHMEM4G/64G.
>>
>> Have I got that right, please? 
>>   
>>     
>
> Enabling HIGHMEM64G enables the use of PAE.  Any processor that support 
> virtualization supports PAE and there's very little harm to enabling it 
> even if you have less than 4G of memory.
>
>   

While that's true, and while pae has an additional benefit (nx support 
on most processors), there's no reason not to support kvm on non-pae.  
set_64bit() is not intrinsically dependent on pae, rather it's the other 
way round.

The following commit causes the issue (note that only newer kvm needs 
set_64bit(); 2.6.22 is fine):

commit 9d9bbd4d247a674deb43565582151acdc22e90d1
Author: Andi Kleen <ak-l3A5Bk7waGM@public.gmane.org>
Date:   Sat Jun 23 02:29:23 2007 +0200

    i386: Make CMPXCHG64 only dependent on PAE

    It is only used for PAE kernels in set_64bit.

    The problem is that due to a old Windows bug many CPUs need magic MSRs
    to enable CMPXCHG64, and we can't do that nicely early enough before
    it is potentially used.

    But since we only need it in PAE kernels so only force the checking
    for CMPXCHG65 with PAE.

    This fixes a boot failure on Transmeta Crusoe

    Signed-off-by: Andi Kleen <ak-l3A5Bk7waGM@public.gmane.org>
    Signed-off-by: Linus Torvalds <torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>

diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig
index 8770a5d..d2f6a24 100644
--- a/arch/i386/Kconfig
+++ b/arch/i386/Kconfig
@@ -541,7 +541,7 @@ config HIGHMEM4G

 config HIGHMEM64G
        bool "64GB"
-       depends on X86_CMPXCHG64
+       depends on !M386 && !M486
        help
          Select this if you have a 32-bit processor and more than 4
          gigabytes of physical RAM.
diff --git a/arch/i386/Kconfig.cpu b/arch/i386/Kconfig.cpu
index d7f6fb0..5c95ceb 100644
--- a/arch/i386/Kconfig.cpu
+++ b/arch/i386/Kconfig.cpu
@@ -299,7 +299,7 @@ config X86_POPAD_OK

 config X86_CMPXCHG64
        bool
-       depends on !M386 && !M486
+       depends on X86_PAE
        default y



Andi, will a patch to have X86_CMPXCHG64 depend on !M386 && !M486, 
default to n, and be selected by HIGHMEM64G and KVM be acceptable?


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/