svn commit: r224217 - in head/sys: amd64/include ia64/include mips/conf sys

[Sergey Kandaurov]

On 21 July 2011 01:48, Pan Tsu <inyaoo at gmail.com> wrote:
> Attilio Rao <attilio at FreeBSD.org> writes:
>
>> Author: attilio
>> Date: Tue Jul 19 13:00:30 2011
>> New Revision: 224217
>> URL: http://svn.freebsd.org/changeset/base/224217
>>
>> Log:
>>   Bump MAXCPU for amd64, ia64 and XLP mips appropriately.
>>   From now on, default values for FreeBSD will be 64 maxiumum supported
>>   CPUs on amd64 and ia64 and 128 for XLP. All the other architectures
>>   seem already capped appropriately (with the exception of sparc64 which
>>   needs further support on jalapeno flavour).
>>
>>   Bump __FreeBSD_version in order to reflect KBI/KPI brekage introduced
>>   during the infrastructure cleanup for supporting MAXCPU > 32. This
>>   covers cpumask_t retiral too.
>>
>>   The switch is considered completed at the present time, so for whatever
>>   bug you may experience that is reconducible to that area, please report
>>   immediately.
>>
>>   Requested by:       marcel, jchandra
>>   Tested by:  pluknet, sbruno
>>   Approved by:        re (kib)
>>
>> Modified:
>>   head/sys/amd64/include/param.h
>>   head/sys/ia64/include/param.h
>>   head/sys/mips/conf/XLP
>>   head/sys/mips/conf/XLP64
>>   head/sys/mips/conf/XLPN32
>>   head/sys/sys/param.h
>>
>> Modified: head/sys/amd64/include/param.h
>> ==============================================================================
>> --- head/sys/amd64/include/param.h    Tue Jul 19 12:41:57 2011        (r224216)
>> +++ head/sys/amd64/include/param.h    Tue Jul 19 13:00:30 2011        (r224217)
>> @@ -65,7 +65,7 @@
>>
>>  #if defined(SMP) || defined(KLD_MODULE)
>>  #ifndef MAXCPU
>> -#define MAXCPU               32
>> +#define MAXCPU               64
>>  #endif
>>  #else
>>  #define MAXCPU               1
>
> Do you plan to bump MEMSTAT_MAXCPU, too?
>
>  $ vmstat -z
>  vmstat: memstat_sysctl_uma: Too many CPUs
>  $ vmstat -m
>  vmstat: memstat_sysctl_malloc: Too many CPUs
>

This is a known pending issue.
MAXCPU is used in libmemstat for internal needs only.
We must get rid of this. libmemstat should get the cpu
available information at runtime and use it to prepare the
structures. libmemstat should not depend on MAXCPU at all.

-- 
wbr,
pluknet