Re: [PATCH] MAX_MP_BUSSES increase

James Cleverdon (jamesclv@us.ibm.com)
Thu, 20 Dec 2001 14:29:32 -0800


On Thursday 20 December 2001 09:03 am, you wrote:
> mingo@elte.hu (Ingo Molnar) writes:
> > On Wed, 19 Dec 2001, Linus Torvalds wrote:
> > > > Marcello and Linus, please apply.
> > >
> > > Can you give a rough description of what kinds of arrays this will
> > > impact, just out of curiosity. Ie do we talk about "5kB more memory in
> > > order to avoid problems", or are we talking about something
> > > noticeable..
> > >
> > > Linus "too lazy to grep" Torvalds
> >
> > the change is OK, it's about +3K RAM used, on SMP kernels.
>
> hmm, I come more to 11K (most of it in mp_irqs)
> ... which could be easily allocated with the bootmem allocator at parse
> time.
>
> -Andi

Thanks to all who replied. My rationale for simply increasing the size of
static arrays was to have a minimum impact on 2.4, as well as to make
something that Cannot Fail(TM). If you like, I could make one for 2.5 that
would do an initial scan of the MPS table, allocate the arrays using the
bootmem allocator, then go about its business as usual. (Special offer for a
limited time only! mpc_* array overflow checking added at NO EXTRA CHARGE!!
;^)

The catch with bootmem allocation is that it only allocates in pages (unless
wli's new bootmem allocator is adopted). So, expect some extra memory to be
lost to internal fragmentation anyway.

Another suggestion through private mail was to make MAX_MP_BUSSES a tunable
config parameter. I didn't know about that. Early boot stuff should work
without fuss, not rely on config tweaks. At the very least, I'd have to add
array overflow checking, because this crashes before the console is opened or
kdb is initialized. Silent crashes like that are bad news.

-- 
James Cleverdon, IBM xSeries Platform (NUMA), Beaverton
jamesclv@us.ibm.com   |   cleverdj@us.ibm.com

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/