[PATCH 4/8] Use vmalloc for large cgroups pidlist allocations
rientjes at google.com
Thu Aug 20 15:56:23 PDT 2009
On Thu, 20 Aug 2009, Jonathan Corbet wrote:
> On Thu, 20 Aug 2009 14:14:00 -0700
> Andrew Morton <akpm at linux-foundation.org> wrote:
> > Hang on. Isn't this why Dave just wrote and I just rush-merged
> > lib/flex_array.c?
> > Was that code evaluated for this application and judged unsuitable? If so,
> > for what reason?
> Should it be helpful: I wrote an overview of the flex_array API here:
> I could format it up for addition to Documentation/ if people want.
It's definitely helpful for this use case, flex array can store 261,632
pid_t's on 64 bit.
Instead of reallocating new memory when the pidlist becomes smaller, it
would then be sufficient to simply shrink the flex array by freeing parts
members that are newly unused (which also avoids the oom possibility in
flex_array_shrink(struct flex_array *) would require element poisoning
when an element is cleared, but that's probably not far off anyways
because it has additional advantages such as preventing flex_array_get()
on an element that has not been put.
More information about the Containers