RE: [Xen-devel] [PATCH] Enlarge default NR_CPUS to 64 for x86_64

["Wei, Gang" <gang.wei@xxxxxxxxx>]

Not sure about whether we can raise it for i386 too. I am a little confused by 
below lines in config.h:

    #ifdef __i386__
    #if NR_CPUS > 32
    #error "Maximum of 32 physical processors supported by Xen on x86_32"
    #endif
    /* Maximum number of virtual CPUs in multi-processor guests. */
    #define MAX_VIRT_CPUS XEN_LEGACY_MAX_VCPUS
    #endif

I don't know why this compiling time error was put here for i386 yet. Any body 
know it?

Jimmy

Keir Fraser wrote:
> Shall we just bump it for i386 too?
> 
> -- Keir
> 
> On 07/08/2009 10:37, "Wei, Gang" <gang.wei@xxxxxxxxx> wrote:
> 
>> By far a lot of systems are with more sockets (>=4), more cores
>> (>=8), meanwhile with HyperThread, so enlarge the default NR_CPUS
>> would easier the build for more threads systems.
>> 
>> Jimmy
>> 
>> -------------------------------------------------------------
>> Enlarge default NR_CPUS to 64 for x86_64.
>> 
>> Signed-off-by: Wei Gang <gang.wei@xxxxxxxxx>
>> 
>> diff -r b9cdcf502aa3 xen/include/asm-x86/config.h
>> --- a/xen/include/asm-x86/config.h Thu Aug 06 11:14:48 2009 +0100
>> +++ b/xen/include/asm-x86/config.h Fri Aug 07 14:54:06 2009 +0800 @@
>>  -49,7 +49,11 @@ #ifdef MAX_PHYS_CPUS
>>  #define NR_CPUS MAX_PHYS_CPUS
>>  #else
>> +#ifdef __i386__
>>  #define NR_CPUS 32
>> +#else
>> +#define NR_CPUS 64
>> +#endif
>>  #endif
>> 
>>  #ifdef __i386__
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel