Date: Thu, 21 Jul 2011 14:19:09 +0400 From: Sergey Kandaurov <pluknet@freebsd.org> To: Attilio Rao <attilio@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Sean Bruno <seanbru@yahoo-inc.com>, Pan Tsu <inyaoo@gmail.com> Subject: Re: svn commit: r224217 - in head/sys: amd64/include ia64/include mips/conf sys Message-ID: <CAE-mSOJ87c4V_Py6-dMG2wqcnN78omgnHeupkdY7nwEf-dmntw@mail.gmail.com> In-Reply-To: <CAJ-FndB5m5%2BzxO=5eyKXBdzHiSPBDGoSPeXNTkp1fdB3EQZZ3A@mail.gmail.com> References: <201107191300.p6JD0UU7073447@svn.freebsd.org> <86zkk8pqrn.fsf@gmail.com> <CAJ-FndB5m5%2BzxO=5eyKXBdzHiSPBDGoSPeXNTkp1fdB3EQZZ3A@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 21 July 2011 14:14, Attilio Rao <attilio@freebsd.org> wrote: > 2011/7/20 Pan Tsu <inyaoo@gmail.com>: >> Attilio Rao <attilio@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 >> >> $ sysctl kern. | grep smp.\*cpus >> kern.smp.maxcpus: 64 >> kern.smp.cpus: 2 >> > > Jeeeez, we seriously need to fix this getting rid of the static values. > > Anyway, can you try the following patch?: > http://www.freebsd.org/~attilio/memstat_maxcpu.diff > > It is going to add some memory overhead for i386 case. > It looks good as a temporal fix. I'm running with this patch for several days (I bumped MEMSTAT_MAXCPU to 128 though). -- wbr, pluknet
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAE-mSOJ87c4V_Py6-dMG2wqcnN78omgnHeupkdY7nwEf-dmntw>